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: webhook handling for gutenberg edge label #82

Merged
merged 2 commits into from
Dec 9, 2019
Merged

Conversation

gwwar
Copy link
Contributor

@gwwar gwwar commented Dec 6, 2019

WIP

Let's trigger a full desktop and mobile E2E run using a special test site that has the gutenberg-edge blog sticker. This loads the latest version of the Gutenberg plugin we're testing in all views (iframe/wp-admin/published) so we should be able to add more types of E2E tests!

  • Create a new wp-calypso label [Status] Needs e2e Testing Gutenberg Edge
  • Create a new site, apply the gutenberg-edge sticker
  • Map new env options in E2E
  • Update this PR with the right options to pass through

Copy link
Contributor

@bsessions85 bsessions85 left a comment

Choose a reason for hiding this comment

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

Looks good! I made a couple minor changes. Thanks!

@gwwar
Copy link
Contributor Author

gwwar commented Dec 9, 2019

Thanks @bsessions85! Are there any follow up steps after merging?

@gwwar gwwar merged commit ecec0bb into master Dec 9, 2019
@gwwar gwwar deleted the add/gutenberg-edge branch December 9, 2019 18:09
@bsessions85
Copy link
Contributor

@gwwar I have calypso and wrapper repo changes out too. This should be ready to go!

@gwwar
Copy link
Contributor Author

gwwar commented Dec 9, 2019

Awesome, @bsessions85 we'll give it a try!

const envVars = { SKIP_DOMAIN_TESTS: true, GUTENBERG_EDGE: true };
description = 'The e2e full WPCOM suite desktop tests are running against your PR with the latest snapshot of Gutenberg';
log.info( 'Executing CALYPSO e2e full WPCOM suite desktop tests for with gutenberg edge branch: \'' + branchName + '\'' );
executeCircleCIBuild( 'false', '', '', e2eBranchName, pullRequestNum, 'ci/wp-e2e-tests-full-desktop-edge', '-s desktop -g', description, sha, false, calypsoProject, null, envVars );
Copy link
Contributor

Choose a reason for hiding this comment

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

@bsessions85 Is there a reason why we don't run tests against live branch for Gutenberg edge user? We had a situation where test was failing on a PR because it was looking for the selector which was introduced in that same PR Automattic/wp-calypso#38507 (comment)

Copy link
Contributor

Choose a reason for hiding this comment

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

No, just an oversight. We were only running on horizon before, so didn't think about using live branches for it. It should be updated.

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