Add missing test case and fix typo in tests (#24915)

This PR adds a missing assertion in the `TestGetOrgUsersByOrgID`
function. It also incidentally fixes a small typo.
pull/24900/head^2
Philip Peterson 2 years ago committed by GitHub
parent 2250ddd112
commit 63d5e762d8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 6
      models/organization/org_test.go
  2. 4
      models/organization/org_user_test.go

@ -265,6 +265,12 @@ func TestGetOrgUsersByOrgID(t *testing.T) {
UID: 4, UID: 4,
IsPublic: false, IsPublic: false,
}, *orgUsers[1]) }, *orgUsers[1])
assert.Equal(t, organization.OrgUser{
ID: orgUsers[2].ID,
OrgID: 3,
UID: 28,
IsPublic: true,
}, *orgUsers[2])
} }
orgUsers, err = organization.GetOrgUsersByOrgID(db.DefaultContext, &organization.FindOrgMembersOpts{ orgUsers, err = organization.GetOrgUsersByOrgID(db.DefaultContext, &organization.FindOrgMembersOpts{

@ -85,7 +85,7 @@ func TestUserListIsPublicMember(t *testing.T) {
{22, map[int64]bool{}}, {22, map[int64]bool{}},
} }
for _, v := range tt { for _, v := range tt {
t.Run(fmt.Sprintf("IsPublicMemberOfOrdIg%d", v.orgid), func(t *testing.T) { t.Run(fmt.Sprintf("IsPublicMemberOfOrgId%d", v.orgid), func(t *testing.T) {
testUserListIsPublicMember(t, v.orgid, v.expected) testUserListIsPublicMember(t, v.orgid, v.expected)
}) })
} }
@ -112,7 +112,7 @@ func TestUserListIsUserOrgOwner(t *testing.T) {
{22, map[int64]bool{}}, // No member {22, map[int64]bool{}}, // No member
} }
for _, v := range tt { for _, v := range tt {
t.Run(fmt.Sprintf("IsUserOrgOwnerOfOrdIg%d", v.orgid), func(t *testing.T) { t.Run(fmt.Sprintf("IsUserOrgOwnerOfOrgId%d", v.orgid), func(t *testing.T) {
testUserListIsUserOrgOwner(t, v.orgid, v.expected) testUserListIsUserOrgOwner(t, v.orgid, v.expected)
}) })
} }

Loading…
Cancel
Save