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 Trace ID Validation to Integration Tests for Organization Management Error Responses #20895

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

BimsaraBodaragama
Copy link
Member

Description

This pull request adds trace ID validation to the integration tests for error responses within the Organization Management API. The updates ensure that the test cases now verify the presence and format of trace IDs in error responses, improving traceability and supporting effective debugging.

Summary of Changes

  • Called the validateErrorResponse method, which includes trace ID validation, in the OrganizationManagementFailureTest.java file.

These changes contribute to better observability and align with best practices for error handling and logging within the codebase.

@jenkins-is-staging
Copy link
Contributor

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

@jenkins-is-staging
Copy link
Contributor

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/10366690441
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/10366690441

dhaura
dhaura previously approved these changes Aug 13, 2024
@SujanSanjula96 SujanSanjula96 merged commit d49c365 into wso2:master Aug 14, 2024
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.

4 participants