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

Mock all embed tests and rename embed class files #4474

Merged
merged 4 commits into from
Apr 9, 2020

Conversation

pierlon
Copy link
Contributor

@pierlon pierlon commented Mar 31, 2020

Summary

As pointed out in #4384 (comment), the Facebook embed tests are not being mocked. This PR fixes that, along with:

  • Fix mocking for Imgur embed tests
  • Standardizing embed handler file and class names, which also fixes the issue of some embed tests being ignored when the external-http test suite is run.

Checklist

  • My pull request is addressing an open issue (please create one otherwise).
  • My code is tested and passes existing tests.
  • My code follows the Engineering Guidelines (updates are often made to the guidelines, check it out periodically).

@googlebot googlebot added the cla: yes Signed the Google CLA label Mar 31, 2020
@pierlon pierlon changed the title Enhancement/mock embed tests Mock all embed tests Mar 31, 2020
@pierlon pierlon requested a review from westonruter March 31, 2020 02:45
@westonruter westonruter added this to the v1.5.3 milestone Apr 9, 2020
@westonruter
Copy link
Member

I'll cherry-pick the commits other than 6fd0c8a into the 1.5 branch.

@westonruter westonruter modified the milestones: v1.5.3, v1.6 Apr 9, 2020
@westonruter westonruter changed the title Mock all embed tests Mock all embed tests and rename embed classes Apr 9, 2020
@westonruter westonruter changed the title Mock all embed tests and rename embed classes Mock all embed tests and rename embed class files Apr 9, 2020
@westonruter westonruter merged commit f6710a0 into develop Apr 9, 2020
@westonruter westonruter deleted the enhancement/mock-embed-tests branch April 9, 2020 01:28
westonruter pushed a commit that referenced this pull request Apr 9, 2020
westonruter pushed a commit that referenced this pull request Apr 9, 2020
westonruter added a commit that referenced this pull request Apr 15, 2020
* tag '1.5.3':
  Bump 1.5.3
  Bump version to 1.5.3-RC1
  Fix handling of Mustache templates (#4583)
  Stub request based on test scenario (#4588)
  Update tests after block-library/style.css changes in Gutenberg 7.9 (#4579)
  Restrict doing plugin upgrade routine when not in admin (#4538)
  Add new accessibility sanitizer (#4535)
  Fix unit tests (#4564)
  Add button into Site Health to reenable CSS transient caching (#4522)
  Restore unification of multi-page post content in Reader mode (#4547)
  Prevent styles from being removed when in Customizer preview with Standard mode (#4553)
  Omit Jetpack from being activated during PHPUnit test runs (#4474)
  Mock Facebook embed tests (#4474)
  Mock Imgur embed tests (#4474)
  Use title case for Paired Browsing link in edit post screen (#4540)
  Ensure that validation query vars persist through redirects (#4544)
  Add requirements to plugin file header (#4543)
  Force status code of validation responses to be 200 (#4533)
  Update optimizer test specs (#4527)
  Bump 1.5.3-alpha
@kmyram kmyram modified the milestones: v1.6, v1.5.3 May 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Signed the Google CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants