2008555 - Refactor ShareResourceFeatureTest to remove MainCoroutineRule and runTestOnMain

Status: RESOLVED FIXED

(Firefox for Android :: Tooling, task)

Description

[Mihai Adrian Carare [:mcarare]]

.

Comment 1

[Mihai Adrian Carare [:mcarare]]

Attachment: Bug 2008555 - Update `ShareResourceFeature` to allow custom dispatchers for store observation and improve unit test reliability

• Modify ShareResourceFeature constructor to accept a mainDispatcher (defaults to Dispatchers.Main) for store observation.
• Update ShareResourceFeatureTest.kt to use StandardTestDispatcher and runTest instead of MainCoroutineRule and runTestOnMain.
• Improve test synchronization by adding testDispatcher.scheduler.advanceUntilIdle() calls where asynchronous actions occur.
• Fix a missing mock in startSharing() test by stubbing shareInternetResource.
• Explicitly pass dispatchers during ShareResourceFeature instantiation in tests to ensure deterministic execution.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/68a801f12c88
https://hg.mozilla.org/integration/autoland/rev/2cc5a37ab8e2
Update `ShareResourceFeature` to allow custom dispatchers for store observation and improve unit test reliability r=android-reviewers,avirvara

Comment 3

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/2cc5a37ab8e2