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

A failing exit phaser after passing all tests appears OK to prove #4384

Closed
p6rt opened this issue Jul 8, 2015 · 7 comments
Closed

A failing exit phaser after passing all tests appears OK to prove #4384

p6rt opened this issue Jul 8, 2015 · 7 comments

Comments

@p6rt
Copy link

p6rt commented Jul 8, 2015

Migrated from rt.perl.org#125573 (status was 'resolved')

Searchable as RT125573$

@p6rt
Copy link
Author

p6rt commented Jul 8, 2015

From @hoelzro

See the attached file.

In summary, this is what runs​:

  LEAVE die "uh-oh";
  pass;

Even though all of the tests in the file pass, die()ing should be treated as anamolous, and the exit status should be non-zero. The exit status here is zero, however.

@p6rt
Copy link
Author

p6rt commented Jul 8, 2015

From @hoelzro

test.p6

@p6rt
Copy link
Author

p6rt commented Oct 27, 2015

From @FROGGS

You don't even need LEAVE. And, for completeness, using Test.pm seems to make a difference​:

~/dev/rakudo$ perl6 -e 'use Test; die "uh-oh"' && echo damn
uh-oh
  in block <unit> at -e​:1
damn

~/dev/rakudo$ perl6 -e 'die "uh-oh"' && echo damn
uh-oh
  in block <unit> at -e​:1

@p6rt
Copy link
Author

p6rt commented Oct 27, 2015

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Oct 27, 2015

From @FROGGS

Patch​: rakudo/rakudo@6063886343
Test​: Raku/roast@2dfcd85181

Closing ticket.

1 similar comment
@p6rt
Copy link
Author

p6rt commented Oct 27, 2015

From @FROGGS

Patch​: rakudo/rakudo@6063886343
Test​: Raku/roast@2dfcd85181

Closing ticket.

@p6rt
Copy link
Author

p6rt commented Oct 27, 2015

@FROGGS - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant