Skip to content

Move tenant-aware integration tests to separate class #405

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

Merged
merged 3 commits into from
May 5, 2020

Conversation

micahstairs
Copy link
Contributor

Move tenant-aware integration tests to separate class. This simplifies the setup and teardown required for these tests.

@micahstairs micahstairs force-pushed the micahstairs-tenant-aware-it branch from 61103ae to 4e73766 Compare May 5, 2020 21:10
@micahstairs micahstairs changed the base branch from micahstairs-list-oidc to idp-config May 5, 2020 21:10
Copy link
Contributor

@hiranya911 hiranya911 left a comment

Choose a reason for hiding this comment

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

Thanks. LGTM 👍

@@ -0,0 +1,442 @@
/*
* Copyright 2017 Google Inc.
Copy link
Contributor

Choose a reason for hiding this comment

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

2020

@hiranya911
Copy link
Contributor

There's a compilation error detected by the CI. Please check.

@hiranya911 hiranya911 assigned micahstairs and unassigned hiranya911 May 5, 2020
@micahstairs micahstairs merged commit 5805fca into idp-config May 5, 2020
@micahstairs micahstairs deleted the micahstairs-tenant-aware-it branch May 5, 2020 21:41
micahstairs added a commit that referenced this pull request Jun 12, 2020
Move tenant-aware integration tests to separate class. This simplifies the setup and teardown required for these tests.
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.

3 participants