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

Add basic GB upgrade (regression) tests for the Premium Content block #47723

Merged
merged 2 commits into from
Dec 1, 2020

Conversation

fullofcaffeine
Copy link
Contributor

@fullofcaffeine fullofcaffeine commented Nov 25, 2020

Changes proposed in this Pull Request

Add basic GB upgrade tests for the Premium Content block:

  • Make sure it doesn't error in the editor and that it renders in the frontend;
  • Make sure it works well after upgrade to GB edge (this is done automatically as part of the upgrade test).

This aims at preventing the regression of issues like #47467 in the future.

Testing instructions

Observations

This branches off this PR (since it hasn't been merged yet). The specific GB 9.4.1 fixes there are needed because we are currently running 9.4.1 on edge sites. Upgrade tests need to run on edge sites too.

Stripe connection

This required me to setup a Stripe connection for the Premium Content block for all the sites under the gutenbergUpgradeUser (see your local-decrypted.json) WPCOM account. Stripe test account info was kindly handed to me by @apeatling.

Closes: #47552

@matticbot
Copy link
Contributor

@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 56e7c30 to 5ccbfd7 Compare November 25, 2020 01:35
@fullofcaffeine fullofcaffeine marked this pull request as ready for review November 25, 2020 01:41
@fullofcaffeine fullofcaffeine self-assigned this Nov 25, 2020
@fullofcaffeine fullofcaffeine added the [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot label Nov 25, 2020
@matticbot
Copy link
Contributor

This PR does not affect the size of JS and CSS bundles shipped to the user's browser.

Generated by performance advisor bot at iscalypsofastyet.com.

@fullofcaffeine fullofcaffeine added [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot and removed [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot labels Nov 25, 2020
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 5ccbfd7 to 9176f2c Compare November 25, 2020 19:54
@fullofcaffeine fullofcaffeine added [Status] Needs e2e Testing and removed [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot [Status] Needs e2e Testing labels Nov 25, 2020
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 9176f2c to 1b184c6 Compare November 25, 2020 23:29
@fullofcaffeine fullofcaffeine added [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot and removed [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot labels Nov 25, 2020
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 1b184c6 to 5867c77 Compare November 26, 2020 01:41
test/e2e/lib/components/secure-payment-component.js Outdated Show resolved Hide resolved
test/e2e/lib/gutenberg/gutenberg-editor-component.js Outdated Show resolved Hide resolved
test/e2e/lib/gutenberg/gutenberg-editor-component.js Outdated Show resolved Hide resolved
test/e2e/lib/gutenberg/gutenberg-editor-component.js Outdated Show resolved Hide resolved
test/e2e/specs/wp-calypso-gutenberg-post-editor-spec.js Outdated Show resolved Hide resolved
test/e2e/specs/wp-calypso-gutenberg-post-editor-spec.js Outdated Show resolved Hide resolved
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 5867c77 to 94d851c Compare November 26, 2020 23:24
@fullofcaffeine fullofcaffeine added [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. and removed [Status] Needs Author Reply [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot labels Nov 26, 2020
@fullofcaffeine fullofcaffeine added [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot and removed [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot labels Nov 27, 2020
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 4414549 to df2a449 Compare November 27, 2020 01:32
@fullofcaffeine fullofcaffeine added [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot and removed [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot labels Nov 27, 2020
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from df2a449 to 5023cfc Compare November 27, 2020 03:00
@fullofcaffeine fullofcaffeine added [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot and removed [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot labels Nov 27, 2020
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 5023cfc to 94e3ea5 Compare November 27, 2020 06:32
Copy link
Member

@WunderBart WunderBart left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

@WunderBart WunderBart added [Status] Ready to Merge and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Nov 27, 2020
@fullofcaffeine fullofcaffeine added [Status] Needs e2e Testing and removed [Status] Needs e2e Testing Gutenberg Edge Run e2e tests with a special site that is running the latest Gutenberg plugin snapshot [Status] Needs e2e Testing labels Nov 27, 2020
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 94e3ea5 to 9297b48 Compare December 1, 2020 00:01
- Make sure it doesn't error in the editor and that it renders in the frontend;
- Make sure it works well after upgrade to GB edge.
@fullofcaffeine fullofcaffeine force-pushed the add/gb-upgrade-test-for-premium-block branch from 9297b48 to c0e1092 Compare December 1, 2020 01:25
@fullofcaffeine fullofcaffeine merged commit 961a9bd into trunk Dec 1, 2020
@fullofcaffeine fullofcaffeine deleted the add/gb-upgrade-test-for-premium-block branch December 1, 2020 02:04
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.

Premium Content Block: Add basic e2e test
3 participants