Apply review suggestions

Author: dacharyc

usage-examples/csharp/driver/Examples/Aggregation/GroupTotal.cs

@@ -4,20 +4,20 @@
 //      "_aggDb": "aggDb"
 //	  }
 //	}
+
 namespace Examples.Aggregation;
 
 using MongoDB.Driver;
 using MongoDB.Bson;
 
 public class GroupTotal
 {
-    private IMongoDatabase _aggDB;
-    private IMongoCollection<Order> _orders;
+    private IMongoDatabase? _aggDB;
+    private IMongoCollection<Order>? _orders;
 
-    public void SeedData()
+    public void LoadSampleData()
     {
-        DotNetEnv.Env.TraversePath().Load();
-        string uri = DotNetEnv.Env.GetString("CONNECTION_STRING", "Env variable not found. Verify you have a .env file with a valid connection string.");
+        var uri = DotNetEnv.Env.GetString("CONNECTION_STRING", "Env variable not found. Verify you have a .env file with a valid connection string.");
         // :snippet-start: connection-string
         // :uncomment-start:
         //var uri = "mongodb+srv://mongodb-example:27017";
@@ -91,12 +91,10 @@ public void SeedData()
 
     public List<BsonDocument> PerformAggregation()
     {
-        DotNetEnv.Env.TraversePath().Load();
-        string uri = DotNetEnv.Env.GetString("CONNECTION_STRING", "Env variable not found. Verify you have a .env file with a valid connection string.");
-        //var uri = "mongodb://localhost:27017";
-        var client = new MongoClient(uri);
-        _aggDB = client.GetDatabase("agg_tutorials_db");
-        _orders = _aggDB.GetCollection<Order>("orders");
+        if (_aggDB == null || _orders == null)  
+        {  
+            throw new InvalidOperationException("You must call LoadSampleData before performing aggregation.");  
+        }
 
         // :snippet-start: match
         var results = _orders.Aggregate()

usage-examples/csharp/driver/Tests/GroupTotalTest.cs

@@ -4,27 +4,26 @@ namespace Tests;
 
 public class GroupTotalTest
 {
+    private GroupTotal _example;
     [SetUp]
     public void Setup()
     {
-        var obj = new GroupTotal();
-        obj.SeedData();
+        _example = new GroupTotal();
+        _example.LoadSampleData();
     }
 
     [Test]
     public void TestOutputMatchesDocs()
     {
-        var obj = new GroupTotal();
-        var results = obj.PerformAggregation();
+        var results = _example.PerformAggregation();
 
-        DotNetEnv.Env.TraversePath().Load();
-        string solutionRoot = DotNetEnv.Env.GetString("SOLUTION_ROOT", "Env variable not found. Verify you have a .env file with a valid connection string.");
-        string outputLocation = "Examples/Aggregation/GroupTotalOutput.txt";
-        string fullPath = Path.Combine(solutionRoot, outputLocation);
+        var solutionRoot = DotNetEnv.Env.GetString("SOLUTION_ROOT", "Env variable not found. Verify you have a .env file with a valid connection string.");
+        var outputLocation = "Examples/Aggregation/GroupTotalOutput.txt";
+        var fullPath = Path.Combine(solutionRoot, outputLocation);
         var fileData = TestUtils.ReadBsonDocumentsFromFile(fullPath);
 
-        Assert.That(results.Count, Is.EqualTo(fileData.Length), "Result count does not match output example length.");
-        for (int i = 0; i < fileData.Length; i++)  
+        Assert.That(results.Count, Is.EqualTo(fileData.Length), $"Result count {results.Count} does not match output example length {fileData.Length}.");
+        for (var i = 0; i < fileData.Length; i++)  
         {  
             Assert.That(fileData[i], Is.EqualTo(results[i]), $"Mismatch at index {i}: expected {fileData[i]}, got {results[i]}.");
         }

usage-examples/csharp/driver/Tests/TestSuiteSetup.cs

@@ -0,0 +1,13 @@
+namespace Tests;
+
+using DotNetEnv;  
+
+[SetUpFixture]  
+public class TestSuiteSetup  
+{  
+    [OneTimeSetUp]  
+    public void GlobalSetup()  
+    {  
+        Env.TraversePath().Load();  
+    }  
+}