gulp-expect-file 2.0.0 → 3.0.0

package/README.md

@@ -1,8 +1,8 @@
-# gulp-expect-file [![NPM version][npm-image]][npm-url] [![Build Status][gh-image]][gh-url] [![Dependency Status][daviddm-image]][daviddm-url]
+# gulp-expect-file [![NPM version][npm-image]][npm-url] [![Build Status][gh-image]][gh-url]
 
-> Expectation on generated files for gulp 3/4
+> Expectation on generated files for Gulp 4
 
-This plugin is intended for testing other gulp plugin.
+This plugin is intended for testing other Gulp plugin.
 
 ![Screen Shot](http://pioug.github.io/gulp-expect-file/screenshot.png)
 
@@ -20,10 +20,7 @@ Then, add it to your `gulpfile.js`:
 var expect = require("gulp-expect-file");
 
 gulp.task("copy", function () {
-  gulp
-    .src(["src/foo.txt"])
-    .pipe(gulp.dest("dest/"))
-    .pipe(expect("dest/foo.txt"));
+  gulp.src(["src/foo.txt"]).pipe(gulp.dest("dest/")).pipe(expect("dest/foo.txt"));
 });
 ```
 
@@ -114,9 +111,7 @@ Default: `false`
 If true, it also checks if the real file exists on the file system by `fs.exists()`.
 
 ```js
-gulp
-  .src(["exist.txt", "nonexist.txt"])
-  .pipe(expect({ checkRealFile: true }, "*.txt"));
+gulp.src(["exist.txt", "nonexist.txt"]).pipe(expect({ checkRealFile: true }, "*.txt"));
 
 // => FAIL: nonexist.txt not exists on filesystem
 ```
@@ -159,5 +154,3 @@ This is just a shortcut for `expect({ checkRealFile: true }, expectation)`.
 [npm-image]: https://img.shields.io/npm/v/gulp-expect-file.svg
 [gh-url]: https://github.com/pioug/gulp-expect-file/actions/workflows/test.yml
 [gh-image]: https://github.com/pioug/gulp-expect-file/actions/workflows/test.yml/badge.svg
-[daviddm-url]: https://david-dm.org/pioug/gulp-expect-file
-[daviddm-image]: https://img.shields.io/david/pioug/gulp-expect-file.svg

package/index.js

@@ -36,7 +36,7 @@ function expect(options, expectation) {
       silent: false,
       verbose: false,
     },
-    options
+    options,
   );
 
   try {
@@ -94,10 +94,7 @@ function expect(options, expectation) {
     if (numFailures > 0 && options.errorOnFailure) {
       this.emit(
         "error",
-        new PluginError(
-          "gulp-expect-file",
-          "Failed " + numFailures + " expectations"
-        )
+        new PluginError("gulp-expect-file", "Failed " + numFailures + " expectations"),
       );
     }
 
@@ -107,20 +104,13 @@ function expect(options, expectation) {
 
   function reportFailure(file, err) {
     if (err instanceof ExpectationError) {
-      options.silent ||
-        log(
-          colors.red("\u2717 FAIL:"),
-          colors.magenta(file.relative),
-          "is",
-          err.message
-        );
+      if (!options.silent) {
+        log(colors.red("\u2717 FAIL:"), colors.magenta(file.relative), "is", err.message);
+      }
     } else {
-      options.silent ||
-        log(
-          colors.red("\u2717 ERROR:"),
-          colors.magenta(file.relative) + ":",
-          err.message || err
-        );
+      if (!options.silent) {
+        log(colors.red("\u2717 ERROR:"), colors.magenta(file.relative) + ":", err.message || err);
+      }
     }
   }
 
@@ -142,13 +132,13 @@ function expect(options, expectation) {
         "Missing",
         colors.cyan(rules.length),
         "expected files:",
-        colors.magenta(missings)
+        colors.magenta(missings),
       );
     }
   }
 
   function reportSummary() {
-    options.silent ||
+    if (!options.silent) {
       log(
         "Tested",
         colors.cyan(numTests),
@@ -157,8 +147,9 @@ function expect(options, expectation) {
         "passes,",
         colors.cyan(numFailures),
         "failures:",
-        numFailures > 0 ? colors.bgRed.white("FAIL") : colors.green("PASS")
+        numFailures > 0 ? colors.bgRed.white("FAIL") : colors.green("PASS"),
       );
+    }
   }
 
   return through.obj(eachFile, endStream);

package/lib/file-tester.js

@@ -27,10 +27,7 @@ function parseExpectation(expectation) {
       return [new Rule(null, expectation)];
     case "object":
       return Object.keys(expectation).map(function (path) {
-        return new Rule(
-          path,
-          expectation[path] === true ? null : expectation[path]
-        );
+        return new Rule(path, expectation[path] === true ? null : expectation[path]);
       });
     default:
       throw new TypeError("Unknown expectation type");
@@ -72,7 +69,7 @@ FileTester.prototype.test = function (file, callback) {
       } else {
         callback();
       }
-    }
+    },
   );
 };
 
@@ -119,8 +116,7 @@ Rule.wrapTester = function (tester) {
     return wrapAssertion(tester);
   }
 
-  var stringTester =
-    tester instanceof StringTester ? tester : new StringTester(tester);
+  var stringTester = tester instanceof StringTester ? tester : new StringTester(tester);
   var streamTester = new StreamTester(stringTester);
   return function (file, callback) {
     if (file.isNull()) {

package/lib/stream-tester.js

@@ -19,11 +19,15 @@ StreamTester.prototype.test = function (stream, callback) {
   return stream
     .pipe(spyStream)
     .on("error", function (err) {
-      yielded || callback(err);
+      if (!yielded) {
+        callback(err);
+      }
       yielded = true;
     })
     .on("end", function () {
-      yielded || callback(null);
+      if (!yielded) {
+        callback(null);
+      }
       yielded = true;
     });
 };
@@ -43,6 +47,6 @@ StreamTester.prototype.createStream = function () {
         _this.emit("error", err);
         cb();
       });
-    }
+    },
   );
 };

package/lib/string-tester.js

@@ -26,9 +26,12 @@ StringTester.prototype.parseExpectation = function (expectation) {
   }
 
   if (typeof expectation === "string") {
-    return wrapAssertion(function (target) {
-      return target.indexOf(expectation) >= 0;
-    }, "not containing " + JSON.stringify(expectation));
+    return wrapAssertion(
+      function (target) {
+        return target.indexOf(expectation) >= 0;
+      },
+      "not containing " + JSON.stringify(expectation),
+    );
   }
   if (expectation instanceof RegExp) {
     return wrapAssertion(function (target) {

package/package.json

@@ -1,40 +1,46 @@
 {
   "name": "gulp-expect-file",
-  "version": "2.0.0",
-  "author": "Kota Saito <kotas.nico@gmail.com>",
-  "copyright": "2014 Kota Saito",
+  "version": "3.0.0",
   "description": "Expect files in pipes for gulp.js",
-  "license": "MIT",
+  "keywords": [
+    "expect",
+    "gulpplugin",
+    "test"
+  ],
   "homepage": "https://github.com/pioug/gulp-expect-file/",
   "bugs": "https://github.com/pioug/gulp-expect-file/issues",
+  "license": "MIT",
+  "author": "Kota Saito <kotas.nico@gmail.com>",
   "repository": {
     "type": "git",
     "url": "git://github.com/pioug/gulp-expect-file.git"
   },
   "main": "index.js",
   "scripts": {
-    "test": "node_modules/.bin/mocha"
+    "test": "node_modules/.bin/mocha",
+    "lint": "oxlint --deny-warnings . --ignore-path .oxignore",
+    "format": "oxfmt --write \"**/*.{js,json,yml,yaml,md}\"",
+    "format:check": "oxfmt --check \"**/*.{js,json,yml,yaml,md}\""
   },
-  "keywords": [
-    "gulpplugin",
-    "test",
-    "expect"
-  ],
   "dependencies": {
     "ansi-colors": "^4.1.1",
     "async": "^3.2.0",
-    "fancy-log": "^1.3.3",
-    "minimatch": "^3.0.4",
-    "plugin-error": "^1.0.1",
+    "fancy-log": "^2.0.0",
+    "minimatch": "^10.2.2",
+    "plugin-error": "^2.0.1",
     "through2": "^4.0.2",
-    "vinyl": "^2.2.1"
+    "vinyl": "^3.0.1"
   },
   "devDependencies": {
-    "eslint": "^7.27.0",
-    "mocha": "^8.4.0",
+    "mocha": "^12.0.0-beta-9",
     "mock-require": "^3.0.3",
-    "prettier": "^2.3.0",
+    "oxfmt": "^0.34.0",
+    "oxlint": "^1.49.0",
     "should": "^13.2.3",
     "temp": "^0.9.4"
-  }
+  },
+  "engines": {
+    "node": ">=24.0.0"
+  },
+  "copyright": "2014 Kota Saito"
 }

package/.eslintrc

@@ -1,7 +0,0 @@
-{
-  "extends": "eslint:recommended",
-  "env": {
-    "node": true,
-    "mocha": true
-  }
-}