Skip to content

assert,deps,repl: fix repl and assert not being able to handle language features properly #27400

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

Closed
BridgeAR wants to merge 13 commits into from
Closed

assert,deps,repl: fix repl and assert not being able to handle language features properly #27400

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
5ad4a1e
deps: add acorn stage-3 plugins
BridgeAR Apr 24, 2019
5bfaab3
assert: use new language features
BridgeAR Apr 24, 2019
dd3f9b1
repl: add new language features to top level await statements
BridgeAR Apr 24, 2019
39a5a38
repl: handle stage-3 language features properly
BridgeAR Apr 24, 2019
c3c18a1
test: add tests for new language features
BridgeAR Apr 24, 2019
022aa35
fixup! test: add tests for new language features
BridgeAR Apr 25, 2019
7a94f40
fixup! deps: add acorn stage-3 plugins
BridgeAR Apr 26, 2019
172d4be
fixup! assert: use new language features
BridgeAR Apr 26, 2019
a4151f0
fixup! repl: add new language features to top level await statements
BridgeAR Apr 26, 2019
4ed5e0b
fixup! repl: handle stage-3 language features properly
BridgeAR Apr 26, 2019
bc81720
fixup! deps: add acorn stage-3 plugins
BridgeAR Apr 26, 2019
0a069a8
fixup! deps: add acorn stage-3 plugins
BridgeAR Apr 26, 2019
7fb74db
doc: update LICENSE locations
BridgeAR Apr 26, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: add tests for new language features
  • Loading branch information
@BridgeAR
BridgeAR committed Apr 24, 2019
commit c3c18a18525d191eb118432fcb522f35c478aeb6
4 changes: 2 additions & 2 deletions test/parallel/test-assert.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -631,13 +631,13 @@ common.expectsError(
}
);
common.expectsError(
() => assert(typeof 123 === 'string'),
() => assert(typeof 123n === 'string'),
{
code: 'ERR_ASSERTION',
type: assert.AssertionError,
generatedMessage: true,
message: 'The expression evaluated to a falsy value:\n\n ' +
"assert(typeof 123 === 'string')\n"
"assert(typeof 123n === 'string')\n"
Comment thread
BridgeAR marked this conversation as resolved.
}
);

Expand Down
4 changes: 2 additions & 2 deletions test/parallel/test-repl.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -338,12 +338,12 @@ const errorTests = [
expect: '... '
},
{
send: 'return 1;',
send: 'return 1n;',
expect: '... '
},
{
send: '})()',
expect: '1'
expect: '1n'
},
// Multiline function call
{
Expand Down