[
https://issues.apache.org/jira/browse/FALCON-942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14247496#comment-14247496
]
Raghav Kumar Gautam commented on FALCON-942:
--------------------------------------------
I talked to [~arpitgupta] about this change and he is -1 about this change. The
main reason being that it is not ok for a test to delete stuff that does not
concern it.
> Cleanup BaseTestClass and setUp/tearDown methods
> ------------------------------------------------
>
> Key: FALCON-942
> URL: https://issues.apache.org/jira/browse/FALCON-942
> Project: Falcon
> Issue Type: Improvement
> Components: merlin
> Affects Versions: 0.6
> Reporter: Ruslan Ostafiychuk
> Assignee: Ruslan Ostafiychuk
> Priority: Minor
> Fix For: 0.7
>
> Attachments: FALCON-942.patch
>
>
> - _BaseTestClass.removeBundle()_ replaced with _CleanupUtil_ methods
> - _TestngListener_ prints information about start/stop so we don't need this
> in before/after methods
> - it's easier to clean base test dir in _BaseTestClass_ constructor instead
> of adding @AfterClass every time
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
