From 29bc021dcd083197a29106feb43f4f9aa66ddff4 Mon Sep 17 00:00:00 2001
From: Brad Fitzpatrick <bradfitz@tailscale.com>
Date: Fri, 11 Nov 2022 18:54:11 -0800
Subject: [PATCH] cmd/tailscale/cli: add outline of serve CLI tests

Updates tailscale/corp#7515

Change-Id: Ib3956d4756bdcea0da89238a3c520ce8ac8004ec
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
---
 cmd/tailscale/cli/cli_test.go | 48 +++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/cmd/tailscale/cli/cli_test.go b/cmd/tailscale/cli/cli_test.go
index 57cd9303a..f929f9688 100644
--- a/cmd/tailscale/cli/cli_test.go
+++ b/cmd/tailscale/cli/cli_test.go
@@ -11,6 +11,7 @@ import (
 	"fmt"
 	"net/netip"
 	"reflect"
+	"runtime"
 	"strings"
 	"testing"
 
@@ -1156,3 +1157,50 @@ func TestUpWorthWarning(t *testing.T) {
 		t.Errorf("want false for other misc errors")
 	}
 }
+
+func TestServeConfigMutations(t *testing.T) {
+	// Stateful mutations, starting from an empty config.
+	type step struct {
+		command []string // serve args
+		reset   bool     // if true, reset all ServeConfig state
+		want    *ipn.ServeConfig
+		wantErr string
+		line    int // line number of addStep call, for error messages
+	}
+	var steps []step
+	add := func(s step) {
+		_, _, s.line, _ = runtime.Caller(1)
+		steps = append(steps, s)
+	}
+	add(step{reset: true})
+	add(step{
+		want: nil,
+	})
+	var current *ipn.ServeConfig
+	for i, st := range steps {
+		t.Logf("Executing step #%d (line %v) ... ", i, st.line)
+		if st.reset {
+			t.Logf("(resetting state)")
+			current = nil
+		}
+		newState, err := applyServeMutation(current, st.command)
+		var gotErr string
+		if err != nil {
+			gotErr = err.Error()
+		}
+		if gotErr != st.wantErr {
+			t.Fatalf("[%d] %v: got error %q, want %q", i, st.command, gotErr, st.wantErr)
+		}
+		if !reflect.DeepEqual(newState, st.want) {
+			t.Fatalf("[%d] %v: bad state. got:\n%s\n\nwant:\n%s\n",
+				i, st.command, asJSON(newState), asJSON(st.want))
+		}
+	}
+}
+
+func applyServeMutation(current *ipn.ServeConfig, command []string) (*ipn.ServeConfig, error) {
+	if len(command) == 0 {
+		return current, nil
+	}
+	panic("TODO") // in cli/serve.go, not here in tests
+}