Performance improvement of unit tests: Remove delays and inject dependencies

[staskus]

Closes: #11228
Context: pdfdoF-64g-p2#comment-7238

Description

This is a continuation of #14802 to investigate the reasons of slow unit tests and look for solutions.

I haven't found more all-encompassing widely applied solutions. Therefore this PR has a moderate average speed execution improvement from around 50ms to 40ms..

Jetpack Setup ViewModel Enhancements:

• Added a new property delayBeforeRetry to the JetpackSetupViewModel and updated its initialization to include this property. This change allows for configurable retry delays during Jetpack setup. [1] [2] [3] [4]
• Updated JetpackSetupViewModelTests to include the new delayBeforeRetry parameter in the test cases. [1] [2] [3] [4] [5] [6]

Push Notification Background Synchronizer Refactoring:

• Introduced PushNotificationBackgroundSynchronizerProtocol and updated PushNotificationBackgroundSynchronizer to conform to this protocol.
• Created PushNotificationBackgroundSynchronizerFactory and its corresponding protocol to handle the creation of synchronizer instances.
• Updated PushNotificationsManager to use the new factory for creating background synchronizers. [1] [2]

Hub Menu ViewModel Enhancements:

• Refactored HubMenuViewModel to make viewDidAppear, refreshGoogleAdsCampaignCheck, and refreshBlazeEligibilityCheck asynchronous functions. Added a helper extension to handle these asynchronous calls. [1] [2] [3]

Remote Order Synchronizer Enhancements:

• Added a debounceDuration property to RemoteOrderSynchronizer to allow configurable debounce intervals for sync requests. [1] [2]

Steps to reproduce

Testing information

⚠️ There has been flakiness with ConfigurableBundleProductViewModelTests test_bundleItemViewModels_have_correct_quantity_when_parent_order_item_quantity_is_more_than_one in CI runs that I cannot quite figure out. It runs super-fast locally but slows down significantly on the CI. I will make sure this branch is stable and has a few consecutive successful runs on CI before merging.

Screenshots

---

• I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

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:

• The PR is small and has a clear, single focus, or a valid explanation is provided in the description. If needed, please request to split it into smaller PRs.
• Ensure Adequate Unit Test Coverage: The changes are reasonably covered by unit tests or an explanation is provided in the PR description.
• Manual Testing: The author listed all the tests they ran, including smoke tests when needed (e.g., for refactorings). The reviewer confirmed that the PR works as expected on all devices (phone/tablet) and no regressions are added.

[wpmobilebot]

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

	App Name	WooCommerce iOS
	Build Number	pr14822-9d9221c
	Version	21.4
	Bundle ID	com.automattic.alpha.woocommerce
	Commit	9d9221c
	App Center Build	WooCommerce - Prototype Builds #12493

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