-
Notifications
You must be signed in to change notification settings - Fork 143
Conversation
…ses. This can be replaced with the combination of package pattern and class pattern.
…and CLI configuration file.
…gin extension and CLI configuration file.
…ion and CLI configuration file.
…defaults to configuration builder rather than callers, for consistency.
…respective fields.
Closes #48. |
AnnotationItem annotation) { | ||
String testMethod = method.method.getMethodName().getStringValue(); | ||
String testClass = getClassName(classDefItem); | ||
boolean ignored = isClassIgnored(annotationDirectoryItem) || isMethodIgnored(annotation); |
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.
This ignored
boolean might be wrong, as the previous filter in convertClassToTestCaseEvents
removes anything but TEST_ANNOTATION
. That said, no ignore annotation
can be present here.
I think we can relax the filter in the method convertClassToTestCaseEvents
to something like:
.filter(annotation -> (TEST_ANNOTATION.equals(stringType(annotation))
|| (IGNORE_ANNOTATION.equals(stringType(annotation)))))
to fix this.
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.
Good catch. I will add some unit tests to this class cause the dex data types is what confuses me. I think your recommendation would create multiple test cases for the same test though so it needs a bit thinking through.
Amazing! I left just a couple of very minor notes in the PR (a possible small bug and a question) Other than that: LGTM! 💯 |
Branch that contains chimprunner, which runs instrumentation-type tests and keeps timings.