Merge pull request #8548 from ProcessMaker/fix-tests

Fix failing tests

Author: nolanpro

ProcessMaker/Jobs/RefreshArtisanCaches.php

@@ -42,6 +42,13 @@ public function middleware(): array
      */
     public function handle()
     {
+        // Skip in testing environment because this reconnects the database
+        // meaning we loose transactions, and sets the console output verbosity
+        // to quiet so we loose expectsOutput assertions.
+        if (app()->environment('testing')) {
+            return;
+        }
+
         $options = [
             '--no-interaction' => true,
             '--quiet' => true,

tests/TestCase.php

@@ -134,18 +134,6 @@ public function setUpMockScriptRunners(): void
         config()->set('script-runners.php-nayra.runner', 'MockRunner');
     }
 
-    /**
-     * Calling the real config:cache command reconnects the database
-     * and since we're using transactions for our tests, we lose any data
-     * saved before the command is run. Instead, mock it out here.
-     */
-    public function setUpMockConfigCache(): void
-    {
-        Bus::fake([
-            RefreshArtisanCaches::class,
-        ]);
-    }
-
     /**
      * Run additional tearDowns from traits.
      */