Skip to content

Commit f48371d

Browse files
committed
chore: Resolve PR feedback
1 parent 76c01c2 commit f48371d

File tree

3 files changed

+22
-5
lines changed

3 files changed

+22
-5
lines changed

platform/src/apis/Platform.Api.LocalAuthorityFinances/Features/HighNeeds/Services/HighNeedsService.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ public class HighNeedsService(IDatabaseFactory dbFactory) : HighNeedsStubService
124124
return results.ToArray();
125125
}
126126

127-
internal static LocalAuthority<Models.HighNeeds> MultiMapToHighNeeds(object[] objects)
127+
private static LocalAuthority<Models.HighNeeds> MultiMapToHighNeeds(object[] objects)
128128
{
129129
var localAuthority = objects[0] as LocalAuthorityBase;
130130
var outturn = objects[1] as HighNeedsBase;

platform/src/apis/Platform.Api.LocalAuthorityFinances/Program.cs

-2
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,8 @@
11
using System.Diagnostics.CodeAnalysis;
2-
using System.Runtime.CompilerServices;
32
using Microsoft.Azure.Functions.Worker.Extensions.OpenApi.Extensions;
43
using Microsoft.Extensions.Hosting;
54
using Platform.Api.LocalAuthorityFinances.Configuration;
65

7-
[assembly: InternalsVisibleTo("Platform.LocalAuthorityFinances.Tests")]
86
var hostBuilder = new HostBuilder()
97
.ConfigureFunctionsWorkerDefaults(Worker.Configure, Worker.Options)
108
.ConfigureServices(Services.Configure)

platform/tests/Platform.LocalAuthorityFinances.Tests/HighNeeds/Services/WhenHighNeedsServiceQueriesAsync.cs

+21-2
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ public async Task ShouldQueryAndMultiMapWhenGetWithValidDimension()
6363
}
6464

6565
[Fact]
66-
public void ShouldMapWhenMultiMapToHighNeeds()
66+
public async Task ShouldMapWhenMultiMapToHighNeeds()
6767
{
6868
var localAuthority = _fixture.Create<LocalAuthorityBase>();
6969
var outturn = _fixture.Create<HighNeedsBase>();
@@ -91,8 +91,27 @@ public void ShouldMapWhenMultiMapToHighNeeds()
9191
budgetPlaceFunding
9292
];
9393

94+
// arrange
95+
string[] codes = ["code1", "code2", "code3"];
96+
const string dimension = "Actuals";
97+
var results = _fixture
98+
.Build<LocalAuthority<Api.LocalAuthorityFinances.Features.HighNeeds.Models.HighNeeds>>()
99+
.CreateMany()
100+
.ToArray();
101+
102+
Func<object[], LocalAuthority<Api.LocalAuthorityFinances.Features.HighNeeds.Models.HighNeeds>> mapper = _ => new LocalAuthority<Api.LocalAuthorityFinances.Features.HighNeeds.Models.HighNeeds>();
103+
_connection
104+
.Setup(c => c.QueryAsync(It.IsAny<PlatformQuery>(), It.IsAny<Type[]>(), It.IsAny<Func<object[], LocalAuthority<Api.LocalAuthorityFinances.Features.HighNeeds.Models.HighNeeds>>>(), It.IsAny<string[]>()))
105+
.Callback<PlatformQuery, Type[], Func<object[], LocalAuthority<Api.LocalAuthorityFinances.Features.HighNeeds.Models.HighNeeds>>, string[]>((_, _, map, _) =>
106+
{
107+
mapper = map;
108+
})
109+
.ReturnsAsync(results);
110+
111+
await _service.Get(codes, dimension);
112+
94113
// act
95-
var actual = HighNeedsService.MultiMapToHighNeeds(objects);
114+
var actual = mapper(objects);
96115

97116
// assert
98117
Assert.Equal(localAuthority.Code, actual.Code);

0 commit comments

Comments
 (0)