- France
- https://amine-bouabdallaoui.fr
- Joined on
2022-07-18
The way I understand your comment is that sometimes, the repository can fail an throw an error.
This should not be handled in a unit test, but handled in the code.
The exception should be…
Be cause of the line 40 of the setup function you have an unread/starred item in your DB.
The network is available, so this will be called. And the next one won't.
In reloadBadges(), there is a missing success = true in the else statement.
Same here, the two values are different in the initialization, they couldn't be the same after.
I don't understand what you wanted to test here. The network is available, so the two will be the same.
This does not seem to do anything. Is it needed ?
Tags and sources caching is handled via the update_sources setting.
Returning null isn't possible, so it should be an empty list.
this should be assertEquals(emptyList(), testTags)
The same comments about Get tags without connection and items caching disabled should be considered here.
Be cause of the line 40 of the setup function you have an unread/starred item in your DB.