Skip to content

Commit bc7dc65

Browse files
davisjamMylesBorins
authored andcommitted
test: add test description to fs.readFile tests
PR-URL: #17610 Refs: #17054 (comment) Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
1 parent 70588f7 commit bc7dc65

5 files changed

+15
-0
lines changed

test/parallel/test-fs-readfile-empty.js

+3
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@
2121

2222
'use strict';
2323
require('../common');
24+
25+
// Trivial test of fs.readFile on an empty file.
26+
2427
const assert = require('assert');
2528
const fs = require('fs');
2629
const fixtures = require('../common/fixtures');

test/parallel/test-fs-readfile-error.js

+3
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@
2121

2222
'use strict';
2323
const common = require('../common');
24+
25+
// Test that fs.readFile fails correctly on a non-existent file.
26+
2427
// `fs.readFile('/')` does not fail on FreeBSD, because you can open and read
2528
// the directory there.
2629
if (common.isFreeBSD)

test/parallel/test-fs-readfile-fd.js

+3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
'use strict';
22
require('../common');
3+
4+
// Test fs.readFile using a file descriptor.
5+
36
const fixtures = require('../common/fixtures');
47
const assert = require('assert');
58
const fs = require('fs');

test/parallel/test-fs-readfile-unlink.js

+3
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@
2121

2222
'use strict';
2323
const common = require('../common');
24+
25+
// Test that unlink succeeds immediately after readFile completes.
26+
2427
const assert = require('assert');
2528
const fs = require('fs');
2629
const path = require('path');

test/parallel/test-fs-readfile-zero-byte-liar.js

+3
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@
2121

2222
'use strict';
2323
const common = require('../common');
24+
25+
// Test that readFile works even when stat returns size 0.
26+
2427
const assert = require('assert');
2528
const fs = require('fs');
2629

0 commit comments

Comments
 (0)