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

Maintainer PR to run tests on external contributor PR #18485 #18488

Merged
merged 2 commits into from
May 3, 2022

Conversation

mokagio
Copy link
Contributor

@mokagio mokagio commented May 2, 2022

Refer to original PR for all the details and conversation: #18485.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented May 2, 2022

You can test the changes in Jetpack from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr18488-c6873f2 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented May 2, 2022

You can test the changes in WordPress from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr18488-c6873f2 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

They are the same object in runtime, since `TestContextManager`
overrides `ContextManager.sharedInstance` with a mock instance.
@mokagio mokagio merged commit 90cc17e into trunk May 3, 2022
@mokagio mokagio deleted the trial/crazytonyli/18485 branch May 3, 2022 04:38
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