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

t/spec/S32-io/dir.t #5149

Closed
p6rt opened this issue Feb 22, 2016 · 4 comments
Closed

t/spec/S32-io/dir.t #5149

p6rt opened this issue Feb 22, 2016 · 4 comments

Comments

@p6rt
Copy link

p6rt commented Feb 22, 2016

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

Searchable as RT127591$

@p6rt
Copy link
Author

p6rt commented Feb 22, 2016

From @coke

This test file crashes with a release candidate for the 2016.02
branch, against the 6.c-errata roast branch​:

$ prove -v -e t/fudgeandrun t/spec/S32-io/dir.t
t/spec/S32-io/dir.t ..
1..13
ok 1 - dir() runs in cwd()
not ok 2 - current directory contains a t/ dir# TODO directories are
not marked with trailing / yet RT \#​124784

# Failed test 'current directory contains a t/ dir'
# at t/spec/S32-io/dir.rakudo.moar line 14
ok 3 - current directory contains a t/ dir
Could not find symbol '&Local'
  in block <unit> at t/spec/S32-io/dir.rakudo.moar line 17

Actually thrown at​:
  in block <unit> at t/spec/S32-io/dir.rakudo.moar line 17

# Looks like you planned 13 tests, but ran 3
Failed 10/13 subtests

Test Summary Report


t/spec/S32-io/dir.t (Wstat​: 0 Tests​: 3 Failed​: 0)
  Parse errors​: Bad plan. You planned 13 tests but ran 3.
Files=1, Tests=3, 1 wallclock secs ( 0.03 usr 0.00 sys + 0.59 cusr
0.12 csys = 0.74 CPU)
Result​: FAIL

--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Feb 22, 2016

From @moritz

On Sun Feb 21 19​:15​:22 2016, coke wrote​:

This test file crashes with a release candidate for the 2016.02
branch, against the 6.c-errata roast branch​:

$ prove -v -e t/fudgeandrun t/spec/S32-io/dir.t
t/spec/S32-io/dir.t ..
1..13
ok 1 - dir() runs in cwd()
not ok 2 - current directory contains a t/ dir# TODO directories are
not marked with trailing / yet RT \#​124784

# Failed test 'current directory contains a t/ dir'
# at t/spec/S32-io/dir.rakudo.moar line 14
ok 3 - current directory contains a t/ dir
Could not find symbol '&Local'
in block <unit> at t/spec/S32-io/dir.rakudo.moar line 17

This is a test that's TODO'd, what's changed is that the role that is tested against is gone.

Since it never worked, I'm inclined to remove that particular subtest

@p6rt
Copy link
Author

p6rt commented Feb 22, 2016

From @moritz

Relaxed the test in Raku/roast@31fa3f9463

@p6rt p6rt closed this as completed Feb 22, 2016
@p6rt
Copy link
Author

p6rt commented Feb 22, 2016

@moritz - Status changed from 'new' to 'resolved'

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

No branches or pull requests

1 participant