patch(vest): Call done callback immediately when field does not exist

Author: ealush

packages/vest/src/core/suite/produce/__tests__/done.test.ts

@@ -316,4 +316,18 @@ describe('done', () => {
       expect(done_0).not.toHaveBeenCalled();
     });
   });
+
+  describe('Passing a field that does not exist', () => {
+    it('Should call the callback immedaitely', () => {
+      const cb = jest.fn();
+
+      const suite = vest.create(() => {
+        vest.test('test', () => {});
+      });
+
+      suite().done('non-existent', cb);
+
+      expect(cb).toHaveBeenCalled();
+    });
+  });
 });

packages/vest/src/core/suite/produce/produceSuiteRunResult.ts

@@ -1,4 +1,9 @@
-import { isEmpty, assign, isFunction, cache as createCache } from 'vest-utils';
+import {
+  assign,
+  isFunction,
+  cache as createCache,
+  numberEquals,
+} from 'vest-utils';
 
 import ctx from 'ctx';
 import hasRemainingTests from 'hasRemainingTests';
@@ -33,8 +38,7 @@ function shouldSkipDoneRegistration(
   // If we do not have any test runs for the current field
   return !!(
     !isFunction(callback) ||
-    (fieldName &&
-      (!output.tests[fieldName] || isEmpty(output.tests[fieldName].testCount)))
+    (fieldName && numberEquals(output.tests[fieldName]?.testCount, 0))
   );
 }