Move location route

src/CarbonAware.WebApi/src/Controllers/CarbonAwareController.cs

@@ -241,18 +241,4 @@ public async Task<IActionResult> GetAverageCarbonIntensityBatch([FromBody] IEnum
             return Ok(result);
         }
     }
-
-    /// <summary>
-    /// Get all locations
-    /// </summary>
-    /// <returns>Dictionary with the locations</returns>
-    [Produces("application/json")]
-    [ProducesResponseType(StatusCodes.Status200OK, Type = typeof(IDictionary<string, Location>))]
-    [ProducesResponseType(StatusCodes.Status204NoContent)]
-    [HttpGet("locations")]
-    public async Task<IActionResult> GetAllLocations()
-    {
-        var response = await _locationSource.GetGeopositionLocationsAsync(); 
-        return response.Any() ? Ok(response) : NoContent();
-    }
 }

src/CarbonAware.WebApi/src/Controllers/LocationsController.cs

@@ -0,0 +1,35 @@
+using CarbonAware.Interfaces;
+using CarbonAware.Model;
+using Microsoft.AspNetCore.Mvc;
+
+namespace CarbonAware.WebApi.Controllers;
+
+[ApiController]
+[Route("locations")]
+public class LocationsController : ControllerBase
+{
+    private readonly ILogger<CarbonAwareController> _logger;
+
+    private readonly ILocationSource _locationSource;
+
+
+    public LocationsController(ILogger<CarbonAwareController> logger, ILocationSource locationSouce)
+    {
+        _logger = logger ?? throw new ArgumentNullException(nameof(logger));
+        _locationSource = locationSouce ?? throw new ArgumentNullException(nameof(locationSouce));
+    }
+
+    /// <summary>
+    /// Get all locations instances
+    /// </summary>
+    /// <returns>Dictionary with <see cref="Location"/> instances</returns>
+    [Produces("application/json")]
+    [ProducesResponseType(StatusCodes.Status200OK, Type = typeof(IDictionary<string, Location>))]
+    [ProducesResponseType(StatusCodes.Status204NoContent)]
+    [HttpGet()]
+    public async Task<IActionResult> GetAllLocations()
+    {
+        var response = await _locationSource.GetGeopositionLocationsAsync(); 
+        return response.Any() ? Ok(response) : NoContent();
+    }
+}

src/CarbonAware.WebApi/test/integrationTests/CarbonAwareControllerTests.cs

@@ -24,7 +24,7 @@ public class CarbonAwareControllerTests : IntegrationTestingBase
     private readonly string batchForecastURI = "/emissions/forecasts/batch";
     private readonly string averageCarbonIntensityURI = "/emissions/average-carbon-intensity";
     private readonly string batchAverageCarbonIntensityURI = "/emissions/average-carbon-intensity/batch";
-    private readonly string locationsURI = "/emissions/locations";
+
 
     public CarbonAwareControllerTests(DataSourceType dataSource) : base(dataSource) { }
 
@@ -306,21 +306,6 @@ public async Task EmissionsMarginalCarbonIntensityBatch_SupportedDataSources_Ret
         }
     }
 
-    [Test]
-    public async Task GetLocations_ReturnsOk()
-    {
-        var result = await _client.GetAsync(locationsURI);
-        Assert.That(result, Is.Not.Null);
-        Assert.That(result.StatusCode, Is.EqualTo(HttpStatusCode.OK));
-        Assert.That(result.Content, Is.Not.Null);
-        using var stream = await result.Content.ReadAsStreamAsync();
-        Assert.That(stream, Is.Not.Null);
-        var data = await JsonSerializer.DeserializeAsync<IDictionary<string, dynamic>>(stream);
-        Assert.That(data, Is.Not.Null);
-        Assert.That(data!.ContainsKey("eastus"), Is.True);
-        Assert.That(data!.ContainsKey("northeurope"), Is.True);
-    }
-
     private void IgnoreTestForDataSource(string reasonMessage, params DataSourceType[] ignoredDataSources)
     {
         if (ignoredDataSources.Contains(_dataSource))

src/CarbonAware.WebApi/test/integrationTests/LocationsControllerTests.cs

@@ -0,0 +1,36 @@
+using CarbonAware.DataSources.Configuration;
+using CarbonAware.WebApi.IntegrationTests;
+using NUnit.Framework;
+using System.Net;
+using System.Text.Json;
+
+namespace CarbonAware.WepApi.IntegrationTests;
+
+/// <summary>
+/// Tests that the Web API controller handles locations instances 
+/// </summary>
+[TestFixture(DataSourceType.JSON)]
+[TestFixture(DataSourceType.WattTime)]
+[TestFixture(DataSourceType.ElectricityMaps)]
+public class LocationsControllerTests : IntegrationTestingBase
+{
+    private readonly string locationsURI = "/locations";
+
+    public LocationsControllerTests(DataSourceType dataSource) : base(dataSource) { }
+
+
+    [Test]
+    public async Task GetLocations_ReturnsOk()
+    {
+        var result = await _client.GetAsync(locationsURI);
+        Assert.That(result, Is.Not.Null);
+        Assert.That(result.StatusCode, Is.EqualTo(HttpStatusCode.OK));
+        Assert.That(result.Content, Is.Not.Null);
+        using var stream = await result.Content.ReadAsStreamAsync();
+        Assert.That(stream, Is.Not.Null);
+        var data = await JsonSerializer.DeserializeAsync<IDictionary<string, dynamic>>(stream);
+        Assert.That(data, Is.Not.Null);
+        Assert.That(data!.ContainsKey("eastus"), Is.True);
+        Assert.That(data!.ContainsKey("northeurope"), Is.True);
+    }
+}

src/CarbonAware.WebApi/test/unitTests/CarbonAwareControllerTests.cs

@@ -238,32 +238,4 @@ public void GetEmissionsDataForLocationsByTime_NoLocations_ThrowsException(param
 
         Assert.ThrowsAsync<ArgumentException>(async () => await controller.GetEmissionsDataForLocationsByTime(parametersDTO));
     }
-
-    /// <summary>
-    /// Test result with content
-    /// </summary>
-    [Test]
-    public async Task GetLocations_ResultsOk()
-    {
-        var controller = new CarbonAwareController(this.MockCarbonAwareLogger.Object, CreateEmissionsAggregator(new List<EmissionsData>()).Object, forecastAggregator, CreateLocations().Object);
-
-        IActionResult result = await controller.GetAllLocations();
-
-        //Assert
-        TestHelpers.AssertStatusCode(result, HttpStatusCode.OK);
-    }
-
-    /// <summary>
-    /// Test result without content
-    /// </summary>
-    [Test]
-    public async Task GetLocations_ResultsNoContent()
-    {
-        var controller = new CarbonAwareController(this.MockCarbonAwareLogger.Object, CreateEmissionsAggregator(new List<EmissionsData>()).Object, forecastAggregator, CreateLocations(false).Object);
-
-        IActionResult result = await controller.GetAllLocations();
-
-        //Assert
-        TestHelpers.AssertStatusCode(result, HttpStatusCode.NoContent);
-    }
 }

src/CarbonAware.WebApi/test/unitTests/LocationsControllerTests.cs

@@ -0,0 +1,41 @@
+using CarbonAware.WebApi.Controllers;
+using Microsoft.AspNetCore.Mvc;
+using NUnit.Framework;
+using System.Net;
+
+namespace CarbonAware.WepApi.UnitTests;
+
+/// <summary>
+/// Tests that the Web API controller handles request for Location instances 
+/// </summary>
+[TestFixture]
+public class LocationsControllerTests : TestsBase
+{
+    /// <summary>
+    /// Test result with content
+    /// </summary>
+    [Test]
+    public async Task GetLocations_ResultsOk()
+    {
+        var controller = new LocationsController(this.MockCarbonAwareLogger.Object, CreateLocations().Object);
+
+        IActionResult result = await controller.GetAllLocations();
+
+        //Assert
+        TestHelpers.AssertStatusCode(result, HttpStatusCode.OK);
+    }
+
+    /// <summary>
+    /// Test result without content
+    /// </summary>
+    [Test]
+    public async Task GetLocations_ResultsNoContent()
+    {
+        var controller = new LocationsController(this.MockCarbonAwareLogger.Object, CreateLocations(false).Object);
+
+        IActionResult result = await controller.GetAllLocations();
+
+        //Assert
+        TestHelpers.AssertStatusCode(result, HttpStatusCode.NoContent);
+    }
+}