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

Update sync e2e tests to new setup flow #2151

Merged
merged 9 commits into from
Nov 14, 2023

Conversation

loremattei
Copy link
Contributor

@loremattei loremattei commented Nov 12, 2023

Task/Issue URL: https://app.asana.com/0/414709148257752/1205935203864974
Tech Design URL:
CC:

Description:

Updates the Sync e2e to use the new setup flow.
Also, adds a step to the workflow to create an Asana task on failures.

Note: The workflow is temporarily triggered on push to make it easier to test. I'll revert to cron once the PR is approved, before merging it back into develop.

Steps to test this PR:

  1. Verify CI is green

Internal references:

Software Engineering Expectations
Technical Design Template

@loremattei loremattei changed the title Loremattei/update sync tests to new setup flow Update sync e2e tests to new setup flow Nov 12, 2023
@loremattei loremattei marked this pull request as ready for review November 12, 2023 22:08
Copy link
Contributor

@ayoy ayoy left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @loremattei!

@loremattei loremattei merged commit 9c4124e into develop Nov 14, 2023
9 checks passed
@loremattei loremattei deleted the loremattei/update-sync-tests-to-new-setup-flow branch November 14, 2023 14:27
samsymons added a commit that referenced this pull request Nov 14, 2023
* develop:
  remove return user pixel (#2146)
  Revert to auto signing (#2158)
  BSK changes for NetP iOS Geoswitching (#2141)
  Update sync e2e tests to new setup flow (#2151)
  Fix for duplicated "atb" in Pixel request (#2139)
  Updating tests to support latest reference tests (#2145)
samsymons added a commit that referenced this pull request Nov 16, 2023
* develop: (36 commits)
  Rolls back a BSK change by mistake
  Updates BSK
  Updates BSK
  Document LinkPresentation usage (#2172)
  Autofill "Never Save for this Site" feature (#2104)
  Use upstream fastlane 2.217.0 (#2161)
  Update about screen (#2152)
  Fixes a crasher due to a test line merged by mistake. (#2162)
  remove return user pixel (#2146)
  Revert to auto signing (#2158)
  BSK changes for NetP iOS Geoswitching (#2141)
  Update sync e2e tests to new setup flow (#2151)
  Fix for duplicated "atb" in Pixel request (#2139)
  Updating tests to support latest reference tests (#2145)
  Add NetP widget (#2142)
  Normalize ampUrl in breakage pixel (#2154)
  Update embedded files
  Update version number
  Bump submodules/privacy-reference-tests from `0d23f76` to `7519c3d` (#2135)
  Properly refresh home page favorites view when updating favorites display mode (#2148)
  ...
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.

2 participants