From da572131db8178cc8f46b96e531d8e0873f6b222 Mon Sep 17 00:00:00 2001
From: Adrian Estrada <edsadr@gmail.com>
Date: Sun, 8 Jan 2017 18:41:26 -0500
Subject: [PATCH] test: improve the code in test-process-hrtime

* use const instead of var
* use assert.strictEqual instead of assert.equal and plain assert
* use arrow functions
* swap assertions arguments to match the standard
* validate the error for assert.throws

PR-URL: https://github.com/nodejs/node/pull/10667
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michal Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
---
 test/parallel/test-process-hrtime.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/test/parallel/test-process-hrtime.js b/test/parallel/test-process-hrtime.js
index db16be0ad9e812..9280b671f316d6 100644
--- a/test/parallel/test-process-hrtime.js
+++ b/test/parallel/test-process-hrtime.js
@@ -3,7 +3,7 @@ require('../common');
 var assert = require('assert');
 
 // the default behavior, return an Array "tuple" of numbers
-var tuple = process.hrtime();
+const tuple = process.hrtime();
 
 // validate the default behavior
 validateTuple(tuple);
@@ -12,16 +12,16 @@ validateTuple(tuple);
 validateTuple(process.hrtime(tuple));
 
 // test that only an Array may be passed to process.hrtime()
-assert.throws(function() {
+assert.throws(() => {
   process.hrtime(1);
-});
+}, /^TypeError: process.hrtime\(\) only accepts an Array tuple$/);
 
 function validateTuple(tuple) {
   assert(Array.isArray(tuple));
-  assert.equal(2, tuple.length);
-  tuple.forEach(function(v) {
-    assert.equal('number', typeof v);
-    assert(isFinite(v));
+  assert.strictEqual(tuple.length, 2);
+  tuple.forEach((v) => {
+    assert.strictEqual(typeof v, 'number');
+    assert.strictEqual(isFinite(v), true);
   });
 }