[ 
https://issues.apache.org/jira/browse/DRILL-5053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15682304#comment-15682304
 ] 

Paul Rogers commented on DRILL-5053:
------------------------------------

{{org.apache.drill.common.expression.parser.TreeTest.testExpressionParsing}} 
"tests" expressions by printing them to the console. The test contains a 
commented-out use of Unit {{assertEquals}}, commented out because the 
{{LogicalExpression}} class does not actually implement {{isEqual()}}.

Fix the test to not do the logging, and instead compare the serialized forms of 
the before and after nodes.

> Remove excessive output from unit test cases
> --------------------------------------------
>
>                 Key: DRILL-5053
>                 URL: https://issues.apache.org/jira/browse/DRILL-5053
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.8.0
>            Reporter: Paul Rogers
>            Assignee: Paul Rogers
>            Priority: Minor
>
> Drill's maven build runs a wide variety of unit tests. Unfortunately, many 
> tests produce very verbose output, sometimes in lieu of checking results, 
> making it hard to determine which test actually fails.
> The ultimate goal is to run the following and have very little output:
> {code}
> mvn clean install -q
> {code}
> Specific issues will appear below as comments.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to