pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Daniel Dai (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PIG-3087) Refactor TestLogicalPlanBuilder to be meaningful
Date Wed, 05 Oct 2016 22:39:20 GMT

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

Daniel Dai commented on PIG-3087:
---------------------------------

The bug fix in ForEachUserSchemaVisitor looks good. I have comments on the test fix:
1. If the query expect to fail, we need to check for that. Now couple of tests pass either
query fail with expected error message, or query pass
2. If we should see error in grunt parser, then we shall test here the same way. Please turn
on validation if that's the reason.

> Refactor TestLogicalPlanBuilder to be meaningful
> ------------------------------------------------
>
>                 Key: PIG-3087
>                 URL: https://issues.apache.org/jira/browse/PIG-3087
>             Project: Pig
>          Issue Type: Bug
>            Reporter: Jonathan Coveney
>            Assignee: Adam Szita
>              Labels: newbie
>             Fix For: 0.17.0
>
>         Attachments: PIG-3087-0.patch, PIG-3087.1.patch
>
>
> I started doing this as part of another patch, but there are some bigger issues, and
I don't have the time to dig in atm.
> That said, a lot of the tests as written don't test anything. I used more modern junit
patterns, and discovered we had a lot of tests that weren't functioning properly. Making them
function properly unveiled that the general buildLp pattern doesn't work properly anymore
for many cases where it would throw an error in grunt, but for whatever reason no error is
thrown in the tests.
> Any test with _1 is a test that previous failed, that now doesn't. Some, however, don't
make sense so I think what really needs to be done is figure out which should be failing,
which shouldn't, and then fix buildLp accordingly.
> I will attach my pass at it, but it is incomplete and needs work.



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

Mime
View raw message