Skip to content

Commit d9d93cb

Browse files
fix(binary): remove unused atomic.Bool in download test.
Go vet rejects copying sync/atomic.Bool via blank assignment. Signed-off-by: Barak Korren <bkorren@redhat.com> Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 408285f commit d9d93cb

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

internal/binary/download_test.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -346,8 +346,6 @@ func TestResolveForVendor_ReleaseFallback(t *testing.T) {
346346
}
347347

348348
func TestResolveForRun_PrefersReleaseBeforeCrossCompile(t *testing.T) {
349-
var crossCompileAttempted atomic.Bool
350-
351349
// Build mock release assets.
352350
var tarBuf bytes.Buffer
353351
gw := gzip.NewWriter(&tarBuf)
@@ -391,8 +389,6 @@ func TestResolveForRun_PrefersReleaseBeforeCrossCompile(t *testing.T) {
391389
require.NoError(t, os.Chdir(tmpDir))
392390
t.Cleanup(func() { _ = os.Chdir(origDir) })
393391

394-
_ = crossCompileAttempted // release succeeds first; cross-compile not reached
395-
396392
result, err := ResolveForRun("0.4.0", "amd64")
397393
require.NoError(t, err)
398394
t.Cleanup(func() { os.RemoveAll(result.TmpDir) })

0 commit comments

Comments
 (0)