test: fix fs test-fs-utimes strictEqual arg order

bnoordhuis · targos · commit 2e015e5b5e48 · 2020-04-11T10:05:42.000+02:00
`actual` is the first argument, `expected` the second, but the test flipped them around and was producing confusing assertion messages as a result. Refs: #32408 (comment) PR-URL: #32420 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/test/parallel/test-fs-utimes.js b/test/parallel/test-fs-utimes.js
@@ -150,15 +150,15 @@ if (!process.arch.includes('arm') &&
   const Y2K38_mtime = 2 ** 31;
   fs.utimesSync(path, Y2K38_mtime, Y2K38_mtime);
   const Y2K38_stats = fs.statSync(path);
-  assert.strictEqual(Y2K38_mtime, Y2K38_stats.mtime.getTime() / 1000);
+  assert.strictEqual(Y2K38_stats.mtime.getTime() / 1000, Y2K38_mtime);
 }
 
 if (common.isWindows) {
   // This value would get converted to (double)1713037251359.9998
   const truncate_mtime = 1713037251360;
   fs.utimesSync(path, truncate_mtime / 1000, truncate_mtime / 1000);
   const truncate_stats = fs.statSync(path);
-  assert.strictEqual(truncate_mtime, truncate_stats.mtime.getTime());
+  assert.strictEqual(truncate_stats.mtime.getTime(), truncate_mtime);
 
   // test Y2K38 for windows
   // This value if treaded as a `signed long` gets converted to -2135622133469.
@@ -168,7 +168,7 @@ if (common.isWindows) {
   const overflow_mtime = 2159345162531;
   fs.utimesSync(path, overflow_mtime / 1000, overflow_mtime / 1000);
   const overflow_stats = fs.statSync(path);
-  assert.strictEqual(overflow_mtime, overflow_stats.mtime.getTime());
+  assert.strictEqual(overflow_stats.mtime.getTime(), overflow_mtime);
 }
 
 const expectTypeError = {
