Minor tweaks (#254)
Minor tweaks
diff --git a/.gitignore b/.gitignore
index d87ef84..9f6d195 100644
--- a/.gitignore
+++ b/.gitignore
@@ -5,3 +5,5 @@
.packages
.pub
pubspec.lock
+
+coverage/
diff --git a/README.md b/README.md
index 61a09dd..fb8587b 100644
--- a/README.md
+++ b/README.md
@@ -1,4 +1,4 @@
-Mock library for Dart inspired by [Mockito](https://github.com/mockito/mockito).
+# Mock library for Dart inspired by [Mockito](https://github.com/mockito/mockito)
[](https://pub.dartlang.org/packages/mockito)
[](https://travis-ci.org/dart-lang/mockito)
@@ -72,10 +72,10 @@
By default, for all methods that return a value, `mock` returns `null`.
Stubbing can be overridden: for example common stubbing can go to fixture setup
-but the test methods can override it. Please note that overridding stubbing is
-a potential code smell that points out too much stubbing. Once stubbed, the
+but the test methods can override it. Please note that overridding stubbing is
+a potential code smell that points out too much stubbing. Once stubbed, the
method will always return stubbed value regardless of how many times it is
-called. Last stubbing is more important, when you stubbed the same method with
+called. Last stubbing is more important, when you stubbed the same method with
the same arguments many times. In other words: the order of stubbing matters,
but it is meaningful rarely, e.g. when stubbing exactly the same method calls
or sometimes when argument matchers are used, etc.
@@ -94,7 +94,7 @@
Instead, use `thenAnswer` to stub methods that return a `Future` or `Stream`.
-```
+```dart
// BAD
when(mock.methodThatReturnsAFuture())
.thenReturn(Future.value('Stub'));
@@ -112,14 +112,13 @@
If, for some reason, you desire the behavior of `thenReturn`, you can return a
pre-defined instance.
-```
+```dart
// Use the above method unless you're sure you want to create the Future ahead
// of time.
final future = Future.value('Stub');
when(mock.methodThatReturnsAFuture()).thenAnswer((_) => future);
```
-
## Argument matchers
Mockito provides the concept of the "argument matcher" (using the class
diff --git a/analysis_options.yaml b/analysis_options.yaml
index 2cdc67b..5a16769 100644
--- a/analysis_options.yaml
+++ b/analysis_options.yaml
@@ -5,15 +5,15 @@
linter:
rules:
- # Errors
- - comment_references
- - control_flow_in_finally
- - empty_statements
- - hash_and_equals
- - test_types_in_equals
- - throw_in_finally
+ # Errors
+ - comment_references
+ - control_flow_in_finally
+ - empty_statements
+ - hash_and_equals
+ - test_types_in_equals
+ - throw_in_finally
- # Style
- - await_only_futures
- - camel_case_types
- - non_constant_identifier_names
+ # Style
+ - await_only_futures
+ - camel_case_types
+ - non_constant_identifier_names
