Fix integration test review comment

Green-Software-Foundation · Jan 9, 2023 · e24ddc2 · e24ddc2
1 parent 349a65e
commit e24ddc2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/CarbonAware.CLI/test/integrationTests/Commands/Location/LocationCommandTests.cs b/src/CarbonAware.CLI/test/integrationTests/Commands/Location/LocationCommandTests.cs
@@ -5,8 +5,7 @@
 namespace CarbonAware.CLI.IntegrationTests.Commands.Location;
 
 /// <summary>
-/// Tests that the CLI handles and packages various responses from aggregators 
-/// and data sources properly, including empty responses and exceptions.
+/// Tests that the CLI handles responses from location sources.
 /// </summary>
 [TestFixture(DataSourceType.JSON)]
 [TestFixture(DataSourceType.WattTime)]
@@ -25,7 +24,7 @@ public async Task Locations_Help_ReturnsHelpText()
     }
 
     [Test]
-    public async Task Locations_ReturnsExpectedData()
+    public async Task Locations_ReturnsKnownData()
     {
         // Act
         var exitCode = await InvokeCliAsync("locations");

diff --git a/src/CarbonAware.LocationSources/src/LocationSource.cs b/src/CarbonAware.LocationSources/src/LocationSource.cs
@@ -126,7 +126,7 @@ private void AddToLocationMap(string key, NamedGeoposition data, string sourceFi
         var counter = keyCounter[key];
         counter++;
         var newKey = $"{key}_{counter}";
-        _logger.LogDebug("New key {newKey} generated from {key}", newKey, key);
+        _logger.LogWarning("New key {newKey} generated from {key}", newKey, key);
         _allLocations.Add(newKey, loc);
         keyCounter[key] = counter;
     }