summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorJamie Davis <davisjam@vt.edu>2017-12-07 12:44:42 -0500
committerRuben Bridgewater <ruben@bridgewater.de>2017-12-12 12:37:18 -0200
commitdd2200ecf831cae5f57b69e3ee8a934efbe20af8 (patch)
treeaa278914a7dfac157b79445971a2a35b057e02e5 /test
parent8d5b3de8cb3467ad9f29232f0ecbbd2fd6f99438 (diff)
downloadnode-new-dd2200ecf831cae5f57b69e3ee8a934efbe20af8.tar.gz
test: add test description to fs.readFile tests
PR-URL: https://github.com/nodejs/node/pull/17610 Refs: https://github.com/nodejs/node/pull/17054#discussion_r155406755 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Jon Moss <me@jonathanmoss.me> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Diffstat (limited to 'test')
-rw-r--r--test/parallel/test-fs-readfile-empty.js3
-rw-r--r--test/parallel/test-fs-readfile-error.js3
-rw-r--r--test/parallel/test-fs-readfile-fd.js3
-rw-r--r--test/parallel/test-fs-readfile-unlink.js3
-rw-r--r--test/parallel/test-fs-readfile-zero-byte-liar.js3
5 files changed, 15 insertions, 0 deletions
diff --git a/test/parallel/test-fs-readfile-empty.js b/test/parallel/test-fs-readfile-empty.js
index bbc2e9c81d..21f99fc6be 100644
--- a/test/parallel/test-fs-readfile-empty.js
+++ b/test/parallel/test-fs-readfile-empty.js
@@ -21,6 +21,9 @@
'use strict';
require('../common');
+
+// Trivial test of fs.readFile on an empty file.
+
const assert = require('assert');
const fs = require('fs');
const fixtures = require('../common/fixtures');
diff --git a/test/parallel/test-fs-readfile-error.js b/test/parallel/test-fs-readfile-error.js
index e7c52f19a8..616760b066 100644
--- a/test/parallel/test-fs-readfile-error.js
+++ b/test/parallel/test-fs-readfile-error.js
@@ -21,6 +21,9 @@
'use strict';
const common = require('../common');
+
+// Test that fs.readFile fails correctly on a non-existent file.
+
// `fs.readFile('/')` does not fail on FreeBSD, because you can open and read
// the directory there.
if (common.isFreeBSD)
diff --git a/test/parallel/test-fs-readfile-fd.js b/test/parallel/test-fs-readfile-fd.js
index 7881666cad..7458af7b2e 100644
--- a/test/parallel/test-fs-readfile-fd.js
+++ b/test/parallel/test-fs-readfile-fd.js
@@ -1,5 +1,8 @@
'use strict';
require('../common');
+
+// Test fs.readFile using a file descriptor.
+
const fixtures = require('../common/fixtures');
const assert = require('assert');
const fs = require('fs');
diff --git a/test/parallel/test-fs-readfile-unlink.js b/test/parallel/test-fs-readfile-unlink.js
index abcbed7ad5..9ec2e849be 100644
--- a/test/parallel/test-fs-readfile-unlink.js
+++ b/test/parallel/test-fs-readfile-unlink.js
@@ -21,6 +21,9 @@
'use strict';
const common = require('../common');
+
+// Test that unlink succeeds immediately after readFile completes.
+
const assert = require('assert');
const fs = require('fs');
const path = require('path');
diff --git a/test/parallel/test-fs-readfile-zero-byte-liar.js b/test/parallel/test-fs-readfile-zero-byte-liar.js
index 625438e7c6..ec0b9cf692 100644
--- a/test/parallel/test-fs-readfile-zero-byte-liar.js
+++ b/test/parallel/test-fs-readfile-zero-byte-liar.js
@@ -21,6 +21,9 @@
'use strict';
const common = require('../common');
+
+// Test that readFile works even when stat returns size 0.
+
const assert = require('assert');
const fs = require('fs');