glitch commented on a change in pull request #289: ACCUMULO-4677 Sanitizing
PathParam values in REST-based Monitor
URL: https://github.com/apache/accumulo/pull/289#discussion_r153373502
##########
File path:
server/monitor/src/test/java/org/apache/accumulo/monitor/view/WebViewsTest.java
##########
@@ -130,6 +130,7 @@ public void testGetTracesSummaryValidationConstraint() {
/**
* Silly stub to handle MessageBodyWriter for Hashmap. Registered in
configure method and auto-wired by Jersey.
*/
+ @SuppressWarnings("rawtypes")
Review comment:
RE: IntegrationTests which @Category class should this annotation point at?
SunnyDay?
EDIT: Also, presumably I will need to move it to the **accumulo-test**
module. I see things under src/main/java/... with test annotations in them and
I see things under /src/test/java/... so which one does it go under, main or
test?
Thanks
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
