[RSDK-4265] Sensor widget tests #81
Conversation
| expect(playButton, findsNothing); | ||
| expect(pauseButton, findsNothing); | ||
| expect(refreshButton, findsNothing); | ||
| }); |
There was a problem hiding this comment.
(q) how come you aren't calling tester.pump() in this test like you are in the other ones?
There was a problem hiding this comment.
tester.pump only needs to be called if the state changes in order to reevaluate the UI with the new state see docs
| expect(playButton, findsNothing); | ||
| expect(pauseButton, findsNothing); | ||
| }); | ||
| }); |
There was a problem hiding this comment.
(q) maybe obvious but does it also hide the refresh button? should we still test for that?
There was a problem hiding this comment.
Nope! When refreshInterval is null, only the play/pause button is hidden. The manual refresh button needs to be available so that users can manually refresh
There was a problem hiding this comment.
But you are correct that we should have a test to make sure the refresh button is visible so I added it :)
Documentation and tests for sensor widget