Skip to content

Commit 37de1a5

Browse files
Copilotkbeaugrand
andcommitted
Improve test comment clarity based on code review feedback
Co-authored-by: kbeaugrand <9513635+kbeaugrand@users.noreply.github.com>
1 parent cac7407 commit 37de1a5

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/IoTHub.Portal.Tests.Unit/Client/Dialogs/Layer/LinkDeviceLayerDialogTest.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -244,13 +244,13 @@ public async Task LinkDeviceLayerDialog_UpdateChecked_UnselectingAlreadyRegister
244244
checkboxButton.Click();
245245

246246
// Assert - after clicking, the checkbox should visually update
247-
// The LayerId is set to null, so the condition should no longer match "Already registered"
248-
// and should show either the "Add device" state or intermediate state
247+
// The LayerId is set to null and the device is added to DeviceRemoveList,
248+
// so the condition (context.LayerId == InitLayer.Id) no longer matches.
249+
// This should trigger the else branch showing the unchecked state (CheckBoxOutlineBlank icon).
249250
cut.WaitForAssertion(() =>
250251
{
251252
var updatedButton = cut.Find("table tbody tr td:last-child button");
252-
// After unselecting, the button should not show the CheckBox icon anymore
253-
// It should show CheckBoxOutlineBlank (unselected state)
253+
// After unselecting, the button should show CheckBoxOutlineBlank icon (unchecked state)
254254
updatedButton.OuterHtml.Should().Contain("CheckBoxOutlineBlank");
255255
});
256256
}

0 commit comments

Comments
 (0)