test: refactor to eliminate __defineGetter__

Trott · Trott · commit 37a5a1c49757 · 2016-05-19T22:07:50.000-07:00
In preparation for a lint rule to flag `__defineGetter__`, refactor the one remaining instance in the code base. PR-URL: #6774 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com>
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
