Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update integration tests to use REST APIs #15906

Merged
merged 71 commits into from
Jun 21, 2023

Conversation

ashanthamara
Copy link
Contributor

@ashanthamara ashanthamara commented May 24, 2023

This PR will remove the soap based api implementations used in IS Integration Tests

  • Added RestBaseClient for the common rest functions used in restapi
  • Added OAuth2RestClient for application management rest api implementation 681f334
  • Added SCIM2RestClient for SCIM2 rest api implementation 8186bac
  • Added AuthenticatorRestClient dc1aaa4

Modified following test cases

OAuth2

  • OAuth2RoleClaimTestCase 64b443f
  • OAuth2DeviceFlowTestCase df9d509
  • OAuth2RequestObjectSignatureValidationTestCase b3df124
  • OAuth2IDTokenEncryptionTestCase c32e11c
  • OAuth2ServiceErrorResponseTest 4d4a1a2
  • OAuth2ServiceRefreshTokenGrantTestCase 03cc833
  • PermissionBasedScopeValidatorTestCase a3e44ec
  • OAuth2ServiceClientCredentialTestCase bb891d7
  • OAuth2ServiceSAML2BearerGrantTestCase cd7f8d4
  • OAuth2ServiceImplicitGrantTestCase - f29616c
  • OAuth2ServiceIntrospectionTestCase 63cde8c
  • OAuth2ServiceResourceOwnerTestCase 945201d
  • OAuth2TokenRevocationWithRevokedAccessToken 930ea81
  • OAuth2BackChannelLogoutTestCase 16d1ab2
  • OAuth2TokenRevocationWithMultipleSessionTerminationTestCase a5afc99
  • OAuth2TokenRevocationAfterAccountDisablingTestCase 0daef8b
  • Oauth2OPIframeTestCase 88b91e2

Identity/governance

  • AccountLockEnabledTestCase 12a53c5

Related Issue

Copy link
Member

@madurangasiriwardena madurangasiriwardena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you update the wso2carbon.jks file?

@ashanthamara ashanthamara self-assigned this Jun 19, 2023
@jenkins-is-staging
Copy link
Contributor

PR builder [7.x.x] started
Link: https://github.com/wso2/product-is/actions/runs/5330101015

@jenkins-is-staging
Copy link
Contributor

PR builder [7.x.x] completed
Link: https://github.com/wso2/product-is/actions/runs/5330101015
Status: success

Copy link
Contributor

@jenkins-is-staging jenkins-is-staging left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/5330101015

@madurangasiriwardena madurangasiriwardena changed the title Soap Service Removal Update integration tests to use REST APIs Jun 21, 2023
@@ -211,6 +301,8 @@ public OAuthConsumerAppDTO createApplication(OAuthConsumerAppDTO appDTO, String
return appDtoResult;
}


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove additional line


updateApplication(appId, applicationPatch);
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Additional line

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants