ShermanMarshall commented on a change in pull request #448: Tomee 2482 
synchronize examples
URL: https://github.com/apache/tomee/pull/448#discussion_r268380651
 
 

 ##########
 File path: 
examples/bval-evaluation-redeployment/WebApp2/src/main/java/org/superbiz/webapp2/service/WebApp2Service.java
 ##########
 @@ -32,7 +31,7 @@
 public class WebApp2Service {
 
     @POST
-    public Response getInfo(@Pattern(regexp = "valid") final String input) {
 
 Review comment:
   After further review, I now see the bval-jsr dependency in openejb-core.
   
   I think that was the cause of my confusion. It was implicitly available 
through openejb-core, but I didn't see it in the bval-evaluation-redeployment 
project.
   
   It seems there is already some code coverage in openejb-core for testing 
bval. I'm uncertain whether or not we need an example for further testing here, 
but I will do as advised.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to