search

theboocock / shunit2

Automatically exported from code.google.com/p/shunit2
0 stars 0 forks source link

Do not pipe "oneTimeTearDown" #36

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
oneTimeSetUp is not piped to the output file although onTimeTearDown is.

It is not consistent. 

Given that piping onTimeTearDown creates a subshell and prevent users of 
modifying variables of shunit2 (e.g. suite name) I would propose not to pipe it 
either.

Original issue reported on code.google.com by barthele...@gmail.com on 30 Apr 2013 at 9:20

GoogleCodeExporter commented 9 years ago 
Could you provide some sample code that demonstrates the behavior you see? 
Neither functions are piped in shUnit2, and both should behave exactly the same.

Original comment by kate.w...@forestent.com on 20 May 2013 at 6:16

GoogleCodeExporter commented 9 years ago 
Hello, 

I apologize for the confusion. I had used the patch proposed in issue 16. The 
pipe I was referring to is not present in the official version of shunit2, only 
in the patch. Thus we agree.

You can close this issue. 

Thank you

Original comment by barthele...@gmail.com on 21 May 2013 at 12:12