fix(vest): clear cache before each test

packages/vest/src/core/VestBus/VestBus.ts

@@ -21,7 +21,20 @@ export function useInitVestBus() {
   const VestBus = Bus.useBus();
 
   on('TEST_COMPLETED', () => {});
-  // on("TEST_RUN_STARTED", () => {});
+
+  on('TEST_RUN_STARTED', () => {
+    // Bringin this back due to https://github.com/ealush/vest/issues/1157
+    // This is a very pecluiar bug in which we're seeing vest behaving differently between
+    // runs when suite.get() is called.
+    // In the bug we experienced that failing tests were skipped in the second run.
+    // The reason: suite.get() built the failures cache. Calling suite.get() before the test run
+    // made Vest think that the field already had failing tests (even though it was the same test!)
+    // and it skipped the test.
+    // A better solution is to be able to identify each failure to its actual position in the suite
+    // but this requires some rearchitecting within Vest.
+    // This is an easy enough solution - we just reset the cache before the test run, let's hope we don't see
+    // any performance issues.
+  });
 
   VestBus.on(RuntimeEvents.ISOLATE_PENDING, (isolate: TIsolate) => {
     if (VestTest.is(isolate)) {

packages/vest/src/suite/__tests__/subscribe.test.ts

@@ -98,3 +98,26 @@ describe('suite.subscribe', () => {
     });
   });
 });
+
+describe('#1157 (@codrin-iftimie) suite.get() in subscribe() skips the first validation of the field', () => {
+  it('Should fail for the first field in both runs', () => {
+    const suite = vest.create(data => {
+      vest.test('a', 'Enter a value', () => {
+        enforce(data.a).isNotEmpty();
+      });
+
+      vest.test('a', 'Enter a value 2', () => {
+        enforce(data.a).isNotEmpty();
+      });
+    });
+
+    suite.subscribe(() => {
+      suite.get();
+    });
+
+    suite({ a: '' });
+    expect(suite.getErrors('a')).toEqual(['Enter a value']);
+    suite({ a: '' });
+    expect(suite.getErrors('a')).toEqual(['Enter a value']);
+  });
+});