ipn/ipnlocal: fix TestOnTailnetDefaultAutoUpdate on unsupported platforms

Fixes #11894

Signed-off-by: Andrew Lytvynov <awly@tailscale.com>
pull/11921/head
Andrew Lytvynov 1 month ago
parent b2b49cb3d5
commit a3062e06f9
No known key found for this signature in database

@ -15,7 +15,6 @@ import (
"net/netip" "net/netip"
"os" "os"
"reflect" "reflect"
"runtime"
"slices" "slices"
"sync" "sync"
"testing" "testing"
@ -2269,9 +2268,6 @@ func TestPreferencePolicyInfo(t *testing.T) {
} }
func TestOnTailnetDefaultAutoUpdate(t *testing.T) { func TestOnTailnetDefaultAutoUpdate(t *testing.T) {
if runtime.GOOS == "darwin" {
t.Skip("test broken on macOS; see https://github.com/tailscale/tailscale/issues/11894")
}
tests := []struct { tests := []struct {
before, after opt.Bool before, after opt.Bool
tailnetDefault bool tailnetDefault bool
@ -2316,7 +2312,14 @@ func TestOnTailnetDefaultAutoUpdate(t *testing.T) {
t.Fatal(err) t.Fatal(err)
} }
b.onTailnetDefaultAutoUpdate(tt.tailnetDefault) b.onTailnetDefaultAutoUpdate(tt.tailnetDefault)
if want, got := tt.after, b.pm.CurrentPrefs().AutoUpdate().Apply; got != want { want := tt.after
// On platforms that don't support auto-update we can never
// transition to auto-updates being enabled. The value should
// remain unchanged after onTailnetDefaultAutoUpdate.
if !clientupdate.CanAutoUpdate() && want.EqualBool(true) {
want = tt.before
}
if got := b.pm.CurrentPrefs().AutoUpdate().Apply; got != want {
t.Errorf("got: %q, want %q", got, want) t.Errorf("got: %q, want %q", got, want)
} }
}) })

Loading…
Cancel
Save