Avoid errors caused by automatically saving invalid shipping times during onboarding

[eason9487]

Changes proposed in this Pull Request:

This PR avoids errors caused by automatically saving invalid shipping times during onboarding.

Kapture.2024-12-03.at.12.34.29.mp4

The issue also causes a flaky E2E test case although it can pass on the second retry:

Screenshots:

Kapture.2024-12-03.at.12.33.07.mp4

Detailed test instructions:

1. Go to step 2 of the onboarding flow
2. Enter a shipping time setup where the min days is greater than its max days
3. Check if the API error, "There was an error saving shipping times.", doesn't occur
4. View the E2E tests run on GitHub Actions to see if the flaky E2E test case no longer occur
   • https://github.com/woocommerce/google-listings-and-ads/actions/runs/12133002898

Changelog entry

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 63.4%. Comparing base (253012d) to head (2b00666).
Report is 7 commits behind head on develop.

Files with missing lines	Patch %	Lines
...ponents/free-listings/setup-free-listings/index.js	0.0%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             develop   #2706      +/-   ##
============================================
- Coverage       66.1%   63.4%    -2.7%     
============================================
  Files            479     340     -139     
  Lines          19543    5210   -14333     
  Branches           0    1275    +1275     
============================================
- Hits           12927    3305    -9622     
+ Misses          6616    1730    -4886     
- Partials           0     175     +175     

Flag	Coverage Δ
js-unit-tests	63.4% <0.0%> (?)
php-unit-tests	?

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

Files with missing lines	Coverage Δ
...ponents/free-listings/setup-free-listings/index.js	6.2% <0.0%> (ø)

... and 818 files with indirect coverage changes

[ianlin]

Thanks for catching and fixing it. Tested locally and confirmed the invalid shipping times weren't saved automatically.