Fixing broken work instruction editor service unit test (LoadForEditAsync_NotFound_ShouldThrow) to comply with exception type changed to KeyNotFoundException

EyeBrawler · EyeBrawler · commit 9ee7ae557e41 · 2026-04-14T11:25:01.000-05:00
diff --git a/MESS/MESS.Tests/Services/WorkInstructionEditorServiceTests.cs b/MESS/MESS.Tests/Services/WorkInstructionEditorServiceTests.cs
@@ -84,9 +84,13 @@ public async Task LoadForEditAsync_ShouldLoadExistingInstruction()
     [Fact]
     public async Task LoadForEditAsync_NotFound_ShouldThrow()
     {
-        _mockWorkInstructionService.Setup(s => s.GetFormByIdAsync(99)).ReturnsAsync((WorkInstructionFormDTO?)null);
+        _mockWorkInstructionService
+            .Setup(s => s.GetFormByIdAsync(99))
+            .ReturnsAsync((WorkInstructionFormDTO?)null);
+
+        var ex = await Assert.ThrowsAsync<KeyNotFoundException>(() => _sut.LoadForEditAsync(99));
 
-        await Assert.ThrowsAsync<Exception>(() => _sut.LoadForEditAsync(99));
+        Assert.Equal("Work instruction with id 99 was not found.", ex.Message);
     }
 
     [Fact]

Original file line number	Diff line number	Diff line change
`@@ -84,9 +84,13 @@ public async Task LoadForEditAsync_ShouldLoadExistingInstruction()`
`84`	`84`	`[Fact]`
`85`	`85`	`public async Task LoadForEditAsync_NotFound_ShouldThrow()`
`86`	`86`	`{`
`87`		`- _mockWorkInstructionService.Setup(s => s.GetFormByIdAsync(99)).ReturnsAsync((WorkInstructionFormDTO?)null);`
	`87`	`+ _mockWorkInstructionService`
	`88`	`+ .Setup(s => s.GetFormByIdAsync(99))`
	`89`	`+ .ReturnsAsync((WorkInstructionFormDTO?)null);`
	`90`	`+`
	`91`	`+ var ex = await Assert.ThrowsAsync<KeyNotFoundException>(() => _sut.LoadForEditAsync(99));`
`88`	`92`
`89`		`- await Assert.ThrowsAsync<Exception>(() => _sut.LoadForEditAsync(99));`
	`93`	`+ Assert.Equal("Work instruction with id 99 was not found.", ex.Message);`
`90`	`94`	`}`
`91`	`95`
`92`	`96`	`[Fact]`