From 400b6f5bff1086544cd4102d28075fa57f780911 Mon Sep 17 00:00:00 2001 From: Vijet M Date: Wed, 28 Sep 2022 08:09:21 -0700 Subject: [PATCH] Enables only critical tests for PDV (#766) * Enables only critical tests for PDV * Updated config.yml * Revert config.yml --- .circleci/config.yml | 4 +- .../okta/sdk/tests/it/ApplicationsIT.groovy | 32 ++++++------- .../sdk/tests/it/AuthorizationServerIT.groovy | 37 +++++++------- .../com/okta/sdk/tests/it/EventHooksIT.groovy | 12 ++--- .../com/okta/sdk/tests/it/FactorsIT.groovy | 26 +++++----- .../com/okta/sdk/tests/it/FeaturesIT.groovy | 2 +- .../com/okta/sdk/tests/it/GroupRolesIT.groovy | 8 ++-- .../okta/sdk/tests/it/GroupSchemaIT.groovy | 2 +- .../com/okta/sdk/tests/it/GroupsIT.groovy | 8 ++-- .../groovy/com/okta/sdk/tests/it/IdpIT.groovy | 8 ++-- .../okta/sdk/tests/it/InlineHooksIT.groovy | 8 ++-- .../okta/sdk/tests/it/LinkedObjectsIT.groovy | 8 ++-- .../okta/sdk/tests/it/NetworkZoneIT.groovy | 4 +- .../okta/sdk/tests/it/OrgSettingsIT.groovy | 12 ++--- .../okta/sdk/tests/it/PolicyRulesIT.groovy | 4 +- .../it/ThreatInsightConfigurationIT.groovy | 2 +- .../com/okta/sdk/tests/it/UserTypesIT.groovy | 6 +-- .../com/okta/sdk/tests/it/UsersIT.groovy | 48 +++++++++---------- 18 files changed, 115 insertions(+), 116 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index e5a59947a8b..3b12bb32759 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -17,14 +17,14 @@ jobs: jdk8: docker: - - image: cimg/openjdk:8.0.322 + - image: cimg/openjdk:8.0-node environment: JVM_OPTS: -Xmx3200m steps: *build_steps jdk11: docker: - - image: cimg/openjdk:11.0.13 + - image: cimg/openjdk:11.0-node environment: JVM_OPTS: -Xmx3200m steps: *build_steps diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/ApplicationsIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/ApplicationsIT.groovy index fbe8ceb8111..5c58e9113cd 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/ApplicationsIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/ApplicationsIT.groovy @@ -160,7 +160,7 @@ class ApplicationsIT extends ITSupport { .setTokenEndpointAuthMethod(OAuthEndpointAuthenticationMethod.CLIENT_SECRET_POST)))) } - @Test (groups = "group1") + @Test void crudAutoLogin() { doCrudTest(client.instantiate(AutoLoginApplication) .setVisibility(client.instantiate(ApplicationVisibility) @@ -174,7 +174,7 @@ class ApplicationsIT extends ITSupport { .setLoginUrl("http://swaprimaryloginurl.okta.com")))) } - @Test (groups = "bacon") + @Test void crudWsFed() { doCrudTest(client.instantiate(WsFederationApplication) .setSettings(client.instantiate(WsFederationApplicationSettings) @@ -193,7 +193,7 @@ class ApplicationsIT extends ITSupport { .setUsernameAttribute("username")))) } - @Test (groups = "group1", enabled = false) // OKTA-530437 + @Test void crudSaml20() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -244,7 +244,7 @@ class ApplicationsIT extends ITSupport { .setValues(["Value"])]))))) } - @Test (groups = "group1") + @Test void crudSecurePasswordStore() { doCrudTest(client.instantiate(SecurePasswordStoreApplication) .setSettings(client.instantiate(SecurePasswordStoreApplicationSettings) @@ -260,7 +260,7 @@ class ApplicationsIT extends ITSupport { .setOptionalField3Value("finalvalue")))) } - @Test (groups = "group1") + @Test void crudBrowserPlugin() { doCrudTest(client.instantiate(SwaApplication) .setLabel(uniqueTestName) @@ -273,7 +273,7 @@ class ApplicationsIT extends ITSupport { // .setLoginUrlRegex("REGEX_EXPRESSION")) } - @Test (groups = "group1") + @Test void crudBasicAuth() { doCrudTest(client.instantiate(BasicAuthApplication) .setSettings(client.instantiate(BasicApplicationSettings) @@ -282,7 +282,7 @@ class ApplicationsIT extends ITSupport { .setUrl("https://example.com/login.html")))) } - @Test (groups = "group1") + @Test void crudBasicAuth_editUsernameAndPassword() { doCrudTest(client.instantiate(BasicAuthApplication) .setCredentials(client.instantiate(SchemeApplicationCredentials) @@ -293,7 +293,7 @@ class ApplicationsIT extends ITSupport { .setUrl("https://example.com/login.html")))) } - @Test (groups = "group1") + @Test void crudBasicAuth_adminSetsCredentials() { doCrudTest(client.instantiate(BasicAuthApplication) .setCredentials(client.instantiate(SchemeApplicationCredentials) @@ -305,7 +305,7 @@ class ApplicationsIT extends ITSupport { .setUrl("https://example.com/login.html")))) } - @Test (groups = "group1") + @Test void invalidApplicationCredentialsSchemeTest() { expect(IllegalArgumentException, { client.instantiate(BasicAuthApplication) @@ -319,7 +319,7 @@ class ApplicationsIT extends ITSupport { }) } - @Test (groups = "group1") + @Test void invalidOAuthEndpointAuthenticationMethodTest() { expect(IllegalArgumentException ,{client.instantiate(OpenIdConnectApplication) .setSettings(client.instantiate(OpenIdConnectApplicationSettings) @@ -343,7 +343,7 @@ class ApplicationsIT extends ITSupport { .setTokenEndpointAuthMethod(OAuthEndpointAuthenticationMethod.SDK_UNKNOWN)))}) } - @Test (groups = "group1") + @Test void invalidOAuthResponseTypeTest() { expect(IllegalArgumentException ,{client.instantiate(OpenIdConnectApplication) .setSettings(client.instantiate(OpenIdConnectApplicationSettings) @@ -368,7 +368,7 @@ class ApplicationsIT extends ITSupport { .setTokenEndpointAuthMethod(OAuthEndpointAuthenticationMethod.CLIENT_SECRET_POST)))}) } - @Test (groups = "group1") + @Test void invalidOAuthGrantTypeTest() { expect(IllegalArgumentException, {client.instantiate(OpenIdConnectApplication) .setSettings(client.instantiate(OpenIdConnectApplicationSettings) @@ -394,7 +394,7 @@ class ApplicationsIT extends ITSupport { }) } - @Test (groups = "group1") + @Test void invalidOpenIdConnectApplicationTypeTest() { expect(IllegalArgumentException ,{ client.instantiate(OpenIdConnectApplication) @@ -420,7 +420,7 @@ class ApplicationsIT extends ITSupport { }) } - @Test (groups = "group1") + @Test void crudBookmark() { doCrudTest(client.instantiate(BookmarkApplication) .setSettings(client.instantiate(BookmarkApplicationSettings) @@ -646,7 +646,7 @@ class ApplicationsIT extends ITSupport { assertThat readAppUser.getCredentials().getUserName(), equalTo("updated-"+user2.getProfile().getEmail()) } - @Test (groups = "group1") + @Test void csrTest() { Client client = getClient() @@ -807,7 +807,7 @@ class ApplicationsIT extends ITSupport { assertThat(response.size(), is(3)) } - @Test (groups = "group1") + @Test void testGetRawResponse() { def app = client.instantiate(SamlApplication) .setVisibility(client.instantiate(ApplicationVisibility)) diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/AuthorizationServerIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/AuthorizationServerIT.groovy index 9974fff22c5..adc60b487a9 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/AuthorizationServerIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/AuthorizationServerIT.groovy @@ -87,7 +87,7 @@ class AuthorizationServerIT extends ITSupport { assertPresent(client.listAuthorizationServers(), createdAuthorizationServer) } - @Test (groups = "group3") + @Test void getAuthorizationServerTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -106,7 +106,7 @@ class AuthorizationServerIT extends ITSupport { assertThat(retrievedAuthorizationServer.getDescription(), equalTo(createdAuthorizationServer.getDescription())) } - @Test (groups = "group3") + @Test void updateAuthorizationServerTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -126,7 +126,7 @@ class AuthorizationServerIT extends ITSupport { assertThat(updatedAuthorizationServer.getDescription(), equalTo("Updated Test Authorization Server")) } - @Test (groups = "group3") + @Test void deleteAuthorizationServerTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -149,7 +149,7 @@ class AuthorizationServerIT extends ITSupport { assertNotPresent(client.listAuthorizationServers(), createdAuthorizationServer) } - @Test (groups = "group3") + @Test void deactivateAuthorizationServerTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -174,7 +174,7 @@ class AuthorizationServerIT extends ITSupport { assertThat(retrievedAuthorizationServer.getStatus(), equalTo(AuthorizationServer.StatusEnum.INACTIVE)) } - @Test (groups = "group3") + @Test void activateAuthorizationServerTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -210,7 +210,6 @@ class AuthorizationServerIT extends ITSupport { } // Policy operations - @Test (groups = "group3") void listAuthorizationServerPoliciesTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -245,7 +244,7 @@ class AuthorizationServerIT extends ITSupport { }) } - @Test (groups = "group3") + @Test void createAuthorizationServerPolicyTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -279,7 +278,7 @@ class AuthorizationServerIT extends ITSupport { assertThat(retrievedPolicy.getName(), equalTo(createdPolicy.getName())) } - @Test (groups = "group3") + @Test void updateAuthorizationServerPolicyTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -316,7 +315,7 @@ class AuthorizationServerIT extends ITSupport { assertThat(updatedPolicy.getDescription(), equalTo("Test Policy Updated")) } - @Test (groups = "group3") + @Test void deleteAuthorizationServerPolicyTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -351,7 +350,7 @@ class AuthorizationServerIT extends ITSupport { assertNotPresent(createdAuthorizationServer.listPolicies(), createdPolicy) } - @Test (groups = "group3") + @Test void activateDeactivateAuthorizationServerPolicyTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -396,7 +395,7 @@ class AuthorizationServerIT extends ITSupport { assertNotPresent(createdAuthorizationServer.listPolicies(), activatedPolicy) } - @Test (groups = "group3") + @Test void listAuthorizationServerPolicyRulesTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -428,7 +427,7 @@ class AuthorizationServerIT extends ITSupport { }) } - @Test (groups = "group3") + @Test void createAuthorizationServerPolicyRuleTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -595,7 +594,7 @@ class AuthorizationServerIT extends ITSupport { assertThat(retrievedPolicyRule.getActions().getToken().getRefreshTokenWindowMinutes(), equalTo(55)) } - @Test (groups = "group3") + @Test void deleteAuthorizationServerPolicyRuleTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -652,7 +651,7 @@ class AuthorizationServerIT extends ITSupport { assertNotPresent(createdPolicy.listPolicyRules(createdAuthorizationServer.getId()), createdPolicyRule) } - @Test (groups = "group3") + @Test void activateDeactivateAuthorizationServerPolicyRuleTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -715,7 +714,7 @@ class AuthorizationServerIT extends ITSupport { // Scope operations - @Test (groups = "group3") + @Test void listOAuth2ScopesTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -795,7 +794,7 @@ class AuthorizationServerIT extends ITSupport { assertThat(updatedOAuth2Scope.getName(), equalTo(tobeUpdatedOAuth2Scope.getName())) } - @Test (groups = "group3") + @Test void deleteOAuth2ScopesTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -825,7 +824,7 @@ class AuthorizationServerIT extends ITSupport { // Claim operations - @Test (groups = "group3") + @Test void listOAuth2ClaimsTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -851,7 +850,7 @@ class AuthorizationServerIT extends ITSupport { assertPresent(createdAuthorizationServer.listOAuth2Claims(), createdOAuth2Claim) } - @Test (groups = "group3") + @Test void getOAuth2ClaimTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -879,7 +878,7 @@ class AuthorizationServerIT extends ITSupport { assertThat(retrievedOAuth2Claim.getId(), equalTo(createdOAuth2Claim.getId())) } - @Test (groups = "group3") + @Test void updateOAuth2ClaimTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/EventHooksIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/EventHooksIT.groovy index 8ff22337bc8..9ae8c732643 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/EventHooksIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/EventHooksIT.groovy @@ -38,7 +38,7 @@ import static com.okta.sdk.tests.it.util.Util.assertPresent */ class EventHooksIT extends ITSupport { - @Test (groups = "group3") + @Test (groups = "group1") void createEventHookTest() { String name = "java-sdk-it-${UUID.randomUUID().toString()}" @@ -59,7 +59,7 @@ class EventHooksIT extends ITSupport { createdEventHook.deactivate() } - @Test (groups = "group3") + @Test void getEventHookTest() { String name = "java-sdk-it-${UUID.randomUUID().toString()}" @@ -84,7 +84,7 @@ class EventHooksIT extends ITSupport { createdEventHook.deactivate() } - @Test (groups = "group3") + @Test void updateEventHookTest() { String name = "java-sdk-it-${UUID.randomUUID().toString()}" @@ -118,7 +118,7 @@ class EventHooksIT extends ITSupport { createdEventHook.deactivate() } - @Test (groups = "group3") + @Test void deleteEventHookTest() { String name = "java-sdk-it-${UUID.randomUUID().toString()}" @@ -147,7 +147,7 @@ class EventHooksIT extends ITSupport { } } - @Test (groups = "group3") + @Test (groups = "group1") void listAllEventHooksTest() { String name = "java-sdk-it-${UUID.randomUUID().toString()}" @@ -166,7 +166,7 @@ class EventHooksIT extends ITSupport { createdEventHook.deactivate() } - @Test (groups = "group3") + @Test void activateDeactivateEventHookTest() { String name = "java-sdk-it-${UUID.randomUUID().toString()}" diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/FactorsIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/FactorsIT.groovy index e7dc3c562e7..5745d0f41d5 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/FactorsIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/FactorsIT.groovy @@ -48,7 +48,7 @@ class FactorsIT extends ITSupport { private String smsTestNumber = "185 635 15491" - @Test (groups = "group2") + @Test (groups = "group3") void factorListTest() { Client client = getClient() @@ -79,7 +79,7 @@ class FactorsIT extends ITSupport { ) } - @Test (groups = "group2") + @Test void testSecurityQuestionFactorCreation() { Client client = getClient() User user = randomUser() @@ -95,7 +95,7 @@ class FactorsIT extends ITSupport { assertThat securityQuestionUserFactor.id, notNullValue() } - @Test (groups = "group2") + @Test void testCallFactorCreation() { Client client = getClient() User user = randomUser() @@ -109,7 +109,7 @@ class FactorsIT extends ITSupport { assertThat callUserFactor.id, notNullValue() } - @Test (groups = "group2") + @Test void testSmsFactorCreation() { Client client = getClient() User user = randomUser() @@ -123,7 +123,7 @@ class FactorsIT extends ITSupport { assertThat smsUserFactor.id, notNullValue() } - @Test (groups = "group2") + @Test void testPushFactorCreation() { Client client = getClient() User user = randomUser() @@ -135,21 +135,21 @@ class FactorsIT extends ITSupport { assertThat pushUserFactor.id, notNullValue() } - @Test (groups = "group2") + @Test void testListSecurityQuestionsNotEmpty() { User user = randomUser() SecurityQuestionList securityQuestions = user.listSupportedSecurityQuestions() assertThat securityQuestions, iterableWithSize(greaterThan(1)) } - @Test (groups = "group2") + @Test void testAvailableFactorsNotEmpty() { User user = randomUser() UserFactorList factors = user.listSupportedFactors() assertThat factors, iterableWithSize(greaterThan(1)) } - @Test (groups = "group2") + @Test (groups = "group3") void activateTotpFactor() { User user = randomUser() assertListFactors(user) @@ -167,7 +167,7 @@ class FactorsIT extends ITSupport { assertThat factorResult, instanceOf(TotpUserFactor) } - @Test (groups = "group2") + @Test void verifyQuestionFactor() { User user = randomUser() @@ -184,7 +184,7 @@ class FactorsIT extends ITSupport { assertThat response.getFactorResult(), is(VerifyUserFactorResponse.FactorResultEnum.SUCCESS) } - @Test (groups = "group2") + @Test void verifyQuestionFactor_withoutBody() { User user = randomUser() @@ -200,7 +200,7 @@ class FactorsIT extends ITSupport { } @NonOIEEnvironmentOnly - @Test (groups = "group2") + @Test (groups = "group3") void testEmailUserFactor() { User user = randomUser() assertThat user.listFactors(), emptyIterable() @@ -223,7 +223,7 @@ class FactorsIT extends ITSupport { assertThat response.getFactorResult(), is(VerifyUserFactorResponse.FactorResultEnum.CHALLENGE) } - @Test (groups = "group2") + @Test (groups = "group3") void testGoogleTotpUserFactorCreation() { User user = randomUser() assertListFactors(user) @@ -238,7 +238,7 @@ class FactorsIT extends ITSupport { assertThat tokenUserFactor.getStatus(), is(FactorStatus.PENDING_ACTIVATION) } - @Test (groups = "group2") + @Test void deleteFactorTest() { User user = randomUser() diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/FeaturesIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/FeaturesIT.groovy index acb82bfff35..6a1a8912965 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/FeaturesIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/FeaturesIT.groovy @@ -29,7 +29,7 @@ import static org.hamcrest.Matchers.* */ class FeaturesIT extends ITSupport { - @Test (groups = "group2") + @Test void featureOperationsTest() { // list features FeatureList featureList = client.listFeatures() diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupRolesIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupRolesIT.groovy index 60597617cd1..027d32c73c6 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupRolesIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupRolesIT.groovy @@ -38,7 +38,7 @@ import static com.okta.sdk.tests.it.util.Util.validateGroup */ class GroupRolesIT extends ITSupport { - @Test (groups = "group2") + @Test (groups = "group1") @Scenario("list-roles-assigned-to-group") void listRolesAssignedToGroupTest() { @@ -76,7 +76,7 @@ class GroupRolesIT extends ITSupport { assertRolePresent(roles, appAdminRole) } - @Test (groups = "group2") + @Test @Scenario("unassigned-roles-for-group") void unassignRoleForGroupTest() { @@ -110,7 +110,7 @@ class GroupRolesIT extends ITSupport { assertRoleAbsent(client.listGroupAssignedRoles(createdGroup.getId()), userAdminRole) } - @Test (groups = "group2") + @Test @Scenario("list-group-targets-for-group") void listGroupTargetsForGroupTest() { @@ -145,7 +145,7 @@ class GroupRolesIT extends ITSupport { assertGroupPresent(client.listGroupTargetsForGroupRole(createdGroup1.getId(), userAdminRole.getId()), createdGroup2) } - @Test (groups = "group2") + @Test @Scenario("list-group-targets-for-group") void removeGroupTargetFromGroupAdministratorRoleGivenToGroupTest() { diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupSchemaIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupSchemaIT.groovy index 64470640562..064f135e74f 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupSchemaIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupSchemaIT.groovy @@ -37,7 +37,7 @@ import static org.hamcrest.Matchers.* */ class GroupSchemaIT extends ITSupport { - @Test (groups = "group1") + @Test @Scenario("get-group-schema") void getGroupSchemaTest() { diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupsIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupsIT.groovy index eecb7928680..0217e885f4f 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupsIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/GroupsIT.groovy @@ -67,7 +67,7 @@ class GroupsIT extends ITSupport implements CrudTestSupport { return client.listGroups().iterator() } - @Test (groups = "group2") + @Test (groups = "group1") @Scenario("list-groups") void listGroupsTest() { @@ -85,7 +85,7 @@ class GroupsIT extends ITSupport implements CrudTestSupport { assertGroupPresent(client.listGroups(), createdGroup) } - @Test (groups = "group2") + @Test (groups = "group1") @Scenario("search-groups") void searchGroupTest() { @@ -120,7 +120,7 @@ class GroupsIT extends ITSupport implements CrudTestSupport { assertPresent(client.listGroups(null, "profile.name eq \"" + groupName + "\"", null), group) } - @Test (groups = "group2") + @Test @Scenario("update-group") void updateGroupTest() { @@ -142,7 +142,7 @@ class GroupsIT extends ITSupport implements CrudTestSupport { validateGroup(group, groupNameUpdated, 'Description updated') } - @Test (groups = "group2") + @Test (groups = "group1") @Scenario("group-user-operations") void groupUserOperationsTest() { diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/IdpIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/IdpIT.groovy index 9d4bc4c8645..f17fe5a133c 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/IdpIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/IdpIT.groovy @@ -167,7 +167,7 @@ class IdpIT extends ITSupport { assertNotPresent(client.listIdentityProviders(), createdIdp) } - @Test (groups = "group2") + @Test void oidcIdpUserTest() { // create user @@ -292,7 +292,7 @@ class IdpIT extends ITSupport { assertNotPresent(client.listIdentityProviders(), createdIdp) } - @Test (groups = "group2") + @Test void facebookIdpTest() { // create user @@ -401,7 +401,7 @@ class IdpIT extends ITSupport { assertNotPresent(client.listIdentityProviders(), createdIdp) } - @Test (groups = "group2") + @Test void linkedInIdpTest() { // create user @@ -455,7 +455,7 @@ class IdpIT extends ITSupport { assertNotPresent(client.listIdentityProviders(), createdIdp) } - @Test (groups = "group2") + @Test void idpWithStringTypeTest() { // create user diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/InlineHooksIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/InlineHooksIT.groovy index fd30891ef68..b8893b5f6c0 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/InlineHooksIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/InlineHooksIT.groovy @@ -59,7 +59,7 @@ class InlineHooksIT extends ITSupport { createdInlineHook.deactivate() } - @Test (groups = "group2") + @Test void getInlineHookTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -84,7 +84,7 @@ class InlineHooksIT extends ITSupport { createdInlineHook.deactivate() } - @Test (groups = "group2") + @Test void updateInlineHookTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -124,7 +124,7 @@ class InlineHooksIT extends ITSupport { updatedInlineHook.deactivate() } - @Test (groups = "group2") + @Test void deleteInlineHookTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() @@ -150,7 +150,7 @@ class InlineHooksIT extends ITSupport { assertNotPresent(client.listInlineHooks(), createdInlineHook) } - @Test (groups = "group2") + @Test void listAllInlineHooksTest() { String name = "java-sdk-it-" + UUID.randomUUID().toString() diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/LinkedObjectsIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/LinkedObjectsIT.groovy index 5efd9e9972d..16d00d7bcb8 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/LinkedObjectsIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/LinkedObjectsIT.groovy @@ -33,7 +33,7 @@ import static org.hamcrest.Matchers.notNullValue */ class LinkedObjectsIT extends ITSupport { - @Test (groups = "group2") + @Test(groups = "group3") void addLinkedObjectDefinitionTest() { String primaryName = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(25) String associatedName = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(25) @@ -69,7 +69,7 @@ class LinkedObjectsIT extends ITSupport { assertThat(createdLinkedObjectDefinition.getAssociated().getType(), equalTo(LinkedObjectDetailsType.USER)) } - @Test (groups = "group2") + @Test void getLinkedObjectDefinitionByPrimaryNameTest() { String primaryName = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(25) String associatedName = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(25) @@ -107,7 +107,7 @@ class LinkedObjectsIT extends ITSupport { assertThat(retrievedLinkedObject.getAssociated().getType(), equalTo(LinkedObjectDetailsType.USER)) } - @Test (groups = "group2") + @Test void getLinkedObjectDefinitionByAssociatedNameTest() { String primaryName = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(25) String associatedName = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(25) @@ -145,7 +145,7 @@ class LinkedObjectsIT extends ITSupport { assertThat(retrievedLinkedObject.getAssociated().getType(), equalTo(LinkedObjectDetailsType.USER)) } - @Test (groups = "group2") + @Test void getAllLinkedObjectDefinitionsTest() { // create first linked object definition diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/NetworkZoneIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/NetworkZoneIT.groovy index ded3b65817a..2001d3fb55b 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/NetworkZoneIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/NetworkZoneIT.groovy @@ -30,7 +30,7 @@ import static org.hamcrest.Matchers.* */ class NetworkZoneIT extends ITSupport { - @Test (groups = "group2") + @Test void listNetworkZonesTest() { getClient().listNetworkZones() @@ -88,7 +88,7 @@ class NetworkZoneIT extends ITSupport { assertNotPresent(getClient().listNetworkZones(), createdNetworkZone) } - @Test (groups = "group2") + @Test void updateNetworkZoneTest() { String networkZoneName = "network-zone-it-${uniqueTestName}" diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/OrgSettingsIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/OrgSettingsIT.groovy index da1c377abf5..76dbca7d0ee 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/OrgSettingsIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/OrgSettingsIT.groovy @@ -39,7 +39,7 @@ import static org.hamcrest.Matchers.* */ class OrgSettingsIT extends ITSupport { - @Test (groups = "group1") + @Test (groups = "group3") @Scenario("get-partial-update-org-settings") void getPartialUpdateOrgSettingsTest() { @@ -106,7 +106,7 @@ class OrgSettingsIT extends ITSupport { orgSetting.partialUpdate() } - @Test (groups = "group2") + @Test @Scenario("get-org-contacts") void getOrgContactsTest() { def contactTypes = client.getOrgSettings().getContactTypes() @@ -118,7 +118,7 @@ class OrgSettingsIT extends ITSupport { }) } - @Test (groups = "group3") + @Test @Scenario("get-user-of-contact-type-test") void getUserOfContactTypeTest() { def orgContactUser = client.getOrgSettings().getOrgContactUser(OrgContactType.BILLING.toString()) @@ -152,7 +152,7 @@ class OrgSettingsIT extends ITSupport { assertThat(updatedOrgContactUser.getUserId(), equalTo(billingUser.getId())) } - @Test (groups = "group1") + @Test @Scenario("get-org-preferences-test") void getOrgPreferencesTest() { def orgPreferences = client.getOrgSettings().orgPreferences() @@ -176,7 +176,7 @@ class OrgSettingsIT extends ITSupport { } } - @Test (groups = "group2") + @Test @Scenario("get-okta-communication-settings-test") void getOktaCommunicationSettingsTest() { def commSettings = client.getOrgSettings().communicationSettings() @@ -200,7 +200,7 @@ class OrgSettingsIT extends ITSupport { } } - @Test (groups = "group3") + @Test @Scenario("get-org-okta-support-settings-test") void getOrgOktaSupportSettingsTest() { def supportSettings = client.getOrgSettings().getSupportSettings() diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/PolicyRulesIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/PolicyRulesIT.groovy index 99d1e12a193..947c6f660f3 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/PolicyRulesIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/PolicyRulesIT.groovy @@ -81,7 +81,7 @@ class PolicyRulesIT extends ITSupport implements CrudTestSupport { return crudTestPolicy.listPolicyRules().iterator() } - @Test (groups = "group2") + @Test void deactivateTest() { def group = randomGroup() @@ -105,7 +105,7 @@ class PolicyRulesIT extends ITSupport implements CrudTestSupport { assertThat(policyRule.getStatus(), is(PolicyRule.StatusEnum.INACTIVE)) } - @Test (groups = "group2") + @Test void listPolicyRulesTest() { def group = randomGroup() def policy = randomSignOnPolicy(group.getId()) diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/ThreatInsightConfigurationIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/ThreatInsightConfigurationIT.groovy index d75ed8bb0b0..372e9df3024 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/ThreatInsightConfigurationIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/ThreatInsightConfigurationIT.groovy @@ -37,7 +37,7 @@ import static org.hamcrest.Matchers.notNullValue */ class ThreatInsightConfigurationIT extends ITSupport { - @Test (groups = "group3") + @Test void getThreatInsightConfigurationTest() { ThreatInsightConfiguration currentConfiguration = getClient().getCurrentConfiguration() diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/UserTypesIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/UserTypesIT.groovy index 5f3f11da21f..94599bb231f 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/UserTypesIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/UserTypesIT.groovy @@ -33,7 +33,7 @@ import static org.hamcrest.Matchers.notNullValue */ class UserTypesIT extends ITSupport { - @Test (groups = "group2") + @Test void createUserTypeTest() { String name = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(15) @@ -64,7 +64,7 @@ class UserTypesIT extends ITSupport { }) } - @Test (groups = "group2") + @Test void getUserTypeTest() { String name = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(15) @@ -81,7 +81,7 @@ class UserTypesIT extends ITSupport { assertThat(retrievedUserType.getName(), equalTo(createdUserType.getName())) } - @Test (groups = "group2") + @Test void updateUserTypeTest() { String name = "java_sdk_it_" + RandomStringUtils.randomAlphanumeric(15) diff --git a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/UsersIT.groovy b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/UsersIT.groovy index be1ffe65fdd..07d341123cb 100644 --- a/integration-tests/src/test/groovy/com/okta/sdk/tests/it/UsersIT.groovy +++ b/integration-tests/src/test/groovy/com/okta/sdk/tests/it/UsersIT.groovy @@ -79,7 +79,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { ensureCustomProperties() } - @Test (groups = "group2") + @Test void userProfileNumberValues() { def email = "joe.coder+${uniqueTestName}@example.com" @@ -182,7 +182,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { return client.listUsers().iterator() } - @Test (groups = "group2") + @Test (groups = "group3") @Scenario("create-user-with-user-type") void createUserWithUserTypeTest() { @@ -226,7 +226,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertPresent(users, user) } - @Test (groups = "group2") + @Test (groups = "group3") @Scenario("user-activate") void userActivateTest() { @@ -256,7 +256,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertPresent(users, user) } - @Test (groups = "group2") + @Test @Scenario("user-with-special-character") void userWithSpecialCharacterTest() { def password = 'Passw0rd!2@3#' @@ -277,7 +277,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat(client.getUser(email), equalTo(user)) } - @Test (groups = "group2") + @Test (groups = "group3") @Scenario("user-role-assign") void roleAssignTest() { @@ -316,7 +316,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertNotPresent(user.listAssignedRoles(), role) } - @Test (groups = "group2") + @Test (groups = "group3") @Scenario("user-change-password") void changePasswordTest() { @@ -347,7 +347,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { client.getUser(user.getId()) } - @Test (groups = "group2") + @Test void changeStrictPasswordTest() { def password = 'Passw0rd!2@3#' @@ -406,7 +406,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { client.getUser(user.getId()) } - @Test(expectedExceptions = ResourceException, groups = "group2") + @Test(expectedExceptions = ResourceException, groups = "group3") @Scenario("user-change-recovery-question") void changeRecoveryQuestionTest() { @@ -472,7 +472,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat response.getResetPasswordUrl(), containsString("/reset_password/") } - @Test (groups = "group2") + @Test @Scenario("user-expire-password") void expirePasswordTest() { @@ -499,7 +499,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { } - @Test (groups = "group2") + @Test @Scenario("user-get-reset-password-url") void resetPasswordUrlTest() { @@ -524,7 +524,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat token.getResetPasswordUrl(), notNullValue() } - @Test (groups = "group2") + @Test (groups = "group3") @Scenario("user-get") void getUserTest() { @@ -562,7 +562,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { } } - @Test (groups = "group2") + @Test @Scenario("user-group-target-role") void groupTargetRoleTest() { @@ -620,7 +620,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertGroupTargetPresent(user, group, role) } - @Test (groups = "group2") + @Test (groups = "group3") @Scenario("user-profile-update") void userProfileUpdate() { @@ -652,7 +652,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat(updatedUser.getProfile().get("nickName"), equalTo("Batman")) } - @Test (groups = "group2") + @Test @Scenario("user-suspend") void userSuspendTest() { @@ -685,7 +685,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertPresent(client.listUsers(null, 'status eq \"ACTIVE\"', null, null, null), user) } - @Test (groups = "group2") + @Test void getUserInvalidUserId() { try { def userId = "invalid-user-id-${uniqueTestName}@example.com" @@ -696,7 +696,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { } } - @Test (groups = "group2") + @Test void getUserNullUserId() { try { getClient().getUser(null) @@ -706,7 +706,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { } } - @Test (groups = "group2") + @Test void getUserEmptyUserId() { try { getClient().getUser("") @@ -716,7 +716,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { } } - @Test (groups = "group2") + @Test (groups = "group3") void createUserWithGroups() { def groupName = "Group-to-Assign-${uniqueTestName}" @@ -750,7 +750,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat groups.get(1).getId(), equalTo(group.id) } - @Test (groups = "group3") + @Test void setUserPasswordWithoutReset() { def user = randomUser() @@ -770,7 +770,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat(result, instanceOf(User)) } - @Test (groups = "group3") + @Test void importUserWithSha512Password() { def salt = "aSalt" @@ -789,7 +789,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat user.getCredentials().getProvider().getType(), is(AuthenticationProviderType.IMPORT) } - @Test (groups = "group3") + @Test @Scenario("user-forgot-password-generate-one-time-token") void forgotPasswordGenerateOttTest() { @@ -815,7 +815,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat response.getResetPasswordUrl(), containsString("/signin/reset-password/") } - @Test (groups = "group3") + @Test @Scenario("user-forgot-password-set-new-password") void forgotPasswordSetNewPasswordTest() { @@ -850,7 +850,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat response.get("provider")["name"], equalTo("OKTA") } - @Test (groups = "group3") + @Test void userWithAuthenticationProviderTest() { def firstName = 'John' @@ -877,7 +877,7 @@ class UsersIT extends ITSupport implements CrudTestSupport { assertThat user.getCredentials().getProvider().getName(), equalTo(AuthenticationProviderType.FEDERATION.name()) } - @Test (groups = "group3") + @Test void testGetNextPageUrl() { def user1 = create(client) def user2 = create(client)