Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

Use ArchTaskExecutor instead of InstantTaskExecutorRule #378

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Use ArchTaskExecutor instead of InstantTaskExecutorRule #378

wants to merge 2 commits into from

Conversation

skydoves
Copy link
Contributor

Use ArchTaskExecutor instead of InstantTaskExecutorRule

@get:Rule
var instantTaskExecutorRule = InstantTaskExecutorRule()

The issue is that this one does not provide a real thread for main thread and accepts all threads as main; hence might hide some bugs by having a different behavior than the real implementation. (and it is likely to be deprecated - removed). by yigit.

Modified:

  • removed InstantTaskExecutorRule
  • added TaskExecutorUtils : registerTaskExecutor(), unRegisterTaskExecutor()

@XinyueZ
Copy link
Contributor

XinyueZ commented Apr 2, 2019

@skydoves I think Google has updated the test framework internal already, could you check?

@XinyueZ
Copy link
Contributor

XinyueZ commented Apr 25, 2019

@tiembo I think the newest google's repo has updated corresponded APIs.

@codingjeremy codingjeremy changed the base branch from master to main September 29, 2020 19:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants