[
https://issues.apache.org/jira/browse/PIG-3087?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14111354#comment-14111354
]
Hao Gao commented on PIG-3087:
------------------------------
Can I try this?
> 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
> Labels: newbie
> Fix For: 0.14.0
>
> Attachments: PIG-3087-0.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.2#6252)
