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

jay vyas edited comment on BIGTOP-1316 at 6/19/14 3:01 AM:
-----------------------------------------------------------

MAybe a simpler way forward if we all agree is  , [~cos] and 
[[email protected]]  : how about a class called {{ 
bigtop-test-framework/src/main/groovy/org/apache/bigtop/itest/shell/JunitShell.groovy
  }} that extends {{ Shell.groovy }} , with the functionality steve wants to 
use in slider?  


was (Author: jayunit100):
[[email protected]] would you feel about just writing a whole new class that 
extends Shell.groovy with the functionality you want to use in slider?  I think 
others will find it useful.

Sounds like the vision for {{Shell.groovy}} is a bare-bones wrapper to the bash 
shell -- and not a testing utility.

> 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
>
>
> 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