Variable scope fix

HicServices · Jan 3, 2024 · b4e4160 · b4e4160
1 parent df1f918
commit b4e4160
Showing 1 changed file with 11 additions and 21 deletions.
diff --git a/Rdmp.Core.Tests/CommandExecution/ExecuteCommandRedactCHIsFromCatalogueTests.cs b/Rdmp.Core.Tests/CommandExecution/ExecuteCommandRedactCHIsFromCatalogueTests.cs
@@ -24,16 +24,13 @@ public void RedactCHIsFromCatalogue_ValidSingle()
         var biochemistry = CatalogueRepository.GetAllObjects<Catalogue>().Where(c => c.Name == "Biochemistry").First();
         var tbl = db.DiscoverTables(true).Where(dt => dt.GetRuntimeName().Contains("Biochemistry")).First();
         var updateSQL = $"UPDATE top (2) {tbl.GetRuntimeName()} set SampleType = 'F1111111111'";
-        using (var con = tbl.Database.Server.GetConnection())
-        {
-            con.Open();
-            var updateCmd = tbl.Database.Server.GetCommand(updateSQL, con);
-            updateCmd.ExecuteNonQuery();
-        }
+        using var con = tbl.Database.Server.GetConnection();
+        con.Open();
+        var updateCmd = tbl.Database.Server.GetCommand(updateSQL, con);
+        updateCmd.ExecuteNonQuery();
         var cmd = new ExecuteCommandRedactCHIsFromCatalogue(new ThrowImmediatelyActivator(RepositoryLocator), biochemistry, null);
         Assert.DoesNotThrow(() => cmd.Execute());
         using var dt = new DataTable();
-        using var con = tbl.Database.Server.GetConnection();
         using var findCmd = tbl.Database.Server.GetCommand($"select * from {tbl.GetRuntimeName()} where SampleType = 'F#########1'", con);
         using var da = tbl.Database.Server.GetDataAdapter(findCmd);
         da.Fill(dt);
@@ -52,27 +49,20 @@ public void RedactCHIsFromCatalogue_Allowlist()
         var biochemistry = CatalogueRepository.GetAllObjects<Catalogue>().Where(c => c.Name == "Biochemistry").First();
         var tbl = db.DiscoverTables(true).Where(dt => dt.GetRuntimeName().Contains("Biochemistry")).First();
         var updateSQL = $"UPDATE top (2) {tbl.GetRuntimeName()} set SampleType = 'F1111111111'";
-        using (var con = tbl.Database.Server.GetConnection())
-        {
-            con.Open();
-            var updateCmd = tbl.Database.Server.GetCommand(updateSQL, con);
-            updateCmd.ExecuteNonQuery();
-        }
+        using var con = tbl.Database.Server.GetConnection();
+        con.Open();
+        var updateCmd = tbl.Database.Server.GetCommand(updateSQL, con);
+        updateCmd.ExecuteNonQuery();
         var tempFileToCreate = Path.Combine(TestContext.CurrentContext.TestDirectory, "allowList.yaml");
         var allowList = File.Create(tempFileToCreate);
         allowList.Close();
         File.WriteAllLines(tempFileToCreate, new string[] { "Biochemistry:", "- SampleType" });
         var cmd = new ExecuteCommandRedactCHIsFromCatalogue(new ThrowImmediatelyActivator(RepositoryLocator), biochemistry, tempFileToCreate);
         Assert.DoesNotThrow(() => cmd.Execute());
         using var dt = new DataTable();
-        using (var con = tbl.Database.Server.GetConnection())
-        {
-            using (var findCmd = tbl.Database.Server.GetCommand($"select * from {tbl.GetRuntimeName()} where SampleType = 'F#########1'", con))
-            {
-                using var da = tbl.Database.Server.GetDataAdapter(findCmd);
-                da.Fill(dt);
-            }
-        }
+        using (var findCmd = tbl.Database.Server.GetCommand($"select * from {tbl.GetRuntimeName()} where SampleType = 'F#########1'", con))
+        using var da = tbl.Database.Server.GetDataAdapter(findCmd);
+        da.Fill(dt);
         Assert.That(dt.Rows.Count, Is.EqualTo(0));
     }
 }