work on tests

Author: defelmnq

docs/resources/app.md

@@ -65,6 +65,7 @@ resource "coder_app" "vim" {
 - `healthcheck` (Block Set, Max: 1) HTTP health checking to determine the application readiness. (see [below for nested schema](#nestedblock--healthcheck))
 - `hidden` (Boolean) Determines if the app is visible in the UI (minimum Coder version: v2.16).
 - `icon` (String) A URL to an icon that will display in the dashboard. View built-in icons here: https://github.com/coder/coder/tree/main/site/static/icon. Use a built-in icon with `"${data.coder_workspace.me.access_url}/icon/<path>"`.
+- `open_in` (String) Determines where the app will be opened. Valid values are `"tab"`, `"window"`, and `"slim-window" (default)`. `"tab"` opens in a new tab in the same browser window. `"window"` opens a fresh browser window with navigation options. `"slim-window"` opens a fresh browser window with slim navigation options.
 - `order` (Number) The order determines the position of app in the UI presentation. The lowest order is shown first and apps with equal order are sorted by name (ascending order).
 - `share` (String) Determines the level which the application is shared at. Valid levels are `"owner"` (default), `"authenticated"` and `"public"`. Level `"owner"` disables sharing on the app, so only the workspace owner can access it. Level `"authenticated"` shares the app with all authenticated users. Level `"public"` shares it with any user, including unauthenticated users. Permitted application sharing levels can be configured site-wide via a flag on `coder server` (Enterprise only).
 - `subdomain` (Boolean) Determines whether the app will be accessed via it's own subdomain or whether it will be accessed via a path on Coder. If wildcards have not been setup by the administrator then apps with `subdomain` set to `true` will not be accessible. Defaults to `false`.

integration/coder-app-open-in/main.tf

@@ -0,0 +1,62 @@
+terraform {
+  required_providers {
+    coder = {
+      source = "coder/coder"
+    }
+    local = {
+      source = "hashicorp/local"
+    }
+  }
+}
+
+data "coder_workspace" "me" {}
+
+resource "coder_agent" "dev" {
+  os   = "linux"
+  arch = "amd64"
+  dir  = "/workspace"
+}
+
+resource "coder_app" "window" {
+  agent_id = coder_agent.dev.id
+  slug     = "window"
+  share    = "owner"
+  open_in  = "window"
+}
+
+resource "coder_app" "slim-window" {
+  agent_id = coder_agent.dev.id
+  slug     = "slim-window"
+  share    = "owner"
+  open_in  = "slim-window"
+}
+
+resource "coder_app" "defaulted" {
+  agent_id = coder_agent.dev.id
+  slug     = "defaulted"
+  share    = "owner"
+}
+
+locals {
+  # NOTE: these must all be strings in the output
+  output = {
+    "coder_app.window.open_in"    = tostring(coder_app.window.open_in)
+    "coder_app.slim-window.open_in"   = tostring(coder_app.slim-window.open_in)
+    "coder_app.defaulted.open_in" = tostring(coder_app.defaulted.open_in)
+  }
+}
+
+variable "output_path" {
+  type = string
+}
+
+resource "local_file" "output" {
+  filename = var.output_path
+  content  = jsonencode(local.output)
+}
+
+output "output" {
+  value     = local.output
+  sensitive = true
+}
+

integration/integration_test.go

@@ -143,6 +143,15 @@ func TestIntegration(t *testing.T) {
 				"workspace_owner.login_type":        `password`,
 			},
 		},
+		{
+			name:       "workspace-with-open-in",
+			minVersion: "v2.19.0",
+			expectedOutput: map[string]string{
+				"coder_app.window.open_in":      "window",
+				"coder_app.slim-window.open_in": "slim-window",
+				"coder_app.defaulted.open_in":   "slim-window",
+			},
+		},
 		{
 			name:       "coder-app-hidden",
 			minVersion: "v0.0.0",

provider/app_test.go

@@ -246,6 +246,106 @@ func TestApp(t *testing.T) {
 		}
 	})
 
+	t.Run("OpenIn", func(t *testing.T) {
+		t.Parallel()
+
+		cases := []struct {
+			name        string
+			value       string
+			expectValue string
+			expectError *regexp.Regexp
+		}{
+			{
+				name:        "default",
+				value:       "", // default
+				expectValue: "slim-window",
+			},
+			{
+				name:        "InvalidValue",
+				value:       "nonsense",
+				expectError: regexp.MustCompile(`invalid "coder_app" open_in value, must be one of "tab", "window", "slim-window": "nonsense"`),
+			},
+			{
+				name:        "ExplicitWindow",
+				value:       "window",
+				expectValue: "window",
+			},
+			{
+				name:        "ExplicitSlimWindow",
+				value:       "slim-window",
+				expectValue: "slim-window",
+			},
+			{
+				name:        "ExplicitTab",
+				value:       "tab",
+				expectValue: "tab",
+			},
+		}
+
+		for _, c := range cases {
+			c := c
+
+			t.Run(c.name, func(t *testing.T) {
+				t.Parallel()
+
+				config := `
+				provider "coder" {
+				}
+				resource "coder_agent" "dev" {
+					os = "linux"
+					arch = "amd64"
+				}
+				resource "coder_app" "code-server" {
+					agent_id = coder_agent.dev.id
+					slug = "code-server"
+					display_name = "code-server"
+					icon = "builtin:vim"
+					url = "http://localhost:13337"
+					healthcheck {
+						url = "http://localhost:13337/healthz"
+						interval = 5
+						threshold = 6
+					}`
+
+				if c.value != "" {
+					config += fmt.Sprintf(`
+					open_in = %q
+					`, c.value)
+				}
+
+				config += `
+				}
+				`
+
+				checkFn := func(state *terraform.State) error {
+					require.Len(t, state.Modules, 1)
+					require.Len(t, state.Modules[0].Resources, 2)
+					resource := state.Modules[0].Resources["coder_app.code-server"]
+					require.NotNil(t, resource)
+
+					// Read share and ensure it matches the expected
+					// value.
+					value := resource.Primary.Attributes["open_in"]
+					require.Equal(t, c.expectValue, value)
+					return nil
+				}
+				if c.expectError != nil {
+					checkFn = nil
+				}
+
+				resource.Test(t, resource.TestCase{
+					ProviderFactories: coderFactory(),
+					IsUnitTest:        true,
+					Steps: []resource.TestStep{{
+						Config:      config,
+						Check:       checkFn,
+						ExpectError: c.expectError,
+					}},
+				})
+			})
+		}
+	})
+
 	t.Run("Hidden", func(t *testing.T) {
 		t.Parallel()