Skip to content

Mock all embed tests and rename embed class files#4474

Merged
westonruter merged 4 commits intodevelopfrom
enhancement/mock-embed-tests
Apr 9, 2020
Merged

Mock all embed tests and rename embed class files#4474
westonruter merged 4 commits intodevelopfrom
enhancement/mock-embed-tests

Conversation

@pierlon
Copy link
Copy Markdown
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
Copy Markdown
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