Commit 67bc2ad
test: improve code in test-vm-symbols
* use const instead of var
* use assert.strictEqual instead of assert.equal
PR-URL: #10429
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent f9ff4ec commit 67bc2ad
1 file changed
Lines changed: 8 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1,11 +1,11 @@ | |||
| 1 | 1 | 'use strict'; | |
| 2 | 2 | ||
| 3 | 3 | require('../common'); | |
| 4 | - var assert = require('assert'); | ||
| 4 | + const assert = require('assert'); | ||
| 5 | 5 | ||
| 6 | - var vm = require('vm'); | ||
| 6 | + const vm = require('vm'); | ||
| 7 | 7 | ||
| 8 | - var symbol = Symbol(); | ||
| 8 | + const symbol = Symbol(); | ||
| 9 | 9 | ||
| 10 | 10 | function Document() { | |
| 11 | 11 | this[symbol] = 'foo'; | |
@@ -15,11 +15,11 @@ Document.prototype.getSymbolValue = function() { | |||
| 15 | 15 | return this[symbol]; | |
| 16 | 16 | }; | |
| 17 | 17 | ||
| 18 | - var context = new Document(); | ||
| 18 | + const context = new Document(); | ||
| 19 | 19 | vm.createContext(context); | |
| 20 | 20 | ||
| 21 | - assert.equal(context.getSymbolValue(), 'foo', | ||
| 22 | - 'should return symbol-keyed value from the outside'); | ||
| 21 | + assert.strictEqual(context.getSymbolValue(), 'foo', | ||
| 22 | + 'should return symbol-keyed value from the outside'); | ||
| 23 | 23 | ||
| 24 | - assert.equal(vm.runInContext('this.getSymbolValue()', context), 'foo', | ||
| 25 | - 'should return symbol-keyed value from the inside'); | ||
| 24 | + assert.strictEqual(vm.runInContext('this.getSymbolValue()', context), 'foo', | ||
| 25 | + 'should return symbol-keyed value from the inside'); | ||
0 commit comments