From 1283bf7349ba6bbfc09d774d01c768fe1fe18af6 Mon Sep 17 00:00:00 2001 From: "arthur.bassi" Date: Thu, 13 Aug 2026 13:45:25 -0300 Subject: [PATCH] fix(tests): align IsAddOn board tests with account-scoped dev APIs Pass ClaimsPrincipal and account resolver args required after rebase onto dev. --- .../WorkOrderBoardCreateServiceTests.cs | 8 ++++---- SeaHavenIndustries.Tests/WorkOrderBoardSearchTests.cs | 6 ++++-- SeaHavenIndustries.Tests/WorkOrderBoardServiceTests.cs | 6 ++++-- 3 files changed, 12 insertions(+), 8 deletions(-) diff --git a/SeaHavenIndustries.Tests/WorkOrderBoardCreateServiceTests.cs b/SeaHavenIndustries.Tests/WorkOrderBoardCreateServiceTests.cs index 6c01b17..16cd08d 100644 --- a/SeaHavenIndustries.Tests/WorkOrderBoardCreateServiceTests.cs +++ b/SeaHavenIndustries.Tests/WorkOrderBoardCreateServiceTests.cs @@ -364,7 +364,7 @@ public class WorkOrderBoardCreateServiceTests SiteCode = "BK5", ScheduledDate = new DateTime(2026, 6, 24), IsAddOn = false - }, "actor-1"); + }, WorkOrderAccountTestHelpers.AccountUser(), "actor-1"); Assert.True(result.IsAddOn); Assert.Equal(WorkOrderType.PM, result.WorkOrderType); @@ -385,7 +385,7 @@ public class WorkOrderBoardCreateServiceTests SiteCode = "BK5", ScheduledDate = new DateTime(2099, 3, 10), IsAddOn = true - }, "actor-1"); + }, WorkOrderAccountTestHelpers.AccountUser(), "actor-1"); Assert.False(result.IsAddOn); Assert.Equal(WorkOrderType.Reactive, result.WorkOrderType); @@ -403,7 +403,7 @@ public class WorkOrderBoardCreateServiceTests WorkOrderType = WorkOrderType.PM, SiteCode = "BK5", IsAddOn = true - }, "actor-1"); + }, WorkOrderAccountTestHelpers.AccountUser(), "actor-1"); Assert.True(result.IsAddOn); var wo = await context.workOrders.SingleAsync(w => w.Id == result.Id); @@ -420,7 +420,7 @@ public class WorkOrderBoardCreateServiceTests { WorkOrderType = WorkOrderType.AddOn, SiteCode = "BK5" - }, "actor-1")); + }, WorkOrderAccountTestHelpers.AccountUser(), "actor-1")); Assert.Contains(ex.Errors, e => e.PropertyName == "WorkOrderType"); } diff --git a/SeaHavenIndustries.Tests/WorkOrderBoardSearchTests.cs b/SeaHavenIndustries.Tests/WorkOrderBoardSearchTests.cs index 9ba1845..79b6f7c 100644 --- a/SeaHavenIndustries.Tests/WorkOrderBoardSearchTests.cs +++ b/SeaHavenIndustries.Tests/WorkOrderBoardSearchTests.cs @@ -652,7 +652,9 @@ public class WorkOrderAdvancedSearchServiceTests await context.SaveChangesAsync(); var dataService = new WorkOrderAdvancedSearchDataService(context); - var service = new SeaHaven.Services.Implementation.WorkOrderAdvancedSearchService(dataService); + var service = new SeaHaven.Services.Implementation.WorkOrderAdvancedSearchService( + dataService, + WorkOrderAccountTestHelpers.Resolver(context)); var result = await service.SearchAsync(new SeaHaven.Services.DTOs.WorkOrderAdvancedSearchQueryDto { @@ -660,7 +662,7 @@ public class WorkOrderAdvancedSearchServiceTests DateFrom = DateOnly.FromDateTime(today.AddMonths(-1)), DateTo = DateOnly.FromDateTime(today.AddMonths(1)), Types = new List { WorkOrderType.AddOn } - }, null); + }, WorkOrderAccountTestHelpers.OrgWideAdmin(), null); Assert.Equal(2, result.TotalCount); Assert.Contains(result.Items, i => i.Id == 1 && i.WorkOrderType == WorkOrderType.AddOn); diff --git a/SeaHavenIndustries.Tests/WorkOrderBoardServiceTests.cs b/SeaHavenIndustries.Tests/WorkOrderBoardServiceTests.cs index 9d82c8c..32a890c 100644 --- a/SeaHavenIndustries.Tests/WorkOrderBoardServiceTests.cs +++ b/SeaHavenIndustries.Tests/WorkOrderBoardServiceTests.cs @@ -430,11 +430,13 @@ public class WorkOrderBoardServiceTests await context.SaveChangesAsync(); - var boardService = new WorkOrderBoardService(new WorkOrderBoardDataService(context)); + var boardService = new WorkOrderBoardService( + new WorkOrderBoardDataService(context), + WorkOrderAccountTestHelpers.Resolver(context)); var response = await boardService.GetBoardAsync(new WorkOrderBoardQueryDto { WeekStart = weekStart - }, null); + }, WorkOrderAccountTestHelpers.OrgWideAdmin(), null); Assert.Single(response.Scheduled); Assert.True(response.Scheduled[0].IsAddOn);