Skip to content

Commit d63c71f

Browse files
fix(saml-alert): add sso precheck
1 parent 60b6231 commit d63c71f

2 files changed

Lines changed: 22 additions & 1 deletion

File tree

ns1/provider_test.go

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package ns1
22

33
import (
4+
"log"
45
"os"
56
"testing"
67

@@ -32,3 +33,23 @@ func testAccPreCheck(t *testing.T) {
3233
t.Fatal("NS1_APIKEY must be set for acceptance tests")
3334
}
3435
}
36+
37+
func testAccPreCheckSso(t *testing.T) {
38+
testAccPreCheck(t)
39+
40+
client, err := sharedClient()
41+
if err != nil {
42+
log.Fatalf("failed to get shared client: %s", err)
43+
}
44+
kl, _, err := client.APIKeys.List()
45+
if err != nil {
46+
t.Skipf("account not authorized for redirects, skipping test")
47+
}
48+
if len(kl) == 0 {
49+
t.Skipf("no api keys found, skipping test")
50+
}
51+
if kl[0].Permissions.Account.ManageUsers == false {
52+
t.Skipf("account not authorized to manage users, skipping test")
53+
}
54+
55+
}

ns1/resource_alert_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ func TestAccAlert_Sso(t *testing.T) {
5252
)
5353

5454
resource.Test(t, resource.TestCase{
55-
PreCheck: func() { testAccPreCheck(t) },
55+
PreCheck: func() { testAccPreCheckSso(t) },
5656
Providers: testAccProviders,
5757
CheckDestroy: testAccCheckAlertDestroy,
5858
Steps: []resource.TestStep{

0 commit comments

Comments
 (0)