[Payment Method Improvements] Add more tests

[backwardstruck]

Closes: #11542

This PR is branched off of #11606

Description

Add more unit tests to increase test coverage.

Testing instructions

Unit tests should pass

• I have considered if this change warrants release notes and have added them to RELEASE-NOTES.txt if necessary. Use the "[Internal]" label for non-user-facing changes.

---

[wpmobilebot]

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

	App Name	WooCommerce Android
	Flavor	Jalapeno
	Build Type	Debug
	Commit	84194cb
	Direct Download	woocommerce-prototype-build-pr11607-84194cb.apk

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.32%. Comparing base (4b1037b) to head (84194cb).
Report is 20 commits behind head on trunk.

Additional details and impacted files

@@            Coverage Diff            @@
##              trunk   #11607   +/-   ##
=========================================
  Coverage     39.32%   39.32%           
- Complexity     5225     5232    +7     
=========================================
  Files          1096     1097    +1     
  Lines         63984    63992    +8     
  Branches       8797     8797           
=========================================
+ Hits          25160    25165    +5     
- Misses        36501    36506    +5     
+ Partials       2323     2321    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[backwardstruck]

Working on resolving the conflicts with the target branch...

[kidinov]

@backwardstruck 👋

Could you please test that all once again as the changes I was doing a bit in a rush as I am quite overwhelmed with the pos?

Then you can just change the feature flag. But please do not forget to remove feature flag, OtherPaymentMethodsEnabled class and all the "old" cash collection code - dialog/tests and so on in the following PR (can be merged to the next release)

[backwardstruck]

@backwardstruck 👋

Could you please test that all once again as the changes I was doing a bit in a rush as I am quite overwhelmed with the pos?

Then you can just change the feature flag. But please do not forget to remove feature flag, OtherPaymentMethodsEnabled class and all the "old" cash collection code - dialog/tests and so on in the following PR (can be merged to the next release)

Sure! I created this ticket for that code removal:
#11625

Testing now. And changing the FF value to true

[dangermattic]

	1 Warning
⚠️	This PR is assigned to the milestone 18.9. This milestone is due in less than 2 days. Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by 🚫 Danger