Skip to content

Commit bdfeb79

Browse files
committed
test: remove obsolete TODO comments
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: #2032 Reviewed-By: Trevor Norris <[email protected]> Reviewed-By: Johan Bergström <[email protected]>
1 parent 834a365 commit bdfeb79

File tree

2 files changed

+0
-8
lines changed

2 files changed

+0
-8
lines changed

test/parallel/test-fs-read-stream-inherit.js

-4
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
var common = require('../common');
33
var assert = require('assert');
44

5-
// TODO Improved this test. test_ca.pem is too small. A proper test would
6-
// great a large utf8 (with multibyte chars) file and stream it in,
7-
// performing sanity checks throughout.
8-
95
var path = require('path');
106
var fs = require('fs');
117
var fn = path.join(common.fixturesDir, 'elipses.txt');

test/parallel/test-fs-read-stream.js

-4
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
var common = require('../common');
33
var assert = require('assert');
44

5-
// TODO Improved this test. test_ca.pem is too small. A proper test would
6-
// great a large utf8 (with multibyte chars) file and stream it in,
7-
// performing sanity checks throughout.
8-
95
var path = require('path');
106
var fs = require('fs');
117
var fn = path.join(common.fixturesDir, 'elipses.txt');

0 commit comments

Comments
 (0)