Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix broken integration test for streaming
The `whenComplete` call was sometimes causing a race condition. It is also isn't needed for the test (to reset the value of isDetecting, given it's local), so removing it makes the test reliable.
  • Loading branch information
Adam Harwood committed Dec 8, 2022
commit 4b844850fe32d785ed182ba4c08e0bc395ca8db1
Original file line number Diff line number Diff line change
Expand Up @@ -272,7 +272,7 @@ void main() {

isDetecting = true;

expectLater(image, isNotNull).whenComplete(() => isDetecting = false);
expectLater(image, isNotNull);
});

expect(controller.value.isStreamingImages, true);
Expand Down