chai 5.1.2 → 5.2.0

package/lib/chai/utils/overwriteProperty.js

@@ -42,49 +42,48 @@ import {transferFlags} from './transferFlags.js';
  * @public
  */
 export function overwriteProperty(ctx, name, getter) {
-  var _get = Object.getOwnPropertyDescriptor(ctx, name)
-    , _super = function () {};
+  var _get = Object.getOwnPropertyDescriptor(ctx, name),
+    _super = function () {};
 
-  if (_get && 'function' === typeof _get.get)
-    _super = _get.get
+  if (_get && 'function' === typeof _get.get) _super = _get.get;
 
-  Object.defineProperty(ctx, name,
-    { get: function overwritingPropertyGetter() {
-        // Setting the `ssfi` flag to `overwritingPropertyGetter` causes this
-        // function to be the starting point for removing implementation frames
-        // from the stack trace of a failed assertion.
-        //
-        // However, we only want to use this function as the starting point if
-        // the `lockSsfi` flag isn't set and proxy protection is disabled.
-        //
-        // If the `lockSsfi` flag is set, then either this assertion has been
-        // overwritten by another assertion, or this assertion is being invoked
-        // from inside of another assertion. In the first case, the `ssfi` flag
-        // has already been set by the overwriting assertion. In the second
-        // case, the `ssfi` flag has already been set by the outer assertion.
-        //
-        // If proxy protection is enabled, then the `ssfi` flag has already been
-        // set by the proxy getter.
-        if (!isProxyEnabled() && !flag(this, 'lockSsfi')) {
-          flag(this, 'ssfi', overwritingPropertyGetter);
-        }
-
-        // Setting the `lockSsfi` flag to `true` prevents the overwritten
-        // assertion from changing the `ssfi` flag. By this point, the `ssfi`
-        // flag is already set to the correct starting point for this assertion.
-        var origLockSsfi = flag(this, 'lockSsfi');
-        flag(this, 'lockSsfi', true);
-        var result = getter(_super).call(this);
-        flag(this, 'lockSsfi', origLockSsfi);
+  Object.defineProperty(ctx, name, {
+    get: function overwritingPropertyGetter() {
+      // Setting the `ssfi` flag to `overwritingPropertyGetter` causes this
+      // function to be the starting point for removing implementation frames
+      // from the stack trace of a failed assertion.
+      //
+      // However, we only want to use this function as the starting point if
+      // the `lockSsfi` flag isn't set and proxy protection is disabled.
+      //
+      // If the `lockSsfi` flag is set, then either this assertion has been
+      // overwritten by another assertion, or this assertion is being invoked
+      // from inside of another assertion. In the first case, the `ssfi` flag
+      // has already been set by the overwriting assertion. In the second
+      // case, the `ssfi` flag has already been set by the outer assertion.
+      //
+      // If proxy protection is enabled, then the `ssfi` flag has already been
+      // set by the proxy getter.
+      if (!isProxyEnabled() && !flag(this, 'lockSsfi')) {
+        flag(this, 'ssfi', overwritingPropertyGetter);
+      }
 
-        if (result !== undefined) {
-          return result;
-        }
+      // Setting the `lockSsfi` flag to `true` prevents the overwritten
+      // assertion from changing the `ssfi` flag. By this point, the `ssfi`
+      // flag is already set to the correct starting point for this assertion.
+      var origLockSsfi = flag(this, 'lockSsfi');
+      flag(this, 'lockSsfi', true);
+      var result = getter(_super).call(this);
+      flag(this, 'lockSsfi', origLockSsfi);
 
-        var newAssertion = new Assertion();
-        transferFlags(this, newAssertion);
-        return newAssertion;
+      if (result !== undefined) {
+        return result;
       }
-    , configurable: true
+
+      var newAssertion = new Assertion();
+      transferFlags(this, newAssertion);
+      return newAssertion;
+    },
+    configurable: true
   });
 }

package/lib/chai/utils/proxify.js

@@ -30,7 +30,7 @@ const builtins = ['__flags', '__methods', '_obj', 'assert'];
  * @namespace Utils
  * @name proxify
  */
-export function proxify(obj ,nonChainableMethodName) {
+export function proxify(obj, nonChainableMethodName) {
   if (!isProxyEnabled()) return obj;
 
   return new Proxy(obj, {
@@ -39,14 +39,22 @@ export function proxify(obj ,nonChainableMethodName) {
       // such as `Symbol.toStringTag`.
       // The values for which an error should be thrown can be configured using
       // the `config.proxyExcludedKeys` setting.
-      if (typeof property === 'string' &&
-          config.proxyExcludedKeys.indexOf(property) === -1 &&
-          !Reflect.has(target, property)) {
+      if (
+        typeof property === 'string' &&
+        config.proxyExcludedKeys.indexOf(property) === -1 &&
+        !Reflect.has(target, property)
+      ) {
         // Special message for invalid property access of non-chainable methods.
         if (nonChainableMethodName) {
-          throw Error('Invalid Chai property: ' + nonChainableMethodName + '.' +
-            property + '. See docs for proper usage of "' +
-            nonChainableMethodName + '".');
+          throw Error(
+            'Invalid Chai property: ' +
+              nonChainableMethodName +
+              '.' +
+              property +
+              '. See docs for proper usage of "' +
+              nonChainableMethodName +
+              '".'
+          );
         }
 
         // If the property is reasonably close to an existing Chai property,
@@ -54,16 +62,14 @@ export function proxify(obj ,nonChainableMethodName) {
         // distance less than 4.
         var suggestion = null;
         var suggestionDistance = 4;
-        getProperties(target).forEach(function(prop) {
+        getProperties(target).forEach(function (prop) {
           if (
+            // we actually mean to check `Object.prototype` here
+            // eslint-disable-next-line no-prototype-builtins
             !Object.prototype.hasOwnProperty(prop) &&
             builtins.indexOf(prop) === -1
           ) {
-            var dist = stringDistanceCapped(
-              property,
-              prop,
-              suggestionDistance
-            );
+            var dist = stringDistanceCapped(property, prop, suggestionDistance);
             if (dist < suggestionDistance) {
               suggestion = prop;
               suggestionDistance = dist;
@@ -72,8 +78,13 @@ export function proxify(obj ,nonChainableMethodName) {
         });
 
         if (suggestion !== null) {
-          throw Error('Invalid Chai property: ' + property +
-            '. Did you mean "' + suggestion + '"?');
+          throw Error(
+            'Invalid Chai property: ' +
+              property +
+              '. Did you mean "' +
+              suggestion +
+              '"?'
+          );
         } else {
           throw Error('Invalid Chai property: ' + property);
         }
@@ -119,17 +130,17 @@ function stringDistanceCapped(strA, strB, cap) {
   // `memo` is a two-dimensional array containing distances.
   // memo[i][j] is the distance between strA.slice(0, i) and
   // strB.slice(0, j).
-  for (var i = 0; i <= strA.length; i++) {
+  for (let i = 0; i <= strA.length; i++) {
     memo[i] = Array(strB.length + 1).fill(0);
     memo[i][0] = i;
   }
-  for (var j = 0; j < strB.length; j++) {
+  for (let j = 0; j < strB.length; j++) {
     memo[0][j] = j;
   }
 
-  for (var i = 1; i <= strA.length; i++) {
+  for (let i = 1; i <= strA.length; i++) {
     var ch = strA.charCodeAt(i - 1);
-    for (var j = 1; j <= strB.length; j++) {
+    for (let j = 1; j <= strB.length; j++) {
       if (Math.abs(i - j) >= cap) {
         memo[i][j] = cap;
         continue;
@@ -137,8 +148,7 @@ function stringDistanceCapped(strA, strB, cap) {
       memo[i][j] = Math.min(
         memo[i - 1][j] + 1,
         memo[i][j - 1] + 1,
-        memo[i - 1][j - 1] +
-          (ch === strB.charCodeAt(j - 1) ? 0 : 1)
+        memo[i - 1][j - 1] + (ch === strB.charCodeAt(j - 1) ? 0 : 1)
       );
     }
   }

package/lib/chai/utils/test.js

@@ -18,7 +18,7 @@ import {flag} from './flag.js';
  * @name test
  */
 export function test(obj, args) {
-  var negate = flag(obj, 'negate')
-    , expr = args[0];
+  var negate = flag(obj, 'negate'),
+    expr = args[0];
   return negate ? !expr : expr;
 }

package/lib/chai/utils/transferFlags.js

@@ -35,8 +35,13 @@ export function transferFlags(assertion, object, includeAll) {
   includeAll = arguments.length === 3 ? includeAll : true;
 
   for (var flag in flags) {
-    if (includeAll ||
-        (flag !== 'object' && flag !== 'ssfi' && flag !== 'lockSsfi' && flag != 'message')) {
+    if (
+      includeAll ||
+      (flag !== 'object' &&
+        flag !== 'ssfi' &&
+        flag !== 'lockSsfi' &&
+        flag != 'message')
+    ) {
       object.__flags[flag] = flags[flag];
     }
   }

package/lib/chai/utils/type-detect.js

@@ -6,7 +6,7 @@ export function type(obj) {
   if (typeof obj === 'undefined') {
     return 'undefined';
   }
-  
+
   if (obj === null) {
     return 'null';
   }

package/lib/chai.js

@@ -29,6 +29,7 @@ export {AssertionError};
  */
 export function use(fn) {
   const exports = {
+    use,
     AssertionError,
     util,
     config,
@@ -44,7 +45,7 @@ export function use(fn) {
   }
 
   return exports;
-};
+}
 
 // Utility Functions
 export {util};

package/package.json

@@ -18,7 +18,7 @@
     "Veselin Todorov <hi@vesln.com>",
     "John Firebaugh <john.firebaugh@gmail.com>"
   ],
-  "version": "5.1.2",
+  "version": "5.2.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/chaijs/chai"
@@ -31,12 +31,15 @@
     "prebuild": "npm run clean",
     "build": "npm run build:esm",
     "build:esm": "esbuild --bundle --format=esm --keep-names --outfile=chai.js index.js",
+    "format": "prettier --write lib",
     "pretest": "npm run lint && npm run build",
     "test": "npm run test-node && npm run test-chrome",
-    "test-node": "mocha --require ./test/bootstrap/index.js --reporter dot test/*.js",
+    "test-node": "c8 --99 --check-coverage mocha --require ./test/bootstrap/index.js test/*.js",
     "test-chrome": "web-test-runner --playwright",
-    "lint": "eslint lib/",
-    "clean": "rm -f chai.js coverage"
+    "lint": "npm run lint:js && npm run lint:format",
+    "lint:js": "eslint lib/",
+    "lint:format": "prettier --check lib",
+    "clean": "rm -rf chai.js coverage/"
   },
   "engines": {
     "node": ">=12"
@@ -49,13 +52,16 @@
     "pathval": "^2.0.0"
   },
   "devDependencies": {
+    "@eslint/js": "^9.17.0",
     "@rollup/plugin-commonjs": "^25.0.7",
     "@web/dev-server-rollup": "^0.6.1",
     "@web/test-runner": "^0.18.0",
     "@web/test-runner-playwright": "^0.11.0",
+    "c8": "^10.1.3",
     "esbuild": "^0.19.10",
     "eslint": "^8.56.0",
     "eslint-plugin-jsdoc": "^48.0.4",
-    "mocha": "^10.2.0"
+    "mocha": "^10.2.0",
+    "prettier": "^3.4.2"
   }
 }

package/lib/chai/utils/getEnumerableProperties.js

@@ -1,25 +0,0 @@
-/*!
- * Chai - getEnumerableProperties utility
- * Copyright(c) 2012-2014 Jake Luer <jake@alogicalparadox.com>
- * MIT Licensed
- */
-
-/**
- * ### .getEnumerableProperties(object)
- *
- * This allows the retrieval of enumerable property names of an object,
- * inherited or not.
- *
- * @param {object} object
- * @returns {Array}
- * @namespace Utils
- * @name getEnumerableProperties
- * @public
- */
-module.exports = function getEnumerableProperties(object) {
-  var result = [];
-  for (var name in object) {
-    result.push(name);
-  }
-  return result;
-};

package/register-assert.cjs

@@ -1,3 +0,0 @@
-const {assert} = require('./chai.cjs');
-
-globalThis.assert = assert;

package/register-expect.cjs

@@ -1,3 +0,0 @@
-const {expect} = require('./chai.cjs');
-
-globalThis.expect = expect;

package/register-should.cjs

@@ -1,3 +0,0 @@
-const {should} = require('./chai.cjs');
-
-globalThis.should = should();