From bdfeb798ad3b3a6eac5d0c24ec2136480907f783 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Mon, 22 Jun 2015 13:58:29 -0700 Subject: [PATCH] test: remove obsolete TODO comments MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Not using test_ca.pem in these files anymore. Using elipses.txt which has multibyte chars. Not clear what constitutes "large" but that can be a different ticket if elipses.txt etc. are insufficiently large. PR-URL: https://github.com/nodejs/io.js/pull/2032 Reviewed-By: Trevor Norris Reviewed-By: Johan Bergström --- test/parallel/test-fs-read-stream-inherit.js | 4 ---- test/parallel/test-fs-read-stream.js | 4 ---- 2 files changed, 8 deletions(-) diff --git a/test/parallel/test-fs-read-stream-inherit.js b/test/parallel/test-fs-read-stream-inherit.js index 3b53578..2261b26 100644 --- a/test/parallel/test-fs-read-stream-inherit.js +++ b/test/parallel/test-fs-read-stream-inherit.js @@ -2,10 +2,6 @@ var common = require('../common'); var assert = require('assert'); -// TODO Improved this test. test_ca.pem is too small. A proper test would -// great a large utf8 (with multibyte chars) file and stream it in, -// performing sanity checks throughout. - var path = require('path'); var fs = require('fs'); var fn = path.join(common.fixturesDir, 'elipses.txt'); diff --git a/test/parallel/test-fs-read-stream.js b/test/parallel/test-fs-read-stream.js index 490ae08..e6aa997 100644 --- a/test/parallel/test-fs-read-stream.js +++ b/test/parallel/test-fs-read-stream.js @@ -2,10 +2,6 @@ var common = require('../common'); var assert = require('assert'); -// TODO Improved this test. test_ca.pem is too small. A proper test would -// great a large utf8 (with multibyte chars) file and stream it in, -// performing sanity checks throughout. - var path = require('path'); var fs = require('fs'); var fn = path.join(common.fixturesDir, 'elipses.txt'); -- 2.7.4