Skip to content

Commit 9ee2834

Browse files
cpcloudclaude
andcommitted
fix(test): correct misleading skipColEdit comment in inline edit test
The comment claimed skipColEdit remaps sub-table col 3 to full-table col 4 (maintenanceColSeason). In reality, maintenanceColSeason is 3 and since 3 < skipAt (maintenanceColAppliance=4), no remap occurs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent a2eb3f4 commit 9ee2834

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

internal/app/handler_crud_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1012,8 +1012,8 @@ func TestApplianceMaintenanceInlineEditSeasonDispatchesCorrectly(t *testing.T) {
10121012
id := meta[0].ID
10131013

10141014
// In the sub-table (Appliance column removed), Season is at index 3.
1015-
// skipColEdit must remap this to maintenanceColSeason (4) in the full table,
1016-
// NOT to maintenanceColAppliance.
1015+
// Since 3 < skipAt (maintenanceColAppliance=4), no remap occurs and
1016+
// col 3 maps directly to maintenanceColSeason (3) in the full table.
10171017
m.exitForm()
10181018
m.closeInlineInput()
10191019
require.NoError(t, h.InlineEdit(m, id, 3))

0 commit comments

Comments
 (0)