Modernize the test case example

fredericDelaporte · Feb 6, 2024 · 102541e · 102541e
1 parent a958ff1
commit 102541e
Show file tree

Hide file tree

Showing 2 changed files with 55 additions and 59 deletions.
diff --git a/src/NHibernate.Test/NHSpecificTest/GH0000/Fixture.cs b/src/NHibernate.Test/NHSpecificTest/GH0000/Fixture.cs
@@ -8,47 +8,45 @@ public class Fixture : BugTestCase
 	{
 		protected override void OnSetUp()
 		{
-			using (var session = OpenSession())
-			using (var transaction = session.BeginTransaction())
-			{
-				var e1 = new Entity {Name = "Bob"};
-				session.Save(e1);
+			using var session = OpenSession();
+			using var transaction = session.BeginTransaction();
 
-				var e2 = new Entity {Name = "Sally"};
-				session.Save(e2);
+			var e1 = new Entity { Name = "Bob" };
+			session.Save(e1);
 
-				transaction.Commit();
-			}
+			var e2 = new Entity { Name = "Sally" };
+			session.Save(e2);
+
+			transaction.Commit();
 		}
 
 		protected override void OnTearDown()
 		{
-			using (var session = OpenSession())
-			using (var transaction = session.BeginTransaction())
-			{
-				// The HQL delete does all the job inside the database without loading the entities, but it does
-				// not handle delete order for avoiding violating constraints if any. Use
-				// session.Delete("from System.Object");
-				// instead if in need of having NHibernate ordering the deletes, but this will cause
-				// loading the entities in the session.
-				session.CreateQuery("delete from System.Object").ExecuteUpdate();
-
-				transaction.Commit();
-			}
+			using var session = OpenSession();
+			using var transaction = session.BeginTransaction();
+
+			// The HQL delete does all the job inside the database without loading the entities, but it does
+			// not handle delete order for avoiding violating constraints if any. Use
+			// session.Delete("from System.Object");
+			// instead if in need of having NHibernate ordering the deletes, but this will cause
+			// loading the entities in the session.
+			session.CreateQuery("delete from System.Object").ExecuteUpdate();
+
+			transaction.Commit();
 		}
 
 		[Test]
 		public void YourTestName()
 		{
-			using (var session = OpenSession())
-			using (session.BeginTransaction())
-			{
-				var result = from e in session.Query<Entity>()
-							 where e.Name == "Bob"
-							 select e;
-
-				Assert.That(result.ToList(), Has.Count.EqualTo(1));
-			}
+			using var session = OpenSession();
+			using var transaction = session.BeginTransaction();
+
+			var result = session
+				.Query<Entity>()
+				.Where(e => e.Name == "Bob");
+			Assert.That(result.ToList(), Has.Count.EqualTo(1));
+
+			transaction.Commit();
 		}
 	}
 }
diff --git a/src/NHibernate.Test/NHSpecificTest/GH0000/FixtureByCode.cs b/src/NHibernate.Test/NHSpecificTest/GH0000/FixtureByCode.cs
@@ -30,48 +30,46 @@ protected override HbmMapping GetMappings()
 
 		protected override void OnSetUp()
 		{
-			using (var session = OpenSession())
-			using (var transaction = session.BeginTransaction())
-			{
-				var e1 = new Entity { Name = "Bob" };
-				session.Save(e1);
+			using var session = OpenSession();
+			using var transaction = session.BeginTransaction();
+
+			var e1 = new Entity { Name = "Bob" };
+			session.Save(e1);
 
-				var e2 = new Entity { Name = "Sally" };
-				session.Save(e2);
+			var e2 = new Entity { Name = "Sally" };
+			session.Save(e2);
 
-				transaction.Commit();
-			}
+			transaction.Commit();
 		}
 
 		protected override void OnTearDown()
 		{
-			using (var session = OpenSession())
-			using (var transaction = session.BeginTransaction())
-			{
-				// The HQL delete does all the job inside the database without loading the entities, but it does
-				// not handle delete order for avoiding violating constraints if any. Use
-				// session.Delete("from System.Object");
-				// instead if in need of having NHbernate ordering the deletes, but this will cause
-				// loading the entities in the session.
-				session.CreateQuery("delete from System.Object").ExecuteUpdate();
+			using var session = OpenSession();
+			using var transaction = session.BeginTransaction();
+
+			// The HQL delete does all the job inside the database without loading the entities, but it does
+			// not handle delete order for avoiding violating constraints if any. Use
+			// session.Delete("from System.Object");
+			// instead if in need of having NHbernate ordering the deletes, but this will cause
+			// loading the entities in the session.
+			session.CreateQuery("delete from System.Object").ExecuteUpdate();
 
-				transaction.Commit();
-			}
+			transaction.Commit();
 		}
 
 		[Test]
 		public void YourTestName()
 		{
-			using (var session = OpenSession())
-			using (var transaction = session.BeginTransaction())
-			{
-				var result = from e in session.Query<Entity>()
-							 where e.Name == "Bob"
-							 select e;
+			using var session = OpenSession();
+			using var transaction = session.BeginTransaction();
+
+			var result = session
+				.Query<Entity>()
+				.Where(e => e.Name == "Bob");
+
+			Assert.That(result.ToList(), Has.Count.EqualTo(1));
 
-				Assert.That(result.ToList(), Has.Count.EqualTo(1));
-				transaction.Commit();
-			}
+			transaction.Commit();
 		}
 	}
 }