Skip to content

test: in test/parallel/test-fs-realpath.js:

Rodrigo Muino Tomonari requested to merge github/fork/mpmckenna8/mpmckenna8tests into master
Checklist
  • make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

test: in test/parallel/test-fs-realpath.js:

Replaced .indexOf() with .includes() for more clarity as to what it is doing.

Many of the instances of var changed to const.

Instances of assert.equal() refactored to assert.strictEqual() or assert.ifError()

Removed the unlinkSync() call in the exit event handler because it probably only handles files in the testing tmp directory so there is no need to unlink them.

Merge request reports