vest 4.3.1 → 4.3.2-dev-2805e3

package/dist/cjs/classnames.development.js

@@ -4,13 +4,6 @@ function isFunction(value) {
     return typeof value === 'function';
 }
 
-/**
- * A safe hasOwnProperty access
- */
-function hasOwnProperty(obj, key) {
-    return Object.prototype.hasOwnProperty.call(obj, key);
-}
-
 function optionalFunctionValue(value) {
     var args = [];
     for (var _i = 1; _i < arguments.length; _i++) {
@@ -19,6 +12,18 @@ function optionalFunctionValue(value) {
     return isFunction(value) ? value.apply(void 0, args) : value;
 }
 
+function defaultTo(callback, defaultValue) {
+    var _a;
+    return (_a = optionalFunctionValue(callback)) !== null && _a !== void 0 ? _a : defaultValue;
+}
+
+/**
+ * A safe hasOwnProperty access
+ */
+function hasOwnProperty(obj, key) {
+    return Object.prototype.hasOwnProperty.call(obj, key);
+}
+
 function invariant(condition, 
 // eslint-disable-next-line @typescript-eslint/ban-types
 message) {
@@ -86,22 +91,20 @@ function parse(res) {
         return res.testCount === 0 || !selectors.tested(fieldName);
     }
     function isValid(fieldName) {
-        var _a, _b;
-        return Boolean(fieldName ? (_b = (_a = res.tests) === null || _a === void 0 ? void 0 : _a[fieldName]) === null || _b === void 0 ? void 0 : _b.valid : res.valid);
+        var _a;
+        return fieldName ? Boolean((_a = res.tests[fieldName]) === null || _a === void 0 ? void 0 : _a.valid) : res.valid;
     }
     function hasWarnings(fieldName) {
-        return hasFailures(res, SeverityCount.WARN_COUNT, fieldName);
+        return hasFailures(SeverityCount.WARN_COUNT, fieldName);
     }
     function hasErrors(fieldName) {
-        return hasFailures(res, SeverityCount.ERROR_COUNT, fieldName);
+        return hasFailures(SeverityCount.ERROR_COUNT, fieldName);
+    }
+    function hasFailures(countKey, fieldName) {
+        var _a;
+        var failureCount = defaultTo(fieldName ? (_a = res.tests[fieldName]) === null || _a === void 0 ? void 0 : _a[countKey] : res[countKey], 0);
+        return isPositive(failureCount);
     }
-}
-function hasFailures(res, countKey, fieldName) {
-    var _a, _b, _c;
-    var failureCount = fieldName
-        ? (_b = (_a = res.tests) === null || _a === void 0 ? void 0 : _a[fieldName]) === null || _b === void 0 ? void 0 : _b[countKey]
-        : (_c = res[countKey]) !== null && _c !== void 0 ? _c : 0;
-    return isPositive(failureCount);
 }
 
 /**

package/dist/cjs/classnames.production.js

@@ -1 +1 @@
-"use strict";function t(t){return"function"==typeof t}function n(t,n){return Object.prototype.hasOwnProperty.call(t,n)}function r(n,r){if(!n)throw r instanceof String?r.valueOf():Error(r?function(n){for(var r=[],e=1;e<arguments.length;e++)r[e-1]=arguments[e];return t(n)?n.apply(void 0,r):n}(r):r)}function e(t){var n=Number(t);return!(isNaN(parseFloat(String(t)))||isNaN(Number(t))||!isFinite(n))}function u(t){return function(t,n){return e(t)&&e(n)&&Number(t)>Number(n)}(t,0)}var o,i,s;function a(t,n,r){var e,o,i;return u(t=r?null===(o=null===(e=t.tests)||void 0===e?void 0:e[r])||void 0===o?void 0:o[n]:null!==(i=t[n])&&void 0!==i?i:0)}(i=o||(o={})).WARNINGS="warnings",i.ERRORS="errors",function(t){t.ERROR_COUNT="errorCount",t.WARN_COUNT="warnCount"}(s||(s={})),module.exports=function(e,o){void 0===o&&(o={});var i=function(t){r(t&&n(t,"valid"),"Vest parser: expected argument at position 0 to be Vest's result object.");var e={},o={invalid:function(n){return a(t,s.ERROR_COUNT,n)},tested:function(r){return r?n(e,r)?e[r]:(e[r]=n(t.tests,r)&&u(t.tests[r].testCount),o.tested(r)):u(t.testCount)},untested:function(n){return 0===t.testCount||!o.tested(n)},valid:function(n){var r,e;return!!(n?null!==(e=null===(r=t.tests)||void 0===r?void 0:r[n])&&void 0!==e&&e.valid:t.valid)},warning:function(n){return a(t,s.WARN_COUNT,n)}};return o}(e);return function(n){var r,e=[];for(r in o){var u=r;t(i[u])&&i[u](n)&&e.push(o[u])}return e.join(" ")}};
+"use strict";function t(t){return"function"==typeof t}function n(n){for(var r=[],e=1;e<arguments.length;e++)r[e-1]=arguments[e];return t(n)?n.apply(void 0,r):n}function r(t,n){return Object.prototype.hasOwnProperty.call(t,n)}function e(t){var n=Number(t);return!(isNaN(parseFloat(String(t)))||isNaN(Number(t))||!isFinite(n))}function u(t){return function(t,n){return e(t)&&e(n)&&Number(t)>Number(n)}(t,0)}var o,i,s;function a(t){function e(r,e){var o;return u(r=function(t,r){var e;return null!==(e=n(t))&&void 0!==e?e:r}(e?null===(o=t.tests[e])||void 0===o?void 0:o[r]:t[r],0))}!function(t,r){if(!t)throw r instanceof String?r.valueOf():Error(r?n(r):r)}(t&&r(t,"valid"),"Vest parser: expected argument at position 0 to be Vest's result object.");var o={},i={invalid:function(t){return e(s.ERROR_COUNT,t)},tested:function(n){return n?r(o,n)?o[n]:(o[n]=r(t.tests,n)&&u(t.tests[n].testCount),i.tested(n)):u(t.testCount)},untested:function(n){return 0===t.testCount||!i.tested(n)},valid:function(n){var r;return n?!(null===(r=t.tests[n])||void 0===r||!r.valid):t.valid},warning:function(t){return e(s.WARN_COUNT,t)}};return i}(i=o||(o={})).WARNINGS="warnings",i.ERRORS="errors",function(t){t.ERROR_COUNT="errorCount",t.WARN_COUNT="warnCount"}(s||(s={})),module.exports=function(n,r){void 0===r&&(r={});var e=a(n);return function(n){var u,o=[];for(u in r){var i=u;t(e[i])&&e[i](n)&&o.push(r[i])}return o.join(" ")}};

package/dist/cjs/enforce/compounds.development.js

@@ -133,13 +133,13 @@ function oneOf(value) {
     }
     var passing = [];
     rules.some(function (rule) {
-        if (longerThan(passing, REQUIRED_COUNT)) {
-            return false;
-        }
         var res = runLazyRule(rule, value);
         if (res.pass) {
             passing.push(res);
         }
+        if (longerThan(passing, REQUIRED_COUNT)) {
+            return false;
+        }
     });
     return ruleReturn(lengthEquals(passing, REQUIRED_COUNT));
 }

package/dist/cjs/enforce/compounds.production.js

@@ -1 +1 @@
-"use strict";var n=require("n4s");function r(n,r){function t(n){u=!0,e=n}for(var u=!1,e=null,f=0;f<n.length;f++)if(r(n[f],t,f),u)return e}function t(n){return"function"==typeof n}function u(n,r){var u;return null!==(u=function(n){for(var r=[],u=1;u<arguments.length;u++)r[u-1]=arguments[u];return t(n)?n.apply(void 0,r):n}(n))&&void 0!==u?u:r}function e(n,r){return n={pass:n},r&&(n.message=r),n}function f(){return e(!1)}function o(n){return u(n,f())}function i(n){return u(n,e(!0))}function c(n,r){try{return n.run(r)}catch(n){return f()}}function a(n){var r=Number(n);return!(isNaN(parseFloat(String(n)))||isNaN(Number(n))||!isFinite(r))}function s(n,r){return function(n,r){return a(n)&&a(r)&&Number(n)===Number(r)}(n.length,r)}function l(n,r){return function(n,r){return a(n)&&a(r)&&Number(n)>Number(r)}(n.length,r)}n.enforce.extend({allOf:function(n){for(var t=[],u=1;u<arguments.length;u++)t[u-1]=arguments[u];return i(r(t,(function(r,t){(r=c(r,n)).pass||t(r)})))},anyOf:function(n){for(var t=[],u=1;u<arguments.length;u++)t[u-1]=arguments[u];return o(r(t,(function(r,t){(r=c(r,n)).pass&&t(r)})))},noneOf:function(n){for(var t=[],u=1;u<arguments.length;u++)t[u-1]=arguments[u];return i(r(t,(function(r,t){c(r,n).pass&&t(f())})))},oneOf:function(n){for(var r=[],t=1;t<arguments.length;t++)r[t-1]=arguments[t];var u=[];return r.some((function(r){if(l(u,1))return!1;(r=c(r,n)).pass&&u.push(r)})),e(s(u,1))}});
+"use strict";var n=require("n4s");function r(n,r){function t(n){u=!0,e=n}for(var u=!1,e=null,f=0;f<n.length;f++)if(r(n[f],t,f),u)return e}function t(n){return"function"==typeof n}function u(n,r){var u;return null!==(u=function(n){for(var r=[],u=1;u<arguments.length;u++)r[u-1]=arguments[u];return t(n)?n.apply(void 0,r):n}(n))&&void 0!==u?u:r}function e(n,r){return n={pass:n},r&&(n.message=r),n}function f(){return e(!1)}function o(n){return u(n,f())}function i(n){return u(n,e(!0))}function c(n,r){try{return n.run(r)}catch(n){return f()}}function a(n){var r=Number(n);return!(isNaN(parseFloat(String(n)))||isNaN(Number(n))||!isFinite(r))}function s(n,r){return function(n,r){return a(n)&&a(r)&&Number(n)===Number(r)}(n.length,r)}function l(n,r){return function(n,r){return a(n)&&a(r)&&Number(n)>Number(r)}(n.length,r)}n.enforce.extend({allOf:function(n){for(var t=[],u=1;u<arguments.length;u++)t[u-1]=arguments[u];return i(r(t,(function(r,t){(r=c(r,n)).pass||t(r)})))},anyOf:function(n){for(var t=[],u=1;u<arguments.length;u++)t[u-1]=arguments[u];return o(r(t,(function(r,t){(r=c(r,n)).pass&&t(r)})))},noneOf:function(n){for(var t=[],u=1;u<arguments.length;u++)t[u-1]=arguments[u];return i(r(t,(function(r,t){c(r,n).pass&&t(f())})))},oneOf:function(n){for(var r=[],t=1;t<arguments.length;t++)r[t-1]=arguments[t];var u=[];return r.some((function(r){if((r=c(r,n)).pass&&u.push(r),l(u,1))return!1})),e(s(u,1))}});

package/dist/cjs/parser.development.js

@@ -2,13 +2,6 @@
 
 Object.defineProperty(exports, '__esModule', { value: true });
 
-/**
- * A safe hasOwnProperty access
- */
-function hasOwnProperty(obj, key) {
-    return Object.prototype.hasOwnProperty.call(obj, key);
-}
-
 function isFunction(value) {
     return typeof value === 'function';
 }
@@ -21,6 +14,18 @@ function optionalFunctionValue(value) {
     return isFunction(value) ? value.apply(void 0, args) : value;
 }
 
+function defaultTo(callback, defaultValue) {
+    var _a;
+    return (_a = optionalFunctionValue(callback)) !== null && _a !== void 0 ? _a : defaultValue;
+}
+
+/**
+ * A safe hasOwnProperty access
+ */
+function hasOwnProperty(obj, key) {
+    return Object.prototype.hasOwnProperty.call(obj, key);
+}
+
 function invariant(condition, 
 // eslint-disable-next-line @typescript-eslint/ban-types
 message) {
@@ -88,22 +93,20 @@ function parse(res) {
         return res.testCount === 0 || !selectors.tested(fieldName);
     }
     function isValid(fieldName) {
-        var _a, _b;
-        return Boolean(fieldName ? (_b = (_a = res.tests) === null || _a === void 0 ? void 0 : _a[fieldName]) === null || _b === void 0 ? void 0 : _b.valid : res.valid);
+        var _a;
+        return fieldName ? Boolean((_a = res.tests[fieldName]) === null || _a === void 0 ? void 0 : _a.valid) : res.valid;
     }
     function hasWarnings(fieldName) {
-        return hasFailures(res, SeverityCount.WARN_COUNT, fieldName);
+        return hasFailures(SeverityCount.WARN_COUNT, fieldName);
     }
     function hasErrors(fieldName) {
-        return hasFailures(res, SeverityCount.ERROR_COUNT, fieldName);
+        return hasFailures(SeverityCount.ERROR_COUNT, fieldName);
+    }
+    function hasFailures(countKey, fieldName) {
+        var _a;
+        var failureCount = defaultTo(fieldName ? (_a = res.tests[fieldName]) === null || _a === void 0 ? void 0 : _a[countKey] : res[countKey], 0);
+        return isPositive(failureCount);
     }
-}
-function hasFailures(res, countKey, fieldName) {
-    var _a, _b, _c;
-    var failureCount = fieldName
-        ? (_b = (_a = res.tests) === null || _a === void 0 ? void 0 : _a[fieldName]) === null || _b === void 0 ? void 0 : _b[countKey]
-        : (_c = res[countKey]) !== null && _c !== void 0 ? _c : 0;
-    return isPositive(failureCount);
 }
 
 exports.parse = parse;

package/dist/cjs/parser.production.js

@@ -1 +1 @@
-"use strict";function t(t,n){return Object.prototype.hasOwnProperty.call(t,n)}function n(t){return"function"==typeof t}function r(t,r){if(!t)throw r instanceof String?r.valueOf():Error(r?function(t){for(var r=[],e=1;e<arguments.length;e++)r[e-1]=arguments[e];return n(t)?t.apply(void 0,r):t}(r):r)}function e(t){var n=Number(t);return!(isNaN(parseFloat(String(t)))||isNaN(Number(t))||!isFinite(n))}function u(t){return function(t,n){return e(t)&&e(n)&&Number(t)>Number(n)}(t,0)}var o,i,s;function a(t,n,r){var e,o,i;return u(t=r?null===(o=null===(e=t.tests)||void 0===e?void 0:e[r])||void 0===o?void 0:o[n]:null!==(i=t[n])&&void 0!==i?i:0)}Object.defineProperty(exports,"__esModule",{value:!0}),(i=o||(o={})).WARNINGS="warnings",i.ERRORS="errors",function(t){t.ERROR_COUNT="errorCount",t.WARN_COUNT="warnCount"}(s||(s={})),exports.parse=function(n){r(n&&t(n,"valid"),"Vest parser: expected argument at position 0 to be Vest's result object.");var e={},o={invalid:function(t){return a(n,s.ERROR_COUNT,t)},tested:function(r){return r?t(e,r)?e[r]:(e[r]=t(n.tests,r)&&u(n.tests[r].testCount),o.tested(r)):u(n.testCount)},untested:function(t){return 0===n.testCount||!o.tested(t)},valid:function(t){var r,e;return!!(t?null!==(e=null===(r=n.tests)||void 0===r?void 0:r[t])&&void 0!==e&&e.valid:n.valid)},warning:function(t){return a(n,s.WARN_COUNT,t)}};return o};
+"use strict";function t(t){return"function"==typeof t}function n(n){for(var r=[],e=1;e<arguments.length;e++)r[e-1]=arguments[e];return t(n)?n.apply(void 0,r):n}function r(t,n){return Object.prototype.hasOwnProperty.call(t,n)}function e(t){var n=Number(t);return!(isNaN(parseFloat(String(t)))||isNaN(Number(t))||!isFinite(n))}function u(t){return function(t,n){return e(t)&&e(n)&&Number(t)>Number(n)}(t,0)}var o,i,s;Object.defineProperty(exports,"__esModule",{value:!0}),(i=o||(o={})).WARNINGS="warnings",i.ERRORS="errors",function(t){t.ERROR_COUNT="errorCount",t.WARN_COUNT="warnCount"}(s||(s={})),exports.parse=function(t){function e(r,e){var o;return u(r=function(t,r){var e;return null!==(e=n(t))&&void 0!==e?e:r}(e?null===(o=t.tests[e])||void 0===o?void 0:o[r]:t[r],0))}!function(t,r){if(!t)throw r instanceof String?r.valueOf():Error(r?n(r):r)}(t&&r(t,"valid"),"Vest parser: expected argument at position 0 to be Vest's result object.");var o={},i={invalid:function(t){return e(s.ERROR_COUNT,t)},tested:function(n){return n?r(o,n)?o[n]:(o[n]=r(t.tests,n)&&u(t.tests[n].testCount),i.tested(n)):u(t.testCount)},untested:function(n){return 0===t.testCount||!i.tested(n)},valid:function(n){var r;return n?!(null===(r=t.tests[n])||void 0===r||!r.valid):t.valid},warning:function(t){return e(s.WARN_COUNT,t)}};return i};

package/dist/cjs/vest.development.js

@@ -300,18 +300,6 @@ function useAllIncomplete() {
         return testObject.isPending() ? testObject : null;
     }));
 }
-function useOmittedFields() {
-    var testObjects = useTestsFlat();
-    return testObjects.reduce(function (omittedFields, testObject) {
-        if (omittedFields[testObject.fieldName]) {
-            return omittedFields;
-        }
-        if (testObject.isOmitted()) {
-            omittedFields[testObject.fieldName] = true;
-        }
-        return omittedFields;
-    }, {});
-}
 var flatCache = createCache();
 function useTestsFlat() {
     var current = useTestObjects()[0].current;
@@ -624,9 +612,7 @@ function useRetainTestKey(key, testObject) {
 
 function isolate(_a, callback) {
     var _b = _a.type, type = _b === void 0 ? IsolateTypes.DEFAULT : _b;
-    if (!isFunction(callback)) {
-        return;
-    }
+    invariant(isFunction(callback));
     var keys = {
         current: {},
         prev: {}
@@ -656,6 +642,11 @@ var SeverityCount;
     SeverityCount["ERROR_COUNT"] = "errorCount";
     SeverityCount["WARN_COUNT"] = "warnCount";
 })(SeverityCount || (SeverityCount = {}));
+function countKeyBySeverity(severity) {
+    return severity === Severity.ERRORS
+        ? SeverityCount.ERROR_COUNT
+        : SeverityCount.WARN_COUNT;
+}
 
 /**
  * A safe hasOwnProperty access
@@ -681,7 +672,7 @@ function isEmpty(value) {
     else if (typeof value === 'object') {
         return lengthEquals(Object.keys(value), 0);
     }
-    return true;
+    return false;
 }
 var isNotEmpty = bindNot(isEmpty);
 
@@ -703,10 +694,23 @@ function nonMatchingSeverityProfile(severity, testObject) {
     return either(severity === Severity.WARNINGS, testObject.warns());
 }
 
+/**
+ * The difference between this file and hasFailures is that hasFailures uses the static
+ * summary object, while this one uses the actual validation state
+ */
+function hasErrorsByTestObjects(fieldName) {
+    return hasFailuresByTestObjects(Severity.ERRORS, fieldName);
+}
+function hasFailuresByTestObjects(severityKey, fieldName) {
+    var testObjects = useTestsFlat();
+    return testObjects.some(function (testObject) {
+        return hasFailuresByTestObject(testObject, severityKey, fieldName);
+    });
+}
 /**
  * Determines whether a certain test profile has failures.
  */
-function hasFailuresLogic(testObject, severityKey, fieldName) {
+function hasFailuresByTestObject(testObject, severityKey, fieldName) {
     if (!testObject.hasFailures()) {
         return false;
     }
@@ -719,51 +723,29 @@ function hasFailuresLogic(testObject, severityKey, fieldName) {
     return true;
 }
 
-function hasErrors(fieldName) {
-    return has(Severity.ERRORS, fieldName);
-}
-function hasWarnings(fieldName) {
-    return has(Severity.WARNINGS, fieldName);
-}
-function has(severityKey, fieldName) {
-    var testObjects = useTestsFlat();
-    return testObjects.some(function (testObject) {
-        return hasFailuresLogic(testObject, severityKey, fieldName);
-    });
-}
-
-function isValid(fieldName) {
-    var _a, _b;
-    var summary = context.useX().summary;
-    invariant(summary);
-    return Boolean(fieldName ? (_b = (_a = summary.tests) === null || _a === void 0 ? void 0 : _a[fieldName]) === null || _b === void 0 ? void 0 : _b.valid : summary.valid);
-}
 // eslint-disable-next-line max-statements, complexity
 function shouldAddValidProp(fieldName) {
     if (fieldIsOmitted(fieldName)) {
         return true;
     }
-    if (hasErrors(fieldName)) {
+    if (hasErrorsByTestObjects(fieldName)) {
         return false;
     }
     var testObjects = useTestsFlat();
     if (isEmpty(testObjects)) {
         return false;
     }
-    if (fieldDoesNotExist(fieldName)) {
-        return false;
-    }
     if (hasNonOptionalIncomplete(fieldName)) {
         return false;
     }
     return noMissingTests(fieldName);
 }
 function fieldIsOmitted(fieldName) {
-    var omittedFields = useOmittedFields();
     if (!fieldName) {
         return false;
     }
-    return !!omittedFields[fieldName];
+    var flatTests = useTestsFlat();
+    return flatTests.some(function (testObject) { return testObject.fieldName === fieldName && testObject.isOmitted(); });
 }
 function hasNonOptionalIncomplete(fieldName) {
     var optionalFields = useOptionalFields()[0];
@@ -774,11 +756,6 @@ function hasNonOptionalIncomplete(fieldName) {
         return optionalFields[testObject.fieldName] !== true;
     }));
 }
-function fieldDoesNotExist(fieldName) {
-    var testObjects = useTestsFlat();
-    return (!!fieldName &&
-        !testObjects.find(function (testObject) { return testObject.fieldName === fieldName; }));
-}
 function noMissingTests(fieldName) {
     var testObjects = useTestsFlat();
     var optionalFields = useOptionalFields()[0];
@@ -792,6 +769,11 @@ function noMissingTests(fieldName) {
     });
 }
 
+function useSummary() {
+    var summary = context.useX().summary;
+    invariant(summary);
+    return summary;
+}
 /**
  * Reads the testObjects list and gets full validation result from it.
  */
@@ -840,13 +822,9 @@ function countFailures(summary) {
     }
     return summary;
 }
-/**
- * Appends the test to a results object
- */
-// eslint-disable-next-line max-statements
 function appendTestObject(summaryKey, testObject) {
     var fieldName = testObject.fieldName, message = testObject.message;
-    summaryKey[fieldName] = summaryKey[fieldName] || baseStats();
+    summaryKey[fieldName] = summaryKey[fieldName] || baseTestStats();
     var testKey = summaryKey[fieldName];
     if (testObject.isNonActionable())
         return testKey;
@@ -859,18 +837,13 @@ function appendTestObject(summaryKey, testObject) {
     }
     return testKey;
     function incrementFailures(severity) {
-        var countKey = getCountKey(severity);
+        var countKey = countKeyBySeverity(severity);
         testKey[countKey]++;
         if (message) {
             testKey[severity] = (testKey[severity] || []).concat(message);
         }
     }
 }
-function getCountKey(severity) {
-    return severity === Severity.ERRORS
-        ? SeverityCount.ERROR_COUNT
-        : SeverityCount.WARN_COUNT;
-}
 function baseStats() {
     return {
         errorCount: 0,
@@ -878,6 +851,12 @@ function baseStats() {
         testCount: 0
     };
 }
+function baseTestStats() {
+    return assign(baseStats(), {
+        errors: [],
+        warnings: []
+    });
+}
 
 // calls collectAll or getByFieldName depending on whether fieldName is provided
 function gatherFailures(testGroup, severityKey, fieldName) {
@@ -890,10 +869,11 @@ function getByFieldName(testGroup, severityKey, fieldName) {
     return ((_a = testGroup === null || testGroup === void 0 ? void 0 : testGroup[fieldName]) === null || _a === void 0 ? void 0 : _a[severityKey]) || [];
 }
 function collectAll(testGroup, severityKey) {
-    var _a;
     var output = {};
     for (var field in testGroup) {
-        output[field] = ((_a = testGroup[field]) === null || _a === void 0 ? void 0 : _a[severityKey]) || [];
+        // We will probably never get to the fallback array
+        // leaving it just in case the implementation changes
+        output[field] = testGroup[field][severityKey] || [];
     }
     return output;
 }
@@ -908,8 +888,7 @@ function getWarnings(fieldName) {
  * @returns suite or field's errors or warnings.
  */
 function getFailures(severityKey, fieldName) {
-    var summary = context.useX().summary;
-    invariant(summary);
+    var summary = useSummary();
     return gatherFailures(summary.tests, severityKey, fieldName);
 }
 
@@ -920,27 +899,59 @@ function getWarningsByGroup(groupName, fieldName) {
     return getFailuresByGroup(groupName, Severity.WARNINGS, fieldName);
 }
 function getFailuresByGroup(groupName, severityKey, fieldName) {
-    var summary = context.useX().summary;
-    invariant(summary);
+    var summary = useSummary();
     return gatherFailures(summary.groups[groupName], severityKey, fieldName);
 }
 
+function isPositive(value) {
+    return greaterThan(value, 0);
+}
+
+function hasErrors(fieldName) {
+    return hasFailures(SeverityCount.ERROR_COUNT, fieldName);
+}
+function hasWarnings(fieldName) {
+    return hasFailures(SeverityCount.WARN_COUNT, fieldName);
+}
+function hasFailures(severityCount, fieldName) {
+    var _a;
+    var summary = useSummary();
+    if (fieldName) {
+        return isPositive((_a = summary.tests[fieldName]) === null || _a === void 0 ? void 0 : _a[severityCount]);
+    }
+    return isPositive(summary[severityCount]);
+}
+
 function hasErrorsByGroup(groupName, fieldName) {
-    return hasByGroup(Severity.ERRORS, groupName, fieldName);
+    return hasFailuresByGroup(Severity.ERRORS, groupName, fieldName);
 }
 function hasWarningsByGroup(groupName, fieldName) {
-    return hasByGroup(Severity.WARNINGS, groupName, fieldName);
+    return hasFailuresByGroup(Severity.WARNINGS, groupName, fieldName);
 }
-/**
- * Checks whether there are failures in a given group.
- */
-function hasByGroup(severityKey, group, fieldName) {
-    var testObjects = useTestsFlat();
-    return testObjects.some(function (testObject) {
-        return group === testObject.groupName
-            ? hasFailuresLogic(testObject, severityKey, fieldName)
-            : false;
-    });
+// eslint-disable-next-line max-statements
+function hasFailuresByGroup(severityKey, groupName, fieldName) {
+    var _a, _b;
+    var summary = useSummary();
+    var severityCount = countKeyBySeverity(severityKey);
+    var group = summary.groups[groupName];
+    if (!group) {
+        return false;
+    }
+    if (fieldName) {
+        return isPositive((_a = group[fieldName]) === null || _a === void 0 ? void 0 : _a[severityCount]);
+    }
+    for (var field in group) {
+        if (isPositive((_b = group[field]) === null || _b === void 0 ? void 0 : _b[severityCount])) {
+            return true;
+        }
+    }
+    return false;
+}
+
+function isValid(fieldName) {
+    var _a;
+    var summary = useSummary();
+    return fieldName ? Boolean((_a = summary.tests[fieldName]) === null || _a === void 0 ? void 0 : _a.valid) : summary.valid;
 }
 
 var cache$1 = createCache(1);
@@ -950,7 +961,7 @@ function produceSuiteResult() {
     return cache$1([testObjects], context.bind(ctxRef, function () {
         var summary = genTestsSummary();
         var suiteName = useSuiteName();
-        var ref = assign({ summary: summary }, ctxRef);
+        var ref = { summary: summary };
         return assign(summary, {
             getErrors: context.bind(ref, getErrors),
             getErrorsByGroup: context.bind(ref, getErrorsByGroup),
@@ -1343,7 +1354,7 @@ function isExcluded(testObject) {
             return keyTests[fieldName] === false;
         }
     }
-    if (isMissingFromIncludedGroup(groupName)) {
+    if (isTopLevelWhenThereIsAnIncludedGroup(groupName)) {
         return true;
     }
     // if field is only'ed
@@ -1352,39 +1363,13 @@ function isExcluded(testObject) {
     // If there is _ANY_ `only`ed test (and we already know this one isn't) return true
     if (hasIncludedTests(keyTests)) {
         // Check if inclusion rules for this field (`include` hook)
+        // TODO: Check if this may need to be moved outside of the condition.
+        // What if there are no included tests? This shouldn't run then?
         return !optionalFunctionValue(inclusion[fieldName]);
     }
     // We're done here. This field is not excluded
     return false;
 }
-// eslint-disable-next-line max-statements
-function isMissingFromIncludedGroup(groupName) {
-    var context$1 = context.useX();
-    var exclusion = context$1.exclusion;
-    if (!hasIncludedGroups()) {
-        return false;
-    }
-    if (!groupName) {
-        return true;
-    }
-    if (groupName in exclusion.groups) {
-        if (exclusion.groups[groupName]) {
-            return false;
-        }
-        return true;
-    }
-    return true;
-}
-function hasIncludedGroups() {
-    var context$1 = context.useX();
-    var exclusion = context$1.exclusion;
-    for (var group in exclusion.groups) {
-        if (exclusion.groups[group]) {
-            return true;
-        }
-    }
-    return false;
-}
 /**
  * Checks whether a given group is excluded from running.
  */
@@ -1399,13 +1384,8 @@ function isGroupExcluded(groupName) {
         return keyGroups[groupName] === false;
     }
     // Group is not present
-    for (var group in keyGroups) {
-        // If any other group is only'ed
-        if (keyGroups[group] === true) {
-            return true;
-        }
-    }
-    return false;
+    // Return whether other groups are included
+    return hasIncludedGroups();
 }
 /**
  * Adds fields to a specified exclusion group.
@@ -1434,6 +1414,24 @@ function hasIncludedTests(keyTests) {
     }
     return false;
 }
+// are we not in a group and there is an included group?
+function isTopLevelWhenThereIsAnIncludedGroup(groupName) {
+    if (!hasIncludedGroups()) {
+        return false;
+    }
+    // Return whether there's an included group, and we're not inside a group
+    return !groupName;
+}
+function hasIncludedGroups() {
+    var context$1 = context.useX();
+    var exclusion = context$1.exclusion;
+    for (var group in exclusion.groups) {
+        if (exclusion.groups[group]) {
+            return true;
+        }
+    }
+    return false;
+}
 
 /**
  * Runs tests within a group so that they can be controlled or queried separately.
@@ -1459,16 +1457,18 @@ function groupErrorMsg(error) {
 function include(fieldName) {
     var context$1 = context.useX();
     var inclusion = context$1.inclusion, exclusion = context$1.exclusion;
-    if (!fieldName) {
-        return { when: when };
-    }
+    invariant(isStringValue(fieldName));
     inclusion[fieldName] = defaultTo(exclusion.tests[fieldName], true);
     return { when: when };
     function when(condition) {
         var context$1 = context.useX();
         var inclusion = context$1.inclusion, exclusion = context$1.exclusion;
+        // This callback will run as part of the "isExcluded" series of checks
         inclusion[fieldName] = function () {
             if (hasOwnProperty(exclusion.tests, fieldName)) {
+                // I suspect this code is technically unreachable because
+                // if there are any skip/only rules applied to the current
+                // field, the "isExcluded" function will have already bailed
                 return defaultTo(exclusion.tests[fieldName], true);
             }
             if (isStringValue(condition)) {
@@ -1502,7 +1502,7 @@ function eager() {
     setMode(Modes.EAGER);
 }
 function shouldSkipBasedOnMode(testObject) {
-    if (isEager() && hasErrors(testObject.fieldName))
+    if (isEager() && hasErrorsByTestObjects(testObject.fieldName))
         return true;
     return false;
 }
@@ -1639,6 +1639,8 @@ function runAsyncTest(testObject) {
         asyncTest.then(done, fail);
     }
     catch (e) {
+        // We will probably never get here, unless the consumer uses a buggy custom Promise
+        // implementation that behaves differently than the native one, or if they for some
         fail();
     }
 }
@@ -1647,22 +1649,7 @@ function runAsyncTest(testObject) {
  * Runs sync tests - or extracts promise.
  */
 function runSyncTest(testObject) {
-    return context.run({ currentTest: testObject }, function () {
-        var result;
-        try {
-            result = testObject.testFn();
-        }
-        catch (e) {
-            if (shouldUseErrorAsMessage(testObject.message, e)) {
-                testObject.message = e;
-            }
-            result = false;
-        }
-        if (result === false) {
-            testObject.fail();
-        }
-        return result;
-    });
+    return context.run({ currentTest: testObject }, function () { return testObject.run(); });
 }
 
 /**
@@ -1881,7 +1868,7 @@ function warn() {
     ctx.currentTest.warn();
 }
 
-var VERSION = "4.3.1";
+var VERSION = "4.3.2-dev-2805e3";
 
 Object.defineProperty(exports, 'enforce', {
   enumerable: true,