Skip to content

test: remove `require('buffer')` on 4 test files

Rodrigo Muino Tomonari requested to merge github/fork/XadillaX/test/delete-some-require-buffer into master

We don't use the global Buffer throughout the lib/ to avoid circular dependency issues in core, but we actually don't need to require it on test files. So remove them on:

  • test/parallel/test-stream-uint8array.js
  • test/parallel/test-stream2-finish-pipe.js
  • test/parallel/test-tls-session-cache.js
  • test/parallel/test-vm-cached-data.js

Refs: https://github.com/nodejs/node/issues/13836

Checklist
  • make -j4 test (UNIX)
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test, buffer

Merge request reports