required changes

Author: beercsaba

Extensions/Xtensive.Orm.Localization.Tests/Model/DictionaryModel.cs

@@ -1,4 +1,7 @@
-using System;
+// Copyright (C) 2020 Xtensive LLC.
+// This code is distributed under MIT license terms.
+// See the License.txt file in the project root for more information.
+
 using System.Globalization;
 using Xtensive.Orm.Model;
 
@@ -8,15 +11,15 @@ namespace Xtensive.Orm.Localization.Tests.Model
   public abstract class AbstractDictionary : Entity
   {
     [Key, Field]
-    public int Id { get; set; }
+    public int Id { get; private set; }
 
     [Field(Nullable = false, Length = 64)]
     public string Identifier { get; set; }
 
     // abstract property
     public abstract string Name { get; set; }
 
-    public AbstractDictionary(Session session) : base(session) { }
+    protected AbstractDictionary(Session session) : base(session) { }
   }
 
   public abstract class AbstractLocalizableDictionary<T, TT> : AbstractDictionary, ILocalizable<TT>
@@ -28,7 +31,7 @@ public abstract class AbstractLocalizableDictionary<T, TT> : AbstractDictionary,
     [Field]
     public LocalizationSet<TT> Localizations { get; private set; }
 
-    public AbstractLocalizableDictionary(Session session) : base(session) { }
+    protected AbstractLocalizableDictionary(Session session) : base(session) { }
   }
 
   public abstract class AbstractDictionaryLocalization<T, TT> : Localization<T>

Extensions/Xtensive.Orm.Localization.Tests/ProjectionToCustomTypesTests.cs

@@ -1,3 +1,7 @@
+// Copyright (C) 2020 Xtensive LLC.
+// This code is distributed under MIT license terms.
+// See the License.txt file in the project root for more information.
+
 using System.Linq;
 using System.Threading;
 using NUnit.Framework;
@@ -10,9 +14,8 @@ public class ProjectionToCustomTypesTests : AutoBuildTest
   {
     protected override void PopulateDatabase()
     {
-      using (var session = Domain.OpenSession()) {
-        using (var ts = session.OpenTransaction()) {
-
+      using (var session = Domain.OpenSession()) 
+      using (var ts = session.OpenTransaction()) {
           // populating database
           var m1 = new Country(session) {
             Identifier = "HUN",
@@ -22,22 +25,22 @@ protected override void PopulateDatabase()
             Identifier = "RUS",
             Name = "Oroszország"
           };
-          using (new LocalizationScope(EnglishCulture))
+          using (new LocalizationScope(EnglishCulture)) {
             m2.Name = "Russia";
-          using (new LocalizationScope(SpanishCulture))
-            m2.Name = "Rusia";
-
-          ts.Complete();
-        }
+          }
+          using (new LocalizationScope(SpanishCulture)) {
+            m2.Name = "Rusia";  
+          }
+        ts.Complete();
       }
     }
 
     [Test]
     public void EntityHierarchyWithAbstractPropertyTest()
     {
       Thread.CurrentThread.CurrentCulture = EnglishCulture;
-      using (var session = Domain.OpenSession()) {
-        using (var ts = session.OpenTransaction()) {
+      using (var session = Domain.OpenSession()) 
+      using (var ts = session.OpenTransaction()) {
           var q = session.Query.All<Country>().OrderBy(e => e.Identifier).Select(e => new { e.Name });
           var l = q.ToList();
           // assertions
@@ -47,9 +50,7 @@ public void EntityHierarchyWithAbstractPropertyTest()
           Assert.AreEqual(propertyInfos.First().Name, nameof(Country.Name));
           Assert.AreEqual(l.First().Name, "Magyarország");
           Assert.AreEqual(l.Last().Name, "Russia");
-        }
       }
     }
-
   }
 }