-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[connectivity]Added integration test. #1455
Conversation
_connectivity = Connectivity(); | ||
}); | ||
|
||
test('test _checkConnectivity', () async { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would remove test and underscore
expect(result, isNotNull); | ||
}); | ||
|
||
test('test connectivity result', () async { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could probably combine this test with the one above.
enableFlutterDriverExtension(handler: (_) => completer.future); | ||
tearDownAll(() => completer.complete(null)); | ||
|
||
group('Connectivity test driver', () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Elsewhere in the plugins repo we use the stringified class name '$Connectivity' to identify group so you may want to be consistent with that, though it's not required. I wouldn't say "test driver" though.
packages/connectivity/CHANGELOG.md
Outdated
@@ -1,3 +1,7 @@ | |||
## 0.4.2+2 | |||
|
|||
* Add test driver. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would say "Added integration test." Although we're using Flutter driver to launch the test, we don't use the FlutterDriver API to do anything, so it may be confusing otherwise.
lgtm modulo the nits above |
looks ready to merge |
Add test driver to the plugin.