Skip to content

Commit b6dc796

Browse files
authored
Merge pull request #5165 from dfe-analytical-services/EES-4670-revert
EES-4670 Revert Data Catalogue code removal
2 parents 81e0c79 + c22471e commit b6dc796

File tree

4 files changed

+54
-4
lines changed

4 files changed

+54
-4
lines changed

src/GovUk.Education.ExploreEducationStatistics.Content.Api.Tests/Controllers/PublicationControllerTests.cs

+2-2
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public async Task GetPublicationTree()
7373
var controller = BuildPublicationController(publicationCacheService: publicationCacheService.Object);
7474

7575
publicationCacheService
76-
.Setup(s => s.GetPublicationTree(PublicationTreeFilter.FastTrack))
76+
.Setup(s => s.GetPublicationTree(PublicationTreeFilter.DataCatalogue))
7777
.ReturnsAsync(new List<PublicationTreeThemeViewModel>
7878
{
7979
new()
@@ -91,7 +91,7 @@ public async Task GetPublicationTree()
9191
}
9292
});
9393

94-
var result = await controller.GetPublicationTree(PublicationTreeFilter.FastTrack);
94+
var result = await controller.GetPublicationTree(PublicationTreeFilter.DataCatalogue);
9595

9696
VerifyAllMocks(publicationCacheService);
9797

src/GovUk.Education.ExploreEducationStatistics.Content.Services.Tests/Cache/PublicationCacheServiceTests.cs

+48-2
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ public async Task GetPublicationTree_NoCachedTreeExists()
178178

179179
var service = BuildService(publicationService.Object);
180180

181-
var result = await service.GetPublicationTree(PublicationTreeFilter.FastTrack);
181+
var result = await service.GetPublicationTree(PublicationTreeFilter.DataCatalogue);
182182

183183
VerifyAllMocks(PublicBlobCacheService);
184184

@@ -213,14 +213,60 @@ public async Task GetPublicationTree_CachedTreeExists()
213213

214214
var service = BuildService();
215215

216-
var result = await service.GetPublicationTree(PublicationTreeFilter.FastTrack);
216+
var result = await service.GetPublicationTree(PublicationTreeFilter.DataCatalogue);
217217

218218
VerifyAllMocks(PublicBlobCacheService);
219219

220220
var filteredTree = result.AssertRight();
221221
filteredTree.AssertDeepEqualTo(ListOf(publicationTree));
222222
}
223223

224+
[Fact]
225+
public async Task GetPublicationTree_DataCatalogue_SomeLiveReleaseHasData_Included()
226+
{
227+
var publicationTree = new PublicationTreeThemeViewModel
228+
{
229+
Title = "Theme A",
230+
Topics = new List<PublicationTreeTopicViewModel>
231+
{
232+
new()
233+
{
234+
Title = "Topic A",
235+
Publications = ListOf(new PublicationTreePublicationViewModel
236+
{
237+
Title = "Publication A",
238+
AnyLiveReleaseHasData = true
239+
})
240+
}
241+
}
242+
};
243+
244+
await AssertPublicationTreeUnfiltered(publicationTree, PublicationTreeFilter.DataCatalogue);
245+
}
246+
247+
[Fact]
248+
public async Task GetPublicationTree_DataCatalogue_NoLiveReleaseHasData_Excluded()
249+
{
250+
var publicationTree = new PublicationTreeThemeViewModel
251+
{
252+
Title = "Theme A",
253+
Topics = new List<PublicationTreeTopicViewModel>
254+
{
255+
new()
256+
{
257+
Title = "Topic A",
258+
Publications = ListOf(new PublicationTreePublicationViewModel
259+
{
260+
Title = "Publication A",
261+
AnyLiveReleaseHasData = false
262+
})
263+
}
264+
}
265+
};
266+
267+
await AssertPublicationTreeEmpty(publicationTree, PublicationTreeFilter.DataCatalogue);
268+
}
269+
224270
[Fact]
225271
public async Task GetPublicationTree_DataTables_NonSupersededPublicationWithDataOnLatestRelease_Included()
226272
{

src/GovUk.Education.ExploreEducationStatistics.Content.Services/Cache/PublicationCacheService.cs

+1
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,7 @@ private static bool FilterPublicationTreePublication(
129129
{
130130
case PublicationTreeFilter.DataTables:
131131
return publication.LatestReleaseHasData;
132+
case PublicationTreeFilter.DataCatalogue:
132133
case PublicationTreeFilter.FastTrack:
133134
return publication.AnyLiveReleaseHasData;
134135
default:
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,11 @@
1+
#nullable enable
2+
13
namespace GovUk.Education.ExploreEducationStatistics.Content.Services.Requests
24
{
35
public enum PublicationTreeFilter
46
{
57
DataTables,
8+
DataCatalogue,
69
FastTrack,
710
}
811
}

0 commit comments

Comments
 (0)