x/tools/internal/imports: TestSimpleCases failures

[gopherbot]

#!watchflakes
post <- pkg == "golang.org/x/tools/internal/imports" && test == "TestSimpleCases"

Issue created automatically to collect these failures.

Example (log):

--- FAIL: TestSimpleCases (22.73s)
    --- FAIL: TestSimpleCases/remove_unused_2_of_2 (0.50s)
        --- FAIL: TestSimpleCases/remove_unused_2_of_2/Modules (0.42s)
            fix_test.go:1228: err: fork/exec C:\workdir\go\bin\go.exe: The paging file is too small for this operation to complete.: stderr: 
    --- FAIL: TestSimpleCases/remove_unused_1_of_1 (0.11s)
        --- FAIL: TestSimpleCases/remove_unused_1_of_1/Modules (0.08s)
            fix_test.go:1228: err: fork/exec C:\workdir\go\bin\go.exe: Insufficient system resources exist to complete the requested service.: stderr: 

— watchflakes

[gopherbot]

Found new dashboard test flakes for:

#!watchflakes
post <- pkg == "golang.org/x/tools/internal/imports" && test == "TestSimpleCases"

2022-12-07 15:31 windows-arm64-11 tools@09fb680d go@dc04f3ba x/tools/internal/imports.TestSimpleCases (log)

--- FAIL: TestSimpleCases (22.73s)
    --- FAIL: TestSimpleCases/remove_unused_2_of_2 (0.50s)
        --- FAIL: TestSimpleCases/remove_unused_2_of_2/Modules (0.42s)
            fix_test.go:1228: err: fork/exec C:\workdir\go\bin\go.exe: The paging file is too small for this operation to complete.: stderr: 
    --- FAIL: TestSimpleCases/remove_unused_1_of_1 (0.11s)
        --- FAIL: TestSimpleCases/remove_unused_1_of_1/Modules (0.08s)
            fix_test.go:1228: err: fork/exec C:\workdir\go\bin\go.exe: Insufficient system resources exist to complete the requested service.: stderr: 

— watchflakes

[bcmills]

Duplicate of #51019

[gopherbot]

Found new dashboard test flakes for:

#!watchflakes
post <- pkg == "golang.org/x/tools/internal/imports" && test == "TestSimpleCases"

2023-04-06 17:53 windows-amd64-race tools@c5f768a4 go@4526fa79 x/tools/internal/imports.TestSimpleCases (log)

runtime: bad pointer in frame os.(*File).readdir at 0xc000187ba8: 0x140
fatal error: invalid pointer found on stack

runtime stack:
runtime.throw({0x140430d04?, 0x140633fe0?})
	C:/workdir/go/src/runtime/panic.go:1076 +0x65 fp=0x2805f5b0 sp=0x2805f580 pc=0x140039405
runtime.adjustpointers(0x2805f828?, 0x2805f670, 0x14005efdf?, {0x2805f828?, 0x0?})
	C:/workdir/go/src/runtime/stack.go:632 +0x1ad fp=0x2805f610 sp=0x2805f5b0 pc=0x14005116d
runtime.adjustframe(0x2805f828, 0x2805f708)
	C:/workdir/go/src/runtime/stack.go:673 +0xbb fp=0x2805f6a0 sp=0x2805f610 pc=0x14005127b
...
	C:/workdir/go/src/runtime/proc.go:392 +0xce fp=0xc000057c78 sp=0xc000057c58 pc=0x14003be4e
runtime.chanrecv(0xc000117180, 0xc000057d5e, 0x1)
	C:/workdir/go/src/runtime/chan.go:583 +0x385 fp=0xc000057cf0 sp=0xc000057c78 pc=0x140006905
runtime.chanrecv1(0x140654180?, 0x1403af240?)
	C:/workdir/go/src/runtime/chan.go:442 +0x12 fp=0xc000057d18 sp=0xc000057cf0 pc=0x140006552
testing.(*T).Run(0xc000085520, {0x1404229ff, 0x11}, 0xc00008ae20)
	C:/workdir/go/src/testing/testing.go:1649 +0x865 fp=0xc000057e38 sp=0xc000057d18 pc=0x14015b0a5
golang.org/x/tools/internal/imports.TestSimpleCases(0x0?)
	C:/workdir/gopath/src/golang.org/x/tools/internal/imports/fix_test.go:1191 +0x85 fp=0xc000057e98 sp=0xc000057e38 pc=0x1403064a5
testing.tRunner(0xc000085520, 0x140441a20)

— watchflakes

[bcmills]

The above was a regression introduced in CL 482536 and rolled back in CL 482817.

I believe that watchflakes didn't file an issue for it otherwise because the windows-amd64-race builder was failing at every commit, and watchflakes currently does not file issues for builders than are completely broken (that's #58819).