From 40d59281f737103b23c1e5e0a997a15b5f4caf7c Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Tue, 17 Nov 2020 11:57:33 -0800 Subject: [PATCH] test: update comments in test-fs-read-offset-null Update comment to refer to the correct ASCII code (120 rather than 66). All other changes are cosmetic. PR-URL: https://github.com/nodejs/node/pull/36152 Reviewed-By: Luigi Pinca Reviewed-By: James M Snell Reviewed-By: Trivikram Kamat --- test/parallel/test-fs-read-offset-null.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/test/parallel/test-fs-read-offset-null.js b/test/parallel/test-fs-read-offset-null.js index 96bcbf0f33d910..27e893f781b2ff 100644 --- a/test/parallel/test-fs-read-offset-null.js +++ b/test/parallel/test-fs-read-offset-null.js @@ -12,17 +12,16 @@ const fixtures = require('../common/fixtures'); const filepath = fixtures.path('x.txt'); const buf = Buffer.alloc(1); -// Reading only one character, hence buffer of one byte is enough +// Reading only one character, hence buffer of one byte is enough. -// Test for callback api +// Test for callback API. fs.open(filepath, 'r', common.mustSucceed((fd) => { fs.read(fd, { offset: null, buffer: buf }, common.mustSucceed((bytesRead, buffer) => { - assert.strictEqual(buffer[0], 120); // Test is done by making sure the first letter in buffer is // same as first letter in file. - // 66 is the hex for ascii code of letter B - + // 120 is the hex for ascii code of letter x. + assert.strictEqual(buffer[0], 120); fs.close(fd, common.mustSucceed(() => {})); })); }));