Skip to content

Commit

Permalink
Document intent of &*chdir tests
Browse files Browse the repository at this point in the history
Can't figure out a way to prove from within roast that process dir
changes as well (without using `nqp` and its `nqp::cwd` op).

For purposes of release, we'll just document the intent of the test
in the comments, and hopefully will come up with proper proof later on

Addresses S#471 Raku#471 sufficiently
enough for 6.d release purposes, to not block the release
  • Loading branch information
zoffixznet committed Nov 4, 2018
1 parent 51639a8 commit 90ca958
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions S32-io/chdir-process.t
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,11 @@ plan :skip-all<Cannot test without $*TMPDIR and $*HOME set to readable dirs>
unless $*TMPDIR ~~ :d & :r and $*HOME ~~ :d & :r;

plan 8;

# XXX TODO: `&*chdir` changes process dir, however `qx` respects `$*CWD`,
# so these tests do not currently manage to prove the process directory changes
# along with the value of `$*CWD` as well:
# https://github.com/perl6/roast/issues/471
my &sys-cwd = $*DISTRO.is-win ?? {qx`echo %cd%`.chomp.IO} !! {qx`pwd`.chomp.IO};

subtest '&*chdir into IO::Path respects its :CWD attribute' => {
Expand Down

0 comments on commit 90ca958

Please sign in to comment.