Function TestNoControlConnWhenDown

tstest/integration/integration_test.go:1131–1175

Issue 2434: when "down" (WantRunning false), tailscaled shouldn't be connected to control.

(t *testing.T)

// Issue 2434: when "down" (WantRunning false), tailscaled shouldn't
// be connected to control.
func TestNoControlConnWhenDown(t *testing.T) {
	tstest.Parallel(t)
	env := NewTestEnv(t)
	n1 := NewTestNode(t, env)

	d1 := n1.StartDaemon()
	n1.AwaitResponding()

	// Come up the first time.
	n1.MustUp()
	ip1 := n1.AwaitIP4()
	n1.AwaitRunning()

	// Then bring it down and stop the daemon.
	n1.MustDown()
	d1.MustCleanShutdown(t)

	env.LogCatcher.Reset()
	d2 := n1.StartDaemon()
	n1.AwaitResponding()

	n1.AwaitBackendState("Stopped")

	// The real test: verify our daemon doesn't have an HTTP request open.
	// Stopping the client may take some time to disconnect from testcontrol.
	if err := tstest.WaitFor(time.Second, func() error {
		if n := env.Control.InServeMap(); n != 0 {
			return fmt.Errorf("in serve map = %d; want 0", n)
		}
		return nil
	}); err != nil {
		t.Fatalf("unexpected connections while stopped: %v", err)
	}

	ip2 := n1.AwaitIP4()
	if ip1 != ip2 {
		t.Errorf("IPs different: %q vs %q", ip1, ip2)
	}

	if n := env.Control.InServeMap(); n != 0 {
		t.Fatalf("unexpected connection triggered by tailscale ip: in serve map = %d; want 0", n)
	}

	d2.MustCleanShutdown(t)
}

// Issue 2137: make sure Windows tailscaled works with the CLI alone,
// without the GUI to kick off a Start.