Skip to content

Conversation

@kraftbj
Copy link
Contributor

@kraftbj kraftbj commented Aug 14, 2020

Our testing now uses phpunit.xml.dist which was not updated in package's .gitattributes files.

Fixes issue introduced in #16810

Changes proposed in this Pull Request:

  • n/a

Jetpack product discussion

n/a

Does this pull request change what data or activity we track or use?

n/a

Testing instructions:

  • Done pre-release. After a new version of a package has this update, should be able to do composer install --no-dev on a project with one of these packages and not see phpunit.xml.dist in the distributed copy.

Proposed changelog entry for your changes:

  • n/a

@kraftbj kraftbj added [Type] Bug When a feature is broken and / or not performing as intended [Status] Needs Review This PR is ready for review. Unit Tests [Pri] Normal [Focus] Jetpack DNA labels Aug 14, 2020
@kraftbj kraftbj requested a review from jeherve August 14, 2020 14:14
@kraftbj kraftbj self-assigned this Aug 14, 2020
Copy link

@test-case-reminder test-case-reminder bot left a comment

Choose a reason for hiding this comment

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

Here are some suggested test cases for this PR.

Connection

  • In-place connection with free plan
  • In-place connection with paid plan
  • In-place connection with product purchase
  • Classic connection. Use Safari, or set a constant JETPACK_SHOULD_NOT_USE_CONNECTION_IFRAME to true
  • Disconnect/reconnect connection
  • Secondary user connection
  • Connection on multisite

If you think that suggestions should be improved please edit the configuration file here. You can also modify/add test-suites to be used in the configuration file.

@kraftbj kraftbj added this to the 8.9 milestone Aug 14, 2020
@jetpackbot
Copy link
Collaborator

Thank you for the great PR description!

When this PR is ready for review, please apply the [Status] Needs Review label. If you are an a11n, please have someone from your team review the code if possible. The Jetpack team will also review this PR and merge it to be included in the next Jetpack release.

E2E results is available here (for debugging purposes): https://jetpack-e2e-dashboard.herokuapp.com/pr-16838

Scheduled Jetpack release: September 1, 2020.
Scheduled code freeze: August 25, 2020

Generated by 🚫 dangerJS against e308537

@jeherve jeherve added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Aug 14, 2020
@kraftbj kraftbj merged commit 41a633d into master Aug 14, 2020
@kraftbj kraftbj deleted the fix/phpunitxml branch August 14, 2020 20:48
@matticbot matticbot added [Status] Needs Changelog [Status] Needs Package Release This PR made changes to a package. Let's update that package now. and removed [Status] Ready to Merge Go ahead, you can push that green button! labels Aug 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Focus] Jetpack DNA [Pri] Normal [Status] Needs Package Release This PR made changes to a package. Let's update that package now. [Type] Bug When a feature is broken and / or not performing as intended Unit Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants