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

Konstantin Boudnik edited comment on BIGTOP-1316 at 6/28/14 10:27 PM:
----------------------------------------------------------------------

Is it an intenation visibility layout:
- void expectExitCode(int expectedExitCode, Object... args)
- void expectSuccess(Object... args)
yet
- public void assertExitCode(int expectedExitCode)

Do you real need an empty constructor 
{code:java}
+  JUnitShell() {
+  }
{code}


was (Author: cos):
Is it an intenation visibility layout:
- void expectExitCode(int expectedExitCode, Object... args) {
- void expectSuccess(Object... args) {
yet
- public void assertExitCode(int expectedExitCode) {

> enhance Shell for better checking of return code & output logging
> -----------------------------------------------------------------
>
>                 Key: BIGTOP-1316
>                 URL: https://issues.apache.org/jira/browse/BIGTOP-1316
>             Project: Bigtop
>          Issue Type: Improvement
>          Components: Tests
>    Affects Versions: 0.8.0
>            Reporter: Steve Loughran
>            Assignee: Steve Loughran
>            Priority: Minor
>         Attachments: BIGTOP-1316-001.patch, BIGTOP-1316-002.patch
>
>
> for the slider tests, we extended Shell with 
> [SliderShellhttps://svn.apache.org/viewvc/incubator/slider/trunk/slider-funtest/src/main/groovy/org/apache/slider/funtest/framework/SliderShell.groovy?view=markup],
>  adding some more features. Some of those are biased towards executing our 
> entry point, but there are some useful core features worth pulling up to the 
> base class
> # {{toString()}} and {{dump()}} methods with diagnostics
> # return code sign correction and assertion checking
> # construction of bash environment and command lines
> The bash setup could be done with a class {{Bash extends Shell}} which is 
> designed purely for bash execution



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to