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

Adjust e2e tests for minimum and maximum shipping times #2619

Open
wants to merge 1 commit into
base: update/add-and-edit-shipping-time-inputs
Choose a base branch
from

Conversation

jorgemd24
Copy link
Contributor

Changes proposed in this Pull Request:

Part of pcTzPl-2qP-p2

This PR updates the e2e tests to align with the changes from this PR: #2594

Screenshots:

Detailed test instructions:

  1. Follow the instructions provided here, run the e2e tests, and verify that all tests are passing.

Additional details:

Changelog entry

@github-actions github-actions bot added the changelog: tweak Small change, that isn't actually very important. label Sep 18, 2024
@jorgemd24 jorgemd24 self-assigned this Sep 18, 2024
@jorgemd24 jorgemd24 changed the base branch from update/shippings-settings-phase-1 to update/add-and-edit-shipping-time-inputs September 18, 2024 16:46
@jorgemd24 jorgemd24 requested a review from a team September 18, 2024 16:47
Copy link

codecov bot commented Sep 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.8%. Comparing base (8652d24) to head (5a9db1a).

Additional details and impacted files

Impacted file tree graph

@@                           Coverage Diff                            @@
##           update/add-and-edit-shipping-time-inputs   #2619   +/-   ##
========================================================================
  Coverage                                      63.8%   63.8%           
========================================================================
  Files                                           323     323           
  Lines                                          5133    5133           
  Branches                                       1257    1257           
========================================================================
  Hits                                           3276    3276           
  Misses                                         1684    1684           
  Partials                                        173     173           
Flag Coverage Δ
js-unit-tests 63.8% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@jorgemd24 jorgemd24 marked this pull request as ready for review September 18, 2024 17:05
Copy link
Contributor

@martynmjones martynmjones left a comment

Choose a reason for hiding this comment

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

Hey @jorgemd24, thanks for updating the e2e tests.

The changes all look good to me and all tests are passing ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: tweak Small change, that isn't actually very important.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants