osundblad / intellij-annotations-instrumenter-maven-plugin

IntelliJ IDEA annotations instrumenter maven plugin
Other
38 stars 11 forks source link

Remove synthetic methods instrumentation #17

Closed atorstling closed 7 years ago

atorstling commented 7 years ago

I removed instrumentation for synthetic methods in the general case. This basically entailed moving the synthetic checks to ThrowOnNullMethodVisitor.

Added test data. Refactored the test to use the same classloader, otherwise the clazz.getMethod(...) didn't really work as expected. Added a way of checking if a method has been instrumented by checking the its constants for validation messages.