-
Notifications
You must be signed in to change notification settings - Fork 428
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
Multi tenancy tests #3108
Multi tenancy tests #3108
Conversation
- Common test allows this and looks up the option in subsequent files until it is found. - This will be used in tests for dynamic domains Also: - Do not read the properties multiple times in a row - Do not assume that 'mim' and 'reg' clusters are always present (reg will be disabled for multi-tenancy tests)
4779dad
to
6db0768
Compare
Codecov Report
@@ Coverage Diff @@
## master #3108 +/- ##
==========================================
+ Coverage 79.15% 79.17% +0.02%
==========================================
Files 387 387
Lines 31832 31832
==========================================
+ Hits 25196 25204 +8
+ Misses 6636 6628 -8
Continue to review full report at Codecov.
|
This is done in ct_mongoose_hook to be automatic and not require changing all test suites.
Run one suite for now, this will be extended in subsequent commits.
6db0768
to
fcd11e1
Compare
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.
looks good
Initial test configuration for dynamic domains. The plan is to extend them in subsequent PR's.
See individual commits for more details.