Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

RunIPhoneUnitTest.sh should have non-zero exit on test failure #12

Closed
GoogleCodeExporter opened this issue Apr 21, 2015 · 4 comments
Closed

RunIPhoneUnitTest.sh should have non-zero exit on test failure #12

GoogleCodeExporter opened this issue Apr 21, 2015 · 4 comments
Labels
auto-migrated Priority-Medium Type-Defect

Comments

@GoogleCodeExporter
Copy link 

What steps will reproduce the problem?
1. Run RunIPhoneUnitTest.sh with a failed test.
2. Check return value $?

What is the expected output? What do you see instead?
Should be non-zero

What version of the product are you using? On what operating system?
trunk/r76; Mac OS X 10.5.5

Please provide any additional information below.
Attached a diff, which is how we patched this locally.

Original issue reported on code.google.com by gabri...@gmail.com on 23 Jan 2009 at 12:31

Attachments:
@GoogleCodeExporter
Copy link
Author 

I just submitted a patch to do something similar! You beat me to the post by 84 
minutes! I'll try your patch and 
see if it works better or worse.

Original comment by clifton....@gmail.com on 23 Jan 2009 at 2:01

@GoogleCodeExporter
Copy link
Author 

Looks like we might want a merge of this and the patch in 
http://code.google.com/p/google-toolbox-for-
mac/issues/detail?id=12 since someone can turn off the exit call from the 
delegate so we'll keep both issues 
opened until we land something for it.

Original comment by thoma...@gmail.com on 24 Jan 2009 at 1:27

@GoogleCodeExporter
Copy link
Author 

oops, i meant 
http://code.google.com/p/google-toolbox-for-mac/issues/detail?id=13

Original comment by thoma...@gmail.com on 24 Jan 2009 at 1:28

@GoogleCodeExporter
Copy link
Author 

just landed a code change so the exit status is set, and for the apps that 
directly drive the delegate, then can get 
out the success/fail numbers.

Original comment by thoma...@gmail.com on 30 Jan 2009 at 3:20

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
auto-migrated Priority-Medium Type-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GoogleCodeExporter