woocommerce / woocommerce-ios

WooCommerce iOS app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
314 stars 113 forks source link

Exclude flaky unit tests related to dashboard cards's custom range date. #14426

Closed hafizrahman closed 1 week ago

hafizrahman commented 1 week ago

See report on https://github.com/woocommerce/woocommerce-ios/pull/14287/files#r1843059760

Description

For more context, the flakiness was attempted to be fixed in a previous PR https://github.com/woocommerce/woocommerce-ios/pull/14287

It still failed on certain time, it turns out, so for now we're excluding them.

still failing at certain time (failed 3 times 7-8AM GMT, but fixed a few hours later).

Steps to reproduce

CI passing should be sufficient.


Reviewer (or Author, in the case of optional code reviews):

Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement:

wpmobilebot commented 1 week ago

WooCommerce iOS📲 You can test the changes from this Pull Request in WooCommerce iOS by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS WooCommerce iOS
Build Numberpr14426-9696085
Version21.1
Bundle IDcom.automattic.alpha.woocommerce
Commit96960850cf043ea730963075637a367a6e216c78
App Center BuildWooCommerce - Prototype Builds #11603

Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

hafizrahman commented 1 week ago

@selanthiraiyan weird, this PR was already mentioned in the issue's message as https://github.com/woocommerce/woocommerce-ios/pull/14426/files but it didn't crosspost. Apparently GH doesn't like the /files part, TIL.