FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

(v6.x backport) test: fix args in parallel/test-fs-null-bytes.js by vsemozhetbyt · Pull Request #12477 · nodejs/node · GitHub

/ node Public

(v6.x backport) test: fix args in parallel/test-fs-null-bytes.js - #12477

Closed
vsemozhetbyt wants to merge 1 commit into
nodejs:v6.x-stagingfrom
vsemozhetbyt:backport-11601-to-v6.x
Closed

(v6.x backport) test: fix args in parallel/test-fs-null-bytes.js#12477
vsemozhetbyt wants to merge 1 commit into
nodejs:v6.x-stagingfrom
vsemozhetbyt:backport-11601-to-v6.x

Conversation

Copy link
Copy Markdown
Contributor
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test, fs

Backport of #11601

The functions fs.appendFile() and fs.writeFile() were being called without the required data argument.

Refs: #11595

The functions `fs.appendFile()` and `fs.writeFile()`
were being called without the required `data` argument.

Refs: #11595
PR-URL: #11601
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Nikolai Vavilov <vvnicholas@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
nodejs-github-bot added test Issues and PRs related to the tests. v6.x labels Apr 18, 2017
vsemozhetbyt added the fs Issues and PRs related to the fs subsystem / file system. label Apr 18, 2017

Copy link
Copy Markdown
Contributor Author

MylesBorins pushed a commit that referenced this pull request Apr 18, 2017
The functions `fs.appendFile()` and `fs.writeFile()`
were being called without the required `data` argument.

Refs: #11595
Backport-PR-URL: #12477
PR-URL: #11601
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Nikolai Vavilov <vvnicholas@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

Copy link
Copy Markdown
Contributor

landed in 4180204

vsemozhetbyt deleted the backport-11601-to-v6.x branch April 18, 2017 09:47
MylesBorins pushed a commit that referenced this pull request Apr 19, 2017
The functions `fs.appendFile()` and `fs.writeFile()`
were being called without the required `data` argument.

Refs: #11595
Backport-PR-URL: #12477
PR-URL: #11601
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Nikolai Vavilov <vvnicholas@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
andrew749 pushed a commit to michielbaird/node that referenced this pull request Jul 19, 2017
The functions `fs.appendFile()` and `fs.writeFile()`
were being called without the required `data` argument.

Refs: nodejs/node#11595
Backport-PR-URL: nodejs/node#12477
PR-URL: nodejs/node#11601
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Nikolai Vavilov <vvnicholas@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fs Issues and PRs related to the fs subsystem / file system. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL