Skip to content

Commit e290027

Browse files
TestAccDatasourceRole_basic: ParallelTest -> Test
1 parent 53d5ed7 commit e290027

File tree

3 files changed

+5
-8
lines changed

3 files changed

+5
-8
lines changed

docker-compose.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
version: '3'
1+
version: "3"
22
services:
33
mysql:
44
ports:
@@ -30,7 +30,7 @@ services:
3030
- GF_SERVER_ROOT_URL=${GRAFANA_URL}
3131
- GF_ENTERPRISE_LICENSE_TEXT=${GF_ENTERPRISE_LICENSE_TEXT:-}
3232
- GF_SERVER_SERVE_FROM_SUB_PATH=${GF_SERVER_SERVE_FROM_SUB_PATH:-}
33-
- GF_FEATURE_TOGGLES_ENABLE=nestedFolders
33+
- GF_FEATURE_TOGGLES_ENABLE=nestedFolders,ssoSettingsApi,ssoSettingsSAML,ssoSettingsLDAP
3434
healthcheck:
3535
test: wget --no-verbose --tries=1 --spider http://0.0.0.0:3000/api/health || exit 1 # Use wget because older versions of Grafana don't have curl
3636
interval: 10s

internal/resources/grafana/data_source_role_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ func TestAccDatasourceRole_basic(t *testing.T) {
2323
})
2424
}
2525

26-
resource.ParallelTest(t, resource.TestCase{
26+
resource.Test(t, resource.TestCase{
2727
ProtoV5ProviderFactories: testutils.ProtoV5ProviderFactories,
2828
CheckDestroy: roleCheckExists.destroyed(&role, nil),
2929
Steps: []resource.TestStep{

internal/resources/grafana/resource_sso_settings_test.go

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ func TestSSOSettings_basic_oauth2(t *testing.T) {
6464
}
6565

6666
func TestSSOSettings_basic_saml(t *testing.T) {
67-
testutils.CheckEnterpriseTestsEnabled(t, ">=12.0")
67+
testutils.CheckEnterpriseTestsEnabled(t, ">=11.1")
6868

6969
provider := "saml"
7070

@@ -136,10 +136,7 @@ func TestSSOSettings_basic_saml(t *testing.T) {
136136
}
137137

138138
func TestSSOSettings_basic_ldap(t *testing.T) {
139-
// The version requirement was 11.3, but the LDAP provider requires some additional settings without
140-
// which the API fails with `failed to fetch the default settings for provider ldap`.
141-
// Incremented the version for now to skip the test, once it's fixed it can be downgraded again.
142-
testutils.CheckOSSTestsEnabled(t, ">=12.0")
139+
testutils.CheckOSSTestsEnabled(t, ">=11.3")
143140

144141
provider := "ldap"
145142

0 commit comments

Comments
 (0)