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

RSH1b5 New push.admin.device_registrations.removeWhere #73

Merged
merged 7 commits into from
Apr 12, 2019
Merged

Conversation

jdavid
Copy link
Contributor

@jdavid jdavid commented Mar 5, 2019

No description provided.

@jdavid
Copy link
Contributor Author

jdavid commented Mar 5, 2019

The push tests fail with the dev and idempotent-dev environments, so I've changed to the sandbox environment.

Then the idempotent tests fail since idempotency is not supported in sandbox, so I skip those tests if in the sandbox environment.

@jdavid
Copy link
Contributor Author

jdavid commented Mar 21, 2019

ping @paddybyers

Copy link
Member

@mattheworiordan mattheworiordan left a comment

Choose a reason for hiding this comment

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

Looks good, but think this PR still assumes idempotent is not available in sandbox

@jdavid
Copy link
Contributor Author

jdavid commented Apr 11, 2019

Not skipping idempotent tests in sandbox now.

Can I merge this? @mattheworiordan @paddybyers

Copy link
Member

@mattheworiordan mattheworiordan left a comment

Choose a reason for hiding this comment

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

LGTM. Know why tests are failing though?

@jdavid
Copy link
Contributor Author

jdavid commented Apr 12, 2019

Don't know. This same test was failing around December if I remember correctly, @paddybyers looked at it as well but didn't find why. Then the failure disappeared, and now it's back.

@jdavid jdavid merged commit ba123b7 into push Apr 12, 2019
@jdavid jdavid deleted the RSH1b5 branch April 12, 2019 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants