Skip to content

Commit

Permalink
test: refactor test-util-inspect.js
Browse files Browse the repository at this point in the history
* Enclose tests that used to introduce module-level variables into
  their own scopes.
* Replace ES5 anonymous functions with arrow functions where it makes
  sense.
* And make one arrow function a regular function thus fixing a bug in a
  getter inside an object created in "Array with dynamic properties"
  test.  This getter has never been invoked though, so the test hasn't been
  failing.
* Convert snake_case identifiers to camelCase.
* Make some variable names more readable.
* Replace regular expressions in maxArrayLength tests with simple
  assert.strictEquals() and assert(...endsWith()) checks, as suggested
  in <nodejs#11576 (comment)>.
  • Loading branch information
aqrln committed Mar 10, 2017
1 parent ec2f098 commit fc23c8d
Showing 1 changed file with 322 additions and 281 deletions.
Loading

0 comments on commit fc23c8d

Please sign in to comment.