New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(dart/transform): AnnotationMatcher tests #7463
Conversation
These were previously not being run. Bring them up to modern usage, move them to package:test, and include them in transform.server.spec.dart.
the test changes LGTM, but I don't see the change that includes them in transform.server.spec.dart. Did that get left out from the commit? |
import 'common/async_string_writer_tests.dart' as asyncStringWriter; | ||
import 'common/code/ng_deps_code_tests.dart' as ngDepsCode; | ||
import 'common/ng_meta_test.dart' as ngMetaTest; | ||
import 'common/url_resolver_tests.dart' as urlResolver; | ||
|
||
main() { | ||
group('AnnotationMatcher', annotationMatcher.allTests); |
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.
@juliemr , this line adds the changes to transform.server.spec.dart
LGTM too. |
TAP - 0 affected targets |
Merging PR #7463 on behalf of @vikerman to branch presubmit-vikerman-pr-7463. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
/cc @juliemr
These tests were previously not being run.
Bring them up to modern usage, move them to package:test, and include
them in transform.server.spec.dart.