Skip to content

Commit 5853726

Browse files
committed
Remove ldap link test which does not make any sense
1 parent 9c9bf7f commit 5853726

File tree

1 file changed

+0
-35
lines changed

1 file changed

+0
-35
lines changed

internal/provider/resource_gitlab_group_ldap_link_test.go

Lines changed: 0 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -47,17 +47,6 @@ func TestAccGitlabGroupLdapLink_basic(t *testing.T) {
4747
accessLevel: fmt.Sprintf("maintainer"), // nolint // TODO: Resolve this golangci-lint issue: S1039: unnecessary use of fmt.Sprintf (gosimple)
4848
})),
4949
},
50-
51-
// Force create the same group LDAP link in a different resource (uses testAccGitlabGroupLdapLinkForceCreateConfig for Config)
52-
{
53-
SkipFunc: isRunningInCE,
54-
Config: testAccGitlabGroupLdapLinkForceCreateConfig(rInt, &testLdapLink),
55-
Check: resource.ComposeTestCheckFunc(
56-
testAccCheckGitlabGroupLdapLinkExists("gitlab_group_ldap_link.bar", &ldapLink),
57-
testAccCheckGitlabGroupLdapLinkAttributes(&ldapLink, &testAccGitlabGroupLdapLinkExpectedAttributes{
58-
accessLevel: fmt.Sprintf("developer"), // nolint // TODO: Resolve this golangci-lint issue: S1039: unnecessary use of fmt.Sprintf (gosimple)
59-
})),
60-
},
6150
},
6251
})
6352
}
@@ -209,27 +198,3 @@ resource "gitlab_group_ldap_link" "foo" {
209198
ldap_provider = "%s"
210199
}`, rInt, rInt, testLdapLink.CN, testLdapLink.Provider)
211200
}
212-
213-
func testAccGitlabGroupLdapLinkForceCreateConfig(rInt int, testLdapLink *gitlab.LDAPGroupLink) string {
214-
return fmt.Sprintf(`
215-
resource "gitlab_group" "foo" {
216-
name = "foo%d"
217-
path = "foo%d"
218-
description = "Terraform acceptance test - Group LDAP Links 3"
219-
}
220-
221-
resource "gitlab_group_ldap_link" "foo" {
222-
group_id = "${gitlab_group.foo.id}"
223-
cn = "%s"
224-
group_access = "maintainer"
225-
ldap_provider = "%s"
226-
}
227-
228-
resource "gitlab_group_ldap_link" "bar" {
229-
group_id = "${gitlab_group.foo.id}"
230-
cn = "%s"
231-
group_access = "developer"
232-
ldap_provider = "%s"
233-
force = true
234-
}`, rInt, rInt, testLdapLink.CN, testLdapLink.Provider, testLdapLink.CN, testLdapLink.Provider)
235-
}

0 commit comments

Comments
 (0)