fix: correctly call fs.statSync when there are no options

- previously if the second parameter was a callback, it'd be passed as options to fs.statSync - who would ignore it, but wouldn't be fast about it (see below)
- also makes performance on-par with enhanced-resolve 5.1.0 which in some benchmarks is about 3x faster than versions >= 5.2.0 || <= 5.8.0

Author: sverweij

lib/SyncAsyncFileSystemDecorator.js

@@ -34,7 +34,7 @@ function SyncAsyncFileSystemDecorator(fs) {
 	this.stat = (arg, options, callback) => {
 		let result;
 		try {
-			result = fs.statSync(arg, options);
+			result = callback ? fs.statSync(arg, options) : fs.statSync(arg);
 		} catch (e) {
 			return (callback || options)(e);
 		}

test/SyncAsyncFileSystemDecorator.js

@@ -0,0 +1,35 @@
+"use strict";
+
+const fs = require("fs");
+const path = require("path");
+const should = require("should");
+const SyncAsyncFileSystemDecorator = require("../lib/SyncAsyncFileSystemDecorator");
+
+describe("SyncAsyncFileSystemDecorator  stat", function () {
+	it("should use options when they're provided", function (done) {
+		const decoratedFs = new SyncAsyncFileSystemDecorator(fs);
+		decoratedFs.stat(
+			path.join(__dirname, "fixtures", "decorated-fs", "exists.js"),
+			{ bigint: true },
+			function (error, result) {
+				should(error).be.null();
+				should(result).have.properties(["size", "birthtime"]);
+				should(result.size).be.of.type("bigint");
+				done();
+			}
+		);
+	});
+
+	it("should work correctly when no options provided", function (done) {
+		const decoratedFs = new SyncAsyncFileSystemDecorator(fs);
+		decoratedFs.stat(
+			path.join(__dirname, "fixtures", "decorated-fs", "exists.js"),
+			function (error, result) {
+				should(error).be.null();
+				should(result).have.properties(["size", "birthtime"]);
+				should(result.size).be.of.type("number");
+				done();
+			}
+		);
+	});
+});