New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

swap actual and expected in assertions #23474

Closed

yitongding wants to merge 2 commits into nodejs:master from yitongding:fixAssertInTestWriteFile

Contributor

yitongding commented Oct 12, 2018

test/parallel/test-fs-write.js contains assertions where the first
argument provided is the expected value and the second value is the
actual value. This is backward from the documentation for assertions
like assert.strictEqual() where the first value should be the actual
value being tested and the second value is the expected value.

Checklist

make -j4 test (UNIX), or vcbuild test (Windows) passes
commit message follows commit guidelines

yitongding added 2 commits

October 12, 2018 10:03


          test: swap actual and expected in assertions

fff9705

'test/parallel/test-fs-write.js' contains assertions where the first
argument provided is the expected value and the second value is the
actual value. This is backward from the documentation for assertions
like 'assert.strictEqual()' where the first value should be the actual
value being tested and the second value is the expected value.


          test: replace function with arrow function

31d05ab

nodejs-github-bot added the test label

cjihrig approved these changes

View reviewed changes

targos added the code-and-learn label

Member

gireeshpunathil commented Oct 12, 2018

thanks for the detailed PR description!

gireeshpunathil approved these changes

View reviewed changes

jasnell approved these changes

View reviewed changes

trivikr approved these changes

View reviewed changes

codebytere approved these changes

View reviewed changes

Trott mentioned this pull request

Code and learn rollup 2 #23651

Closed

Member

Trott commented Oct 14, 2018

CI: https://ci.nodejs.org/job/node-test-pull-request/17843/

addaleax added the fs label

Member

addaleax commented Oct 14, 2018

Landed in cf7b5a8...e48d9aa

Thanks for the contribution! 🎉

(If you're interested in other possible contributions to Node.js but don't have a good idea of where to start looking, some ideas are posted at https://www.nodetodo.org/next-steps/.)

addaleax closed this

addaleax pushed a commit that referenced this pull request


          test: swap actual and expected in assertions

a9e6e68

'test/parallel/test-fs-write.js' contains assertions where the first
argument provided is the expected value and the second value is the
actual value. This is backward from the documentation for assertions
like 'assert.strictEqual()' where the first value should be the actual
value being tested and the second value is the expected value.

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

addaleax pushed a commit that referenced this pull request


          test: replace function with arrow function

e48d9aa

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

yitongding deleted the fixAssertInTestWriteFile branch

October 14, 2018 02:56

addaleax pushed a commit that referenced this pull request


          test: swap actual and expected in assertions

a5c698b

'test/parallel/test-fs-write.js' contains assertions where the first
argument provided is the expected value and the second value is the
actual value. This is backward from the documentation for assertions
like 'assert.strictEqual()' where the first value should be the actual
value being tested and the second value is the expected value.

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

addaleax pushed a commit that referenced this pull request


          test: replace function with arrow function

21fc4ee

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

jasnell pushed a commit that referenced this pull request


          test: swap actual and expected in assertions

923f37f

'test/parallel/test-fs-write.js' contains assertions where the first
argument provided is the expected value and the second value is the
actual value. This is backward from the documentation for assertions
like 'assert.strictEqual()' where the first value should be the actual
value being tested and the second value is the expected value.

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

jasnell pushed a commit that referenced this pull request


          test: replace function with arrow function

ec675b8

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

MylesBorins pushed a commit that referenced this pull request


          test: swap actual and expected in assertions

a6787a7

'test/parallel/test-fs-write.js' contains assertions where the first
argument provided is the expected value and the second value is the
actual value. This is backward from the documentation for assertions
like 'assert.strictEqual()' where the first value should be the actual
value being tested and the second value is the expected value.

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

MylesBorins pushed a commit that referenced this pull request


          test: replace function with arrow function

b436ccc

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

codebytere mentioned this pull request

v10.13.1 proposal #24675

Closed

rvagg pushed a commit that referenced this pull request


          test: swap actual and expected in assertions

f91dd7b

'test/parallel/test-fs-write.js' contains assertions where the first
argument provided is the expected value and the second value is the
actual value. This is backward from the documentation for assertions
like 'assert.strictEqual()' where the first value should be the actual
value being tested and the second value is the expected value.

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

rvagg pushed a commit that referenced this pull request


          test: replace function with arrow function

7a5c900

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

MylesBorins pushed a commit that referenced this pull request


          test: swap actual and expected in assertions

762c2d0

'test/parallel/test-fs-write.js' contains assertions where the first
argument provided is the expected value and the second value is the
actual value. This is backward from the documentation for assertions
like 'assert.strictEqual()' where the first value should be the actual
value being tested and the second value is the expected value.

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

MylesBorins pushed a commit that referenced this pull request


          test: replace function with arrow function

1ae0aaf

PR-URL: #23474
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>

codebytere mentioned this pull request

v10.14.2 proposal #24727

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

trivikr trivikr approved these changes

gireeshpunathil gireeshpunathil approved these changes

cjihrig cjihrig approved these changes

codebytere codebytere approved these changes

jasnell jasnell approved these changes

Labels

code-and-learn fs test