test(sheets): drop redundant copyloopvar copy in required-flag parity test

Go 1.22+ scopes the loop var per iteration, so `cmd, business := cmd, business`
in TestBatchOp_RequiredFlagParity is a no-op that trips the repo's copyloopvar
linter (same cleanup as 2132472). Behavior unchanged; 45 sub-tests still pass.
This commit is contained in:
xiongyuanwen-byted
2026-06-03 14:46:05 +08:00
parent d05fbcf041
commit 1decb4399d

View File

@@ -878,7 +878,6 @@ func TestBatchOp_RequiredFlagParity(t *testing.T) {
if len(business) == 0 {
continue // only-locator commands (sheet-delete/hide/unhide/copy/filter-delete): nothing to omit
}
cmd, business := cmd, business
t.Run(cmd, func(t *testing.T) {
t.Parallel()
rawOp := map[string]interface{}{"shortcut": cmd, "input": sheetSel(cmd)}