-
Notifications
You must be signed in to change notification settings - Fork 29
Closed
Description
Update co19 tests according to the Effective Dart guide
- Use strings in Expect library parts directives. Guide link
- Rename
Expectlibrary toexpect. Guide link - Use lowerCamelCase for constant names in libraries and common files (at least at
expectlibrary). Guide link - Capitalize acronyms and abbreviations longer than two letters like words. For example
isAOT->isAot. Guide link - Rename abbrevations like
ONE_MStooneMillisecond. Guide link - Remove explicit default value
null(at least inexpectlibrary, in particular inexpect_async.dart). Guide link dynamic_check.dartuses deprecatedCastError. Do we really need it? If removing of this class doesn't introduces new test failures then remove it- Feel free to do more refactoring changes according to this guide. Don't try to change all 20K tests, but, at least, common libraries like
expectshould follow this guide
Metadata
Metadata
Assignees
Labels
No labels