-
Notifications
You must be signed in to change notification settings - Fork 7
Multitenant Integration tests setup #213
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
Conversation
| TENANCY_MODEL=multi TENANT=TENANT2 npm run integration-test | ||
| echo "" | ||
| echo " All integration tests completed successfully!" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a deploy+integration test job like Java?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is only Integration test job, earlier there was only for single tenant, Now It runs for both single and multitenant
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For single tenant it is a single job, so can we do the same for multitenant as well? Otherwise we will have to track an extra job after PR merge
| echo "========================================" | ||
| echo " Running Multi-Tenant Integration Tests (Tenant 1)" | ||
| echo "========================================" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please specify the name of the Tenant instead of Tenant 1, like SDM-DEV-CONSUMER-EU12
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The problem with adding it here is I think it won't be printed if it is being run from local system. So can we print it in the code itself?
rishikunnath2747
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replied to some comments
Describe your changes
Automation tests for Multitenant scenario
1)To run Integration test for single tenant use this command
TENANCY_MODEL=single npm run integration-test
2)To run Integration test for Multi tenant (tenant 1) use this command
TENANCY_MODEL=multi TENANT=SDM-DEV-CONSUMER-EU12 npm run integration-test
3)To run Integration test for Multi tenant (tenant 2) use this command
TENANCY_MODEL=multi TENANT=SDMGoogleWorkspaceConsumer npm run integration-test
Any documentation
wiki documentation - https://wiki.one.int.sap/wiki/display/sapecm/Integration+testing+for+SDM+CAP+Plugin
Type of change
Please delete options that are not relevant.
Checklist before requesting a review
Upload Screenshots/lists of the scenarios tested
single tenant

multi tenant 1
multi tenant 2