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

Add API tests that covers managing actions with rules #22227

Merged
merged 4 commits into from
Jan 10, 2025

Conversation

malithie
Copy link
Member

@malithie malithie commented Jan 9, 2025

$subject.

Resolves #22197

@jenkins-is-staging
Copy link
Contributor

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12695106633

@jenkins-is-staging
Copy link
Contributor

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12695106633
Status: failure

@jenkins-is-staging
Copy link
Contributor

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12697404834

@jenkins-is-staging
Copy link
Contributor

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12697404834
Status: failure

@jenkins-is-staging
Copy link
Contributor

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12705640495

@jenkins-is-staging
Copy link
Contributor

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12705640495
Status: failure

@malithie
Copy link
Member Author

malithie commented Jan 10, 2025

PR builder completed Link: https://github.com/wso2/product-is/actions/runs/12705640495 Status: failure

Following failures are noted
[ERROR] OAuthAppsWithSameClientIdTestCase.testOAuthApplicationLoginSuccess:373->authenticateUser:633 SessionDataKeyConsent keyValues map is null. expected object to not be null [ERROR] OIDCPasswordGrantTest.testClear:84->OIDCAbstractIntegrationTest.deleteUser:115 User deletion failed expected:<204> but was:<404> [ERROR] OIDCPasswordGrantTest.testInit:73->OIDCAbstractIntegrationTest.createUser:95 User creation failed expected:<201> but was:<400>

  1. These test cases does not have any relation to the actions improvement added from this PR
  2. These test cases seems to failing in an intermittent manner in consecutive test executions triggered for change sets
  3. These tests passes locally with actions related tests in place.

Considering above reasons this PR will be merged

@malithie malithie merged commit f311a0f into wso2:master Jan 10, 2025
1 check passed
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.

Improve Action Management REST API to manage actions with rules
3 participants