Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(13)

Unified Diff: pkg/unittest/lib/src/interfaces.dart

Issue 18442002: Add missing newline after Actual value in fail message. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « pkg/unittest/lib/src/future_matchers.dart ('k') | pkg/unittest/lib/src/iterable_matchers.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/unittest/lib/src/interfaces.dart
===================================================================
--- pkg/unittest/lib/src/interfaces.dart (revision 24669)
+++ pkg/unittest/lib/src/interfaces.dart (working copy)
@@ -46,14 +46,15 @@
}
/**
- * [expect] Matchers must implement the Matcher class.
- * The base Matcher class that implements this interface has
- * a generic implementation of [describeMismatch] so this does
- * not need to be provided unless a more clear description is
+ * [expect] Matchers must implement/extend the Matcher class.
+ * The base Matcher class has a generic implementation of [describeMismatch]
+ * so this does not need to be provided unless a more clear description is
* required. The other two methods ([matches] and [describe])
* must always be provided as they are highly matcher-specific.
*/
abstract class Matcher {
+ const Matcher();
+
/**
* This does the matching of the actual vs expected values.
* [item] is the actual value. [matchState] can be supplied
@@ -76,7 +77,7 @@
* diagnosing failures, such as stack traces.
*/
Description describeMismatch(item, Description mismatchDescription,
- Map matchState, bool verbose);
+ Map matchState, bool verbose) => mismatchDescription;
}
/**
« no previous file with comments | « pkg/unittest/lib/src/future_matchers.dart ('k') | pkg/unittest/lib/src/iterable_matchers.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698