From 776a65ebcd42d4bbdb24a1b87c79caf424913dfc Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Mon, 22 Jun 2015 11:53:50 -0700 Subject: [PATCH] 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 --- test/parallel/test-fs-readfile-pipe-large.js | 1 - test/parallel/test-fs-readfile-pipe.js | 1 - test/parallel/test-fs-readfilesync-pipe-large.js | 1 - 3 files changed, 3 deletions(-) diff --git a/test/parallel/test-fs-readfile-pipe-large.js b/test/parallel/test-fs-readfile-pipe-large.js index ac50e9c013..690da928e7 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 b55bfb6a22..80cbaeac7c 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 ebd093de69..991c8589de 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();