Fixed TestAccAppOAuthApplication_postLogoutRedirectCrud . Doesn't see…

…m like it ever tested correctly given the code changes?
okta · Mar 22, 2022 · 8256eef · 8256eef
1 parent d6b0669
commit 8256eef
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/examples/okta_app_oauth_post_logout_redirect_uri/basic.tf b/examples/okta_app_oauth_post_logout_redirect_uri/basic.tf
@@ -13,7 +13,7 @@ resource "okta_app_oauth" "test" {
 
   // Since Okta forces us to create it with a redirect URI we have to ignore future changes, they will be detected as config drift.
   lifecycle {
-    ignore_changes = [redirect_uris]
+    ignore_changes = [post_logout_redirect_uris]
   }
 }
 

diff --git a/examples/okta_app_oauth_post_logout_redirect_uri/basic_updated.tf b/examples/okta_app_oauth_post_logout_redirect_uri/basic_updated.tf
@@ -14,7 +14,7 @@ resource "okta_app_oauth" "test" {
 
   // Since Okta forces us to create it with a redirect URI we have to ignore future changes, they will be detected as config drift.
   lifecycle {
-    ignore_changes = [redirect_uris]
+    ignore_changes = [post_logout_redirect_uris]
   }
 }
 

diff --git a/okta/resource_okta_app_oauth_post_logout_redirect_uri_test.go b/okta/resource_okta_app_oauth_post_logout_redirect_uri_test.go
@@ -52,8 +52,8 @@ func TestAccAppOAuthApplication_postLogoutRedirectCrud(t *testing.T) {
 				Config: config,
 				Check: resource.ComposeTestCheckFunc(
 					createPostLogoutRedirectURIExists(resourceName),
-					resource.TestCheckResourceAttr(resourceName, "id", "https://www.example.com"),
-					resource.TestCheckResourceAttr(resourceName, "uri", "https://www.example.com"),
+					resource.TestCheckResourceAttr(resourceName, "id", "http://google.com"),
+					resource.TestCheckResourceAttr(resourceName, "uri", "http://google.com"),
 					resource.TestCheckResourceAttrSet(resourceName, "app_id"),
 				),
 			},