From 20b465f161dc9fed60cfa280a3b7f912bb27d12e Mon Sep 17 00:00:00 2001
From: Benjamin Peinhardt <ben.peinhardt@coder.com>
Date: Tue, 23 Jul 2024 17:21:58 +0000
Subject: [PATCH] make 'owner' default for share

---
 windows-rdp/main.test.ts | 6 +-----
 windows-rdp/main.tf      | 1 +
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/windows-rdp/main.test.ts b/windows-rdp/main.test.ts
index 3363b34..adbf323 100644
--- a/windows-rdp/main.test.ts
+++ b/windows-rdp/main.test.ts
@@ -7,9 +7,9 @@ import {
 } from "../test";
 
 type TestVariables = Readonly<{
-  share: string;
   agent_id: string;
   resource_id: string;
+  share?: string;
   admin_username?: string;
   admin_password?: string;
 }>;
@@ -43,12 +43,10 @@ describe("Web RDP", async () => {
   testRequiredVariables<TestVariables>(import.meta.dir, {
     agent_id: "foo",
     resource_id: "bar",
-    share: "owner",
   });
 
   it("Has the PowerShell script install Devolutions Gateway", async () => {
     const state = await runTerraformApply<TestVariables>(import.meta.dir, {
-      share: "owner",
       agent_id: "foo",
       resource_id: "bar",
     });
@@ -96,7 +94,6 @@ describe("Web RDP", async () => {
       {
         agent_id: "foo",
         resource_id: "bar",
-        share: "owner",
       },
     );
 
@@ -115,7 +112,6 @@ describe("Web RDP", async () => {
     const customizedState = await runTerraformApply<TestVariables>(
       import.meta.dir,
       {
-        share: "owner",
         agent_id: "foo",
         resource_id: "bar",
         admin_username: customAdminUsername,
diff --git a/windows-rdp/main.tf b/windows-rdp/main.tf
index 8b0d5a8..f78e9ad 100644
--- a/windows-rdp/main.tf
+++ b/windows-rdp/main.tf
@@ -12,6 +12,7 @@ terraform {
 variable "share" {
   type        = string
   description = "The share level of the Coder app. Either 'owner' | 'authenticated' | 'public' "
+  default     = "owner"
 }
 
 variable "agent_id" {