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

Enabled local E2E optout tests #2

Merged
merged 5 commits into from
Dec 11, 2023
Merged

Enabled local E2E optout tests #2

merged 5 commits into from
Dec 11, 2023

Conversation

gmsdelmundo
Copy link
Collaborator

No description provided.

@gmsdelmundo gmsdelmundo force-pushed the main branch 4 times, most recently from 91d1ae8 to 02a782e Compare December 7, 2023 07:56
@@ -7,7 +7,8 @@
@Suite
@SelectClasses({
BasicTestSuite.class,
V2ApiOperatorTest.class
V2ApiOperatorTest.class,
OptoutTestSuite.class

Choose a reason for hiding this comment

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

Private operators cannot log out but should respect log out. How can we test the latter only?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is already covered in V2ApiOperatorTest

@gmsdelmundo gmsdelmundo merged commit d0b406a into main Dec 11, 2023
1 check passed
@gmsdelmundo gmsdelmundo deleted the gdm-UID2-988-optout branch December 11, 2023 09:43
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.

2 participants