[ https://issues.apache.org/jira/browse/HDFS-8741?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15893649#comment-15893649 ]
Yiqun Lin edited comment on HDFS-8741 at 3/3/17 3:13 AM: --------------------------------------------------------- Thanks [~surendrasingh] for the patch. Some comments for your test. * It would be better to add message into {{fails()}}, like following: {code} + @Test + public void testHttpOpParams() { + try { + new PostOpParam("TEST"); + Assert.fail("Construct the PostOpParam with param vaule 'TEST' should be failed."); {code} * Suggest using {{GenericTestUtils#assertExceptionContains}} verify the message contained in exception. was (Author: linyiqun): Thanks [~surendrasingh] for the patch. Some comments for your test. * It would be better to add message into {{fails()}}, like following: {code} + @Test + public void testHttpOpParams() { + try { + new PostOpParam("TEST"); + Assert.fail("Construct the PostOpParam with param vaule TEST should be failed."); {code} * Suggest using {{GenericTestUtils#assertExceptionContains}} verify the message contained in exception. > Proper error msg to be printed when invalid operation type is given to > WebHDFS operations. > ------------------------------------------------------------------------------------------ > > Key: HDFS-8741 > URL: https://issues.apache.org/jira/browse/HDFS-8741 > Project: Hadoop HDFS > Issue Type: Improvement > Components: webhdfs > Reporter: Archana T > Assignee: Surendra Singh Lilhore > Priority: Minor > Attachments: HDFS-8741.001.patch, HDFS-8741.002.patch, > HDFS-8741.003.patch > > > When wrong operation type is given to WebHDFS operations, following Error > message is printed -- > For ex: CREATE is called with GET instead of PUT-- > HTTP/1.1 400 Bad Request > ...... > {"RemoteException":{"exception":"IllegalArgumentException","javaClassName":"java.lang.IllegalArgumentException","message":"Invalid > value for webhdfs parameter \"op\": {color:red}No enum constant > org.apache.hadoop.hdfs.web.resources.PutOpParam.Op.CREATE"}}{color} > Expected-- > Valid Error message to be printed -- This message was sent by Atlassian JIRA (v6.3.15#6346) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org