test: refactor to eliminate __defineGetter__

In preparation for a lint rule to flag `__defineGetter__`, refactor the one remaining instance in the code base.
nodejs · May 18, 2016 · 083a23e · 083a23e
1 parent 0350505
commit 083a23e
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/test/parallel/test-util-inspect.js b/test/parallel/test-util-inspect.js
@@ -205,9 +205,14 @@ assert.equal(util.inspect(value), '{ a: [Circular] }');
 
 // Array with dynamic properties
 value = [1, 2, 3];
-value.__defineGetter__('growingLength', function() {
-  this.push(true); return this.length;
-});
+Object.defineProperty(
+  value,
+  'growingLength',
+  {
+    enumerable: true,
+    get: () => { this.push(true); return this.length; }
+  }
+);
 assert.equal(util.inspect(value), '[ 1, 2, 3, growingLength: [Getter] ]');
 
 // Function with properties