woocommerce-android icon indicating copy to clipboard operation
woocommerce-android copied to clipboard

Resolve suppressed Coroutines related `DEPRECATION` warning for `ReviewModerationHandlerTests`

Open ParaskP7 opened this issue 3 years ago • 6 comments

As part of #6902 and this commit, the ReviewModerationHandlerTests test and all its warnings got suppressed with DEPRECATION.

The warnings occur because this ReviewModerationHandlerTests test suite is the only such suite that still uses the old way of testing with Coroutines. As part of the Coroutine 1.6 PR, migrating this test suite to the new way didn't happen as different approaches using both 'Standard' and 'Unconfined' dispatchers didn't work, 3 tests were always failing. One would need to remove this figure out how to use the new way of testing Coroutines on this ReviewModerationHandlerTests, maybe even rewriting it if need be.

ParaskP7 avatar Jul 08 '22 13:07 ParaskP7

Fails
:no_entry_sign: Please add a feature label to this issue. e.g. 'feature: stats'

Generated by :no_entry_sign: dangerJS

peril-woocommerce[bot] avatar Jul 08 '22 13:07 peril-woocommerce[bot]

Fails
:no_entry_sign: Please add a feature label to this issue. e.g. 'feature: stats'

Generated by :no_entry_sign: dangerJS

peril-woocommerce[bot] avatar Jul 08 '22 13:07 peril-woocommerce[bot]

Fails
:no_entry_sign: Please add a feature label to this issue. e.g. 'feature: stats'

Generated by :no_entry_sign: dangerJS

peril-woocommerce[bot] avatar Jul 08 '22 13:07 peril-woocommerce[bot]