Skip to content

Commit fb5b947

Browse files
brettkolodnybrettkolodny
committed
wip
1 parent 3509710 commit fb5b947

File tree

1 file changed

+50
-32
lines changed

1 file changed

+50
-32
lines changed

‎cli/sharing_test.go

Lines changed: 50 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,10 @@ import (
77
"github.com/coder/coder/v2/coderd/coderdtest"
88
"github.com/coder/coder/v2/coderd/database"
99
"github.com/coder/coder/v2/coderd/database/dbfake"
10+
"github.com/coder/coder/v2/codersdk"
1011
"github.com/coder/coder/v2/testutil"
1112
"github.com/stretchr/testify/assert"
13+
"github.com/stretchr/testify/require"
1214
)
1315

1416
func TestSharingShare(t *testing.T) {
@@ -21,63 +23,79 @@ func TestSharingShare(t *testing.T) {
2123
client, db = coderdtest.NewWithDatabase(t, nil)
2224
orgOwner = coderdtest.CreateFirstUser(t, client)
2325
workspaceOwnerClient, workspaceOwner = coderdtest.CreateAnotherUser(t, client, orgOwner.OrganizationID)
24-
_, toShareWithUser = coderdtest.CreateAnotherUser(t, client, orgOwner.OrganizationID)
2526
workspace = dbfake.WorkspaceBuild(t, db, database.WorkspaceTable{
2627
OwnerID: workspaceOwner.ID,
2728
OrganizationID: orgOwner.OrganizationID,
2829
}).Do().Workspace
30+
_, toShareWithUser = coderdtest.CreateAnotherUser(t, client, orgOwner.OrganizationID)
2931
)
3032

31-
var _ = testutil.Context(t, testutil.WaitMedium)
32-
var _, root = clitest.New(t, "sharing", "share", workspace.Name, "--user", toShareWithUser.Username)
33+
ctx := testutil.Context(t, testutil.WaitMedium)
34+
var inv, root = clitest.New(t, "sharing", "share", workspace.Name, "--user", toShareWithUser.Username)
3335
clitest.SetupConfig(t, workspaceOwnerClient, root)
3436

37+
err := inv.WithContext(ctx).Run()
38+
require.NoError(t, err)
39+
3540
// TODO: Test output of the command
3641

37-
// TODO: Test updated ACL
42+
acl, err := workspaceOwnerClient.WorkspaceACL(inv.Context(), workspace.ID)
43+
assert.NoError(t, err)
44+
assert.Equal(t,
45+
codersdk.WorkspaceACL{
46+
Users: []codersdk.WorkspaceUser{{
47+
MinimalUser: codersdk.MinimalUser{
48+
ID: toShareWithUser.ID,
49+
Username: toShareWithUser.Username,
50+
AvatarURL: toShareWithUser.AvatarURL,
51+
},
52+
Role: codersdk.WorkspaceRole("use"),
53+
}},
54+
},
55+
acl)
3856

3957
assert.True(t, false)
4058
})
4159

42-
t.Run("ShareWithUsers+Multiple", func(t *testing.T) {
43-
t.Parallel()
60+
// t.Run("ShareWithUsers+Multiple", func(t *testing.T) {
61+
// t.Parallel()
4462

45-
assert.True(t, false)
46-
})
63+
// assert.True(t, false)
64+
// })
4765

48-
t.Run("ShareWithUsers+UseRole", func(t *testing.T) {
49-
t.Parallel()
66+
// t.Run("ShareWithUsers+UseRole", func(t *testing.T) {
67+
// t.Parallel()
5068

51-
assert.True(t, false)
52-
})
69+
// assert.True(t, false)
70+
// })
5371

54-
t.Run("ShareWithUsers+AdminRole", func(t *testing.T) {
55-
t.Parallel()
72+
// t.Run("ShareWithUsers+AdminRole", func(t *testing.T) {
73+
// t.Parallel()
5674

57-
assert.True(t, false)
58-
})
75+
// assert.True(t, false)
76+
// })
5977

60-
t.Run("ShareWithGroups+Simple", func(t *testing.T) {
61-
t.Parallel()
78+
// t.Run("ShareWithGroups+Simple", func(t *testing.T) {
79+
// t.Parallel()
6280

63-
assert.True(t, false)
64-
})
81+
// assert.True(t, false)
82+
// })
6583

66-
t.Run("ShareWithGroups+Mutliple", func(t *testing.T) {
67-
t.Parallel()
84+
// t.Run("ShareWithGroups+Mutliple", func(t *testing.T) {
85+
// t.Parallel()
6886

69-
assert.True(t, false)
70-
})
87+
// assert.True(t, false)
88+
// })
7189

72-
t.Run("ShareWithGroups+UseRole", func(t *testing.T) {
73-
t.Parallel()
90+
// t.Run("ShareWithGroups+UseRole", func(t *testing.T) {
91+
// t.Parallel()
7492

75-
assert.True(t, false)
76-
})
93+
// assert.True(t, false)
94+
// })
7795

78-
t.Run("ShareWithGroups+AdminRole", func(t *testing.T) {
79-
t.Parallel()
96+
// t.Run("ShareWithGroups+AdminRole", func(t *testing.T) {
97+
// t.Parallel()
8098

81-
assert.True(t, false)
82-
})
99+
// assert.True(t, false)
100+
// })
83101
}

0 commit comments

Comments
 (0)