Skip to content

Commit

Permalink
This test is now passing on OS X.
Browse files Browse the repository at this point in the history 
Closes #274
  • Loading branch information
@coke
coke committed Apr 23, 2018
1 parent a24e43d commit 6c982a8
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions t/nqp/019-file-ops.t
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

# Test nqp::op file operations.

plan(109);
plan(112);

ok( nqp::stat('CREDITS', nqp::const::STAT_EXISTS) == 1, 'nqp::stat exists');
ok( nqp::stat('AARDVARKS', nqp::const::STAT_EXISTS) == 0, 'nqp::stat not exists');
Expand Down Expand Up @@ -347,9 +347,7 @@ else {
ok(nqp::stat_time($file, $flag) == nqp::lstat_time($file, $flag), 'stat_time works as lstat_time on regular file');
ok(nqp::stat($file, $flag) == nqp::lstat($file, $flag), 'stat works as lstat on regular file');
ok(nqp::stat_time($symlink, $flag) == nqp::lstat_time($file, $flag), 'stat_time follows symlink');
### This test was added between 2015.12 and 2016.01, but was failing.
# since it's something new, commenting out for 2016.01 release.
#ok(nqp::lstat_time($symlink, $flag) != nqp::lstat_time($file, $flag), 'lstat_time doesn\'t follow symlink');
ok(nqp::lstat_time($symlink, $flag) != nqp::lstat_time($file, $flag), 'lstat_time doesn\'t follow symlink');
}


Expand Down

2 comments on commit 6c982a8

@zoffixznet
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems Travis disagrees with that assessment: https://travis-ci.org/MoarVM/MoarVM/jobs/370869349

@coke
Copy link
Contributor Author

@coke coke commented on 6c982a8 Apr 25, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was already reverted and the ticket re-opened.

Please sign in to comment.