summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRich Trott <rtrott@gmail.com>2015-06-22 11:53:50 -0700
committerRich Trott <rtrott@gmail.com>2015-06-23 15:19:28 -0700
commit776a65ebcd42d4bbdb24a1b87c79caf424913dfc (patch)
treeb9c5691686806ce456e2c130f2cdc1b46b52093c
parentbdfeb798ad3b3a6eac5d0c24ec2136480907f783 (diff)
downloadnodejs-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.js1
-rw-r--r--test/parallel/test-fs-readfile-pipe.js1
-rw-r--r--test/parallel/test-fs-readfilesync-pipe-large.js1
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();