[ https://issues.apache.org/jira/browse/SUBMARINE-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16821285#comment-16821285 ]
Adam Antal commented on SUBMARINE-49: ------------------------------------- Hi [~snemeth], sorry I didn't have the chance to look at it this week, but I am going to allocate some time next week. > Add more test coverage to RunJobParameters > ------------------------------------------ > > Key: SUBMARINE-49 > URL: https://issues.apache.org/jira/browse/SUBMARINE-49 > Project: Hadoop Submarine > Issue Type: Sub-task > Reporter: Szilard Nemeth > Assignee: Szilard Nemeth > Priority: Minor > Attachments: SUBMARINE-49.001.patch, SUBMARINE-49.002.patch, > SUBMARINE-49.003.patch > > > There are some good tests in > {{org.apache.hadoop.yarn.submarine.client.cli.TestRunJobCliParsing}}, but > these are not testing all fields set by method > {{org.apache.hadoop.yarn.submarine.client.cli.param.RunJobParameters#updateParametersByParsedCommandline}}. > > Some more extensive testing is needed in this area. > As an added bonus, the code > {{org.apache.hadoop.yarn.submarine.client.cli.param.RunJobParameters#updateParametersByParsedCommandline}} > could be cleaned up a bit. -- This message was sent by Atlassian JIRA (v7.6.3#76005)