| Method Todos |
|---|
| verify(): The verify phase should be set on any type of verify call. Need to think of a reliable way to set it rather than in each call. |
| Method Todos |
|---|
| createResourceManager(): Read Locale settings from config file |
| Method Todos |
|---|
| isClassExcluded(): hardcoded to exclude VMTest class as an example |
| Class Todos |
|---|
| Write unit test for this class. |
| Method Todos |
|---|
| isJoinPointExcluded(): expose some API on VM to allow user to dynamically define multiple class exclusion patterns |
| Method Todos |
|---|
| isJoinPointExcluded(): Expose some API on VM to allow user to dynamically define multiple class exclusion patterns. |
| getInvokedCallFromMethodJoinPoint(): write unit test for this |
| getSignatureFromMethodJoinPoint(): Write unit test for this |
| Method Todos |
|---|
| execute(): Add functionality to report on collaborators that are not mocked. This will happen just prior to the JoinPoint.proceed() on the last line. |
| Method Todos |
|---|
| execute(): The code in this class related to primitives is a workaround for an AspectWerkz problem with handling null return values for primitive return types |
| Method Todos |
|---|
| validateReturnValue(): Does this method properly validate non-primitives of an invalid (non-assignable) type? |
| validateReturnValue(): Can the common logic in this method be combined with the logic in validateArguments and abstracted out? |
| Method Todos |
|---|
| hasMatchingArgs(): need to implement support for configurable default ArgMatcher, instead of hardcoding default to EqualsMatcher |
| Method Todos |
|---|
| getMatchingCall(): Implement getMatchingCallOrdered(). |
| getMatchingCallUnordered(): Implement ability to set unlimited invocations via VM. |
| getMatchingCallUnordered(): Need functional test for unlimited invocations. |
| Method Todos |
|---|
| toString(): investigate all possible values for return type string other than not null, null, or void |
| Class Todos |
|---|
| This should be refactored into a generic reusable parser, now that all dependencies are passed in via constructor. |
| Class Todos |
|---|
| revisit design of how rules are stored - current method of storing types is really just a proof of concept to parse the info out of the XML config file. Also update javadoc. |
| Method Todos |
|---|
| getViolationMessage(): make this message cleaner and more descriptive |
| Class Todos |
|---|
| Figure out how rules will be handled - ignore, warn, fail... |
| Method Todos |
|---|
| checkAllRulesInCurrentPhase(): for now, only shows first message for first failed rule. Should optionally show all messages |
| checkAllRulesInCurrentPhase(): for now, processes all rules regardless of phase |
| Method Todos |
|---|
| createClasspath(): Do we need to add anything to the classpath? If so, what? |
| createClasspath(): What about forked mode? |
| Class Todos |
|---|
| What's the easiest way to write a unit test for an Ant task? |
| Method Todos |
|---|
| createClasspath(): Do we need to add anything to the classpath? If so, what? |
| createClasspath(): What about forked mode? |
| execute(): This is currently hardcoded for AspectJ |
| execute(): This is currently hardcoded for absolute path on Chad's machine. Make these task parameters |
| execute(): How can the transformed jar be put at the beginning of the classpath? |