Update OpenAPI spec to 2.12.0 (#730)

arvindkrishnakumar-okta · web-flow · commit 25b5be64ff56 · 2022-04-22T14:31:47.000-05:00
* Update OpenApi spec to 2.12.0

* updates to IT

* minor refactor

* Update SignOnPoliciesIT.groovy
diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/SignOnPoliciesIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/SignOnPoliciesIT.groovy
@@ -221,6 +221,73 @@ class SignOnPoliciesIT implements CrudTestSupport {
         assertThat(updateProfileEnrollmentPolicyRuleActions.getProfileEnrollment().getProfileAttributes().first().getRequired(), is(true))
     }
 
+    @Test (groups = "bacon")
+    void applyPolicyToApplicationTest() {
+
+        String name = "java-sdk-it-" + UUID.randomUUID().toString()
+
+        Application oidcApp = OIDCApplicationBuilder.instance()
+            .setName(name)
+            .setLabel(name)
+            .addRedirectUris("https://www.example.com")
+            .setPostLogoutRedirectUris(Collections.singletonList("https://www.example.com/logout"))
+            .setResponseTypes(Arrays.asList(OAuthResponseType.TOKEN, OAuthResponseType.CODE))
+            .setGrantTypes(Arrays.asList(OAuthGrantType.IMPLICIT, OAuthGrantType.AUTHORIZATION_CODE))
+            .setApplicationType(OpenIdConnectApplicationType.WEB)
+            .setClientId(UUID.randomUUID().toString())
+            .setClientSecret(UUID.randomUUID().toString())
+            .setAutoKeyRotation(true)
+            .setTokenEndpointAuthMethod(OAuthEndpointAuthenticationMethod.CLIENT_SECRET_BASIC)
+            .setIOS(false)
+            .setWeb(true)
+            .setLoginRedirectUrl("https://www.myapp.com")
+            .setErrorRedirectUrl("https://www.myapp.com/error")
+            .buildAndCreate(client)
+        registerForCleanup(oidcApp)
+
+        Policy policy1 = client.instantiate(Policy)
+            .setName(name + "-1")
+            .setType(PolicyType.ACCESS_POLICY)
+            .setStatus(Policy.StatusEnum.ACTIVE)
+            .setDescription("IT created Policy - applyPolicyToApplicationTest")
+
+        Policy createdPolicy1 = client.createPolicy(policy1)
+        registerForCleanup(createdPolicy1)
+
+        assertThat(createdPolicy1, notNullValue())
+
+        Policy policy2 = client.instantiate(Policy)
+            .setName(name + "-2")
+            .setType(PolicyType.ACCESS_POLICY)
+            .setStatus(Policy.StatusEnum.ACTIVE)
+            .setDescription("IT created Policy - applyPolicyToApplicationTest")
+
+        Policy createdPolicy2 = client.createPolicy(policy2)
+        registerForCleanup(createdPolicy2)
+
+        assertThat(createdPolicy2, notNullValue())
+
+        // update app policy to createdPolicy1
+        oidcApp.updateApplicationPolicy(createdPolicy1.getId())
+
+        Application updatedApp = client.getApplication(oidcApp.getId())
+        assertThat(updatedApp, notNullValue())
+
+        // assert if the app access policy resource id matches createdPolicy1 resource id
+        String policyResourceId = extractAccessPolicyResourceIdFromApplication(updatedApp)
+        assertThat(policyResourceId, equalTo(createdPolicy1.getId()))
+
+        // Now, update app policy to createdPolicy2
+        oidcApp.updateApplicationPolicy(createdPolicy2.getId())
+
+        updatedApp = client.getApplication(oidcApp.getId())
+        assertThat(updatedApp, notNullValue())
+
+        // assert if the app app policy resource id matches createdPolicy2 resource id
+        policyResourceId = extractAccessPolicyResourceIdFromApplication(updatedApp)
+        assertThat(policyResourceId, equalTo(createdPolicy2.getId()))
+    }
+
     @Test
     void signOnActionsTest() {
 
@@ -318,4 +385,13 @@ class SignOnPoliciesIT implements CrudTestSupport {
     static void assertRulesExpanded(AuthorizationServerPolicy policy) {
         assertThat policy.getEmbedded(), allOf(notNullValue(), hasKey("rules"))
     }
+
+    static String extractAccessPolicyResourceIdFromApplication(Application application) {
+        assertThat(application, notNullValue())
+        assertThat(application.getLinks(), notNullValue())
+        assertThat(application.getLinks().get("accessPolicy"), notNullValue())
+        String accessPolicyHref = application.getLinks().get("accessPolicy").toString()
+        String accessPolicyResourceId = accessPolicyHref.substring(accessPolicyHref.lastIndexOf("/") + 1).replaceAll("]", "")
+        return accessPolicyResourceId
+    }
 }
diff --git a/package.json b/package.json
@@ -1,5 +1,5 @@
 {
   "dependencies": {
-    "@okta/openapi": "2.11.2"
+    "@okta/openapi": "2.12.0"
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`	`2`	`"dependencies": {`
`3`		`- "@okta/openapi": "2.11.2"`
	`3`	`+ "@okta/openapi": "2.12.0"`
`4`	`4`	`}`
`5`	`5`	`}`