fix: auto-cleanup afterEach

Author: crutchcorn

src/helpers.js

@@ -1,3 +1,5 @@
+import {fireEvent} from "./events";
+
 function jestFakeTimersAreEnabled() {
   /* istanbul ignore else */
   if (typeof jest !== 'undefined' && jest !== null) {
@@ -14,7 +16,16 @@ function jestFakeTimersAreEnabled() {
 
 const instance = {current: undefined};
 
+/**
+ * Should not rely directly on `afterEach` without library detection
+ * @see https://testing-library.com/docs/react-testing-library/setup/#skipping-auto-cleanup
+ */
 afterEach(() => {
+  /**
+   * Behavior should be customizability like it is with `react-testing-library`
+   * @see https://testing-library.com/docs/react-testing-library/setup/#skipping-auto-cleanup
+   */
+  if (instance.current) fireEvent.sigkill(instance.current);
   instance.current = undefined;
 })
 

src/pure.ts

@@ -66,6 +66,15 @@ async function render(
 
   exec.stderr.on('data', (result: string) => {
     if (_readyPromiseInternals && !_isReadyResolved) {
+      /**
+       * TODO: We're getting an error where "result" is only the first line of many drawn.
+       *  Let's go ahead and set a timeout var in `config.js` and debounce the rejection
+       *
+       * Then, we'll set a boolean to not do any other kind of logging to stdout (or promise resolve)
+       * until that timeout/debounce has expired.
+       *
+       * Also merge with the `'error'` field above
+       */
       _readyPromiseInternals.reject(new Error(result))
       _isReadyResolved = true;
     }