Add tests for disallowing all port commands for codespace w/ pending op

cwndrws · cwndrws · commit 3ed2e49bd95f · 2022-03-15T17:27:38.000-04:00
diff --git a/pkg/cmd/codespace/ports_test.go b/pkg/cmd/codespace/ports_test.go
@@ -0,0 +1,71 @@
+package codespace
+
+import (
+	"context"
+	"testing"
+
+	"github.com/cli/cli/v2/internal/codespaces/api"
+	"github.com/cli/cli/v2/pkg/iostreams"
+)
+
+func TestPendingOperationDisallowsListPorts(t *testing.T) {
+	app := testingPortsApp()
+
+	if err := app.ListPorts(context.Background(), "disabledCodespace", nil); err != nil {
+		if err.Error() != "codespace is disabled while it has a pending operation: Some pending operation" {
+			t.Errorf("expected pending operation error, but got: %v", err)
+		}
+	} else {
+		t.Error("expected pending operation error, but got nothing")
+	}
+}
+
+func TestPendingOperationDisallowsUpdatePortVisability(t *testing.T) {
+	app := testingPortsApp()
+
+	if err := app.UpdatePortVisibility(context.Background(), "disabledCodespace", nil); err != nil {
+		if err.Error() != "codespace is disabled while it has a pending operation: Some pending operation" {
+			t.Errorf("expected pending operation error, but got: %v", err)
+		}
+	} else {
+		t.Error("expected pending operation error, but got nothing")
+	}
+}
+
+func TestPendingOperationDisallowsForwardPorts(t *testing.T) {
+	app := testingPortsApp()
+
+	if err := app.ForwardPorts(context.Background(), "disabledCodespace", nil); err != nil {
+		if err.Error() != "codespace is disabled while it has a pending operation: Some pending operation" {
+			t.Errorf("expected pending operation error, but got: %v", err)
+		}
+	} else {
+		t.Error("expected pending operation error, but got nothing")
+	}
+}
+
+func testingPortsApp() *App {
+	user := &api.User{Login: "monalisa"}
+	disabledCodespace := &api.Codespace{
+		Name:                           "disabledCodespace",
+		PendingOperation:               true,
+		PendingOperationDisabledReason: "Some pending operation",
+	}
+	apiMock := &apiClientMock{
+		GetCodespaceFunc: func(_ context.Context, name string, _ bool) (*api.Codespace, error) {
+			if name == "disabledCodespace" {
+				return disabledCodespace, nil
+			}
+			return nil, nil
+		},
+		GetUserFunc: func(_ context.Context) (*api.User, error) {
+			return user, nil
+		},
+		AuthorizedKeysFunc: func(_ context.Context, _ string) ([]byte, error) {
+			return []byte{}, nil
+		},
+	}
+
+	io, _, _, _ := iostreams.Test()
+	return NewApp(io, nil, apiMock, nil)
+}
