Skip to content

Commit b3b2456

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 cdf4a9c commit b3b2456

5 files changed

+15
-0
lines changed

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

+3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
'use strict';
22
require('../common');
3+
4+
// Trivial test of fs.readFile on an empty file.
5+
36
const assert = require('assert');
47
const fs = require('fs');
58
const fixtures = require('../common/fixtures');

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

+3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
'use strict';
22
const common = require('../common');
3+
4+
// Test that fs.readFile fails correctly on a non-existent file.
5+
36
// `fs.readFile('/')` does not fail on FreeBSD, because you can open and read
47
// the directory there.
58
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
@@ -1,5 +1,8 @@
11
'use strict';
22
const common = require('../common');
3+
4+
// Test that unlink succeeds immediately after readFile completes.
5+
36
const assert = require('assert');
47
const fs = require('fs');
58
const path = require('path');

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

+3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
'use strict';
22
const common = require('../common');
3+
4+
// Test that readFile works even when stat returns size 0.
5+
36
const assert = require('assert');
47
const fs = require('fs');
58

0 commit comments

Comments
 (0)