diff options
author | Rich Trott <rtrott@gmail.com> | 2015-06-22 11:53:50 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2015-06-23 15:19:28 -0700 |
commit | 776a65ebcd42d4bbdb24a1b87c79caf424913dfc (patch) | |
tree | b9c5691686806ce456e2c130f2cdc1b46b52093c | |
parent | bdfeb798ad3b3a6eac5d0c24ec2136480907f783 (diff) | |
download | nodejs-776a65ebcd42d4bbdb24a1b87c79caf424913dfc.tar.gz nodejs-776a65ebcd42d4bbdb24a1b87c79caf424913dfc.tar.bz2 nodejs-776a65ebcd42d4bbdb24a1b87c79caf424913dfc.zip |
test: remove obsolete TODO comments
The readfile/pipe tests rely on pre-existing pipes in the system.
This arguably tests the OS functionality and not really io.js
functionality. Removing TODOs.
PR-URL: https://github.com/nodejs/io.js/pull/2033
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
-rw-r--r-- | test/parallel/test-fs-readfile-pipe-large.js | 1 | ||||
-rw-r--r-- | test/parallel/test-fs-readfile-pipe.js | 1 | ||||
-rw-r--r-- | test/parallel/test-fs-readfilesync-pipe-large.js | 1 |
3 files changed, 0 insertions, 3 deletions
diff --git a/test/parallel/test-fs-readfile-pipe-large.js b/test/parallel/test-fs-readfile-pipe-large.js index ac50e9c01..690da928e 100644 --- a/test/parallel/test-fs-readfile-pipe-large.js +++ b/test/parallel/test-fs-readfile-pipe-large.js @@ -5,7 +5,6 @@ var path = require('path'); // simulate `cat readfile.js | node readfile.js` -// TODO: Have some way to make this work on windows. if (process.platform === 'win32') { console.error('No /dev/stdin on windows. Skipping test.'); process.exit(); diff --git a/test/parallel/test-fs-readfile-pipe.js b/test/parallel/test-fs-readfile-pipe.js index b55bfb6a2..80cbaeac7 100644 --- a/test/parallel/test-fs-readfile-pipe.js +++ b/test/parallel/test-fs-readfile-pipe.js @@ -4,7 +4,6 @@ var assert = require('assert'); // simulate `cat readfile.js | node readfile.js` -// TODO: Have some way to make this work on windows. if (process.platform === 'win32') { console.error('No /dev/stdin on windows. Skipping test.'); process.exit(); diff --git a/test/parallel/test-fs-readfilesync-pipe-large.js b/test/parallel/test-fs-readfilesync-pipe-large.js index ebd093de6..991c8589d 100644 --- a/test/parallel/test-fs-readfilesync-pipe-large.js +++ b/test/parallel/test-fs-readfilesync-pipe-large.js @@ -5,7 +5,6 @@ var path = require('path'); // simulate `cat readfile.js | node readfile.js` -// TODO: Have some way to make this work on windows. if (process.platform === 'win32') { console.error('No /dev/stdin on windows. Skipping test.'); process.exit(); |