Moved the fs.Stats assertions to a utility function in a separate file so it can be used by other tests

Author: JamesMessinger

test/specs/stats.spec.js

@@ -2,8 +2,8 @@
 
 const readdir = require("../../");
 const dir = require("../utils/dir");
+const isStats = require("../utils/is-stats");
 const { expect } = require("chai");
-const fs = require("fs");
 
 describe("options.stats", () => {
   describe("Synchronous API", () => {
@@ -79,16 +79,7 @@ describe("options.stats", () => {
 
       // Each item should be a valid fs.Stats object
       for (let stat of data) {
-        expect(stat).to.be.an("object");
-        expect(stat).to.be.an.instanceOf(fs.Stats);
-        expect(stat.mode).to.be.a("number");
-        expect(stat.uid).to.be.a("number");
-        expect(stat.gid).to.be.a("number");
-        expect(stat.size).to.be.a("number");
-        expect(stat.atime).to.be.an.instanceOf(Date);
-        expect(stat.mtime).to.be.an.instanceOf(Date);
-        expect(stat.ctime).to.be.an.instanceOf(Date);
-        expect(stat.path).to.be.a("string");
+        expect(stat).to.satisfy(isStats);
       }
 
       done();

test/utils/is-stats.js

@@ -0,0 +1,35 @@
+"use strict";
+
+const fs = require("fs");
+const { expect } = require("chai");
+
+module.exports = isStats;
+
+/**
+ * Assets that the given objects is a Readdir Enhanced Stats object.
+ */
+function isStats (stats) {
+  expect(stats).to.be.an("object").and.instanceOf(fs.Stats);
+  expect(stats.atime).to.be.an.instanceOf(Date);
+  expect(stats.atimeMs).to.be.a("number").above(0);
+  expect(stats.birthtime).to.be.an.instanceOf(Date);
+  expect(stats.birthtimeMs).to.be.a("number").above(0);
+  expect(stats.blksize).to.be.a("number").above(0);
+  expect(stats.blocks).to.be.a("number").at.least(0);
+  expect(stats.ctime).to.be.an.instanceOf(Date);
+  expect(stats.ctimeMs).to.be.a("number").above(0);
+  expect(stats.depth).to.be.a("number").at.least(0);
+  expect(stats.dev).to.be.a("number").above(0);
+  expect(stats.gid).to.be.a("number").at.least(0);
+  expect(stats.ino).to.be.a("number").above(0);
+  expect(stats.mode).to.be.a("number").above(0);
+  expect(stats.mtime).to.be.an.instanceOf(Date);
+  expect(stats.mtimeMs).to.be.a("number").above(0);
+  expect(stats.nlink).to.be.a("number").above(0);
+  expect(stats.path).to.be.a("string").with.length.above(0);
+  expect(stats.rdev).to.be.a("number").at.least(0);
+  expect(stats.size).to.be.a("number").at.least(0);
+  expect(stats.uid).to.be.a("number").at.least(0);
+
+  return true;
+}