FOUR-24917: Fix unit tests

ProcessMaker/Providers/AuthServiceProvider.php

@@ -57,8 +57,6 @@ class AuthServiceProvider extends ServiceProvider
      */
     public function boot()
     {
-        \Log::info('AuthServiceProvider boot');
-
         $this->registerPolicies();
 
         Passport::enablePasswordGrant();

phpunit.package.xml

@@ -43,7 +43,7 @@
 
     <!-- Workflow Database -->
     <env name="DB_ADAPTER" value="mysql" />
-    <env name="DB_DATABASE" value="test" />
+    <env name="DB_DATABASE" value="processmaker" />
     <env name="DB_TIMEZONE" value="+00:00" />
     <env name="NAYRA_DOCKER_NETWORK" value="pm4-tools_default" />
 

phpunit.xml

@@ -48,7 +48,7 @@
 
       <!-- Workflow Database -->
       <env name="DB_ADAPTER" value="mysql" />
-      <env name="DB_DATABASE" value="test" />
+      <env name="DB_DATABASE" value="processmaker" />
       <env name="DB_TIMEZONE" value="+00:00" />
       <env name="NAYRA_DOCKER_NETWORK" value="pm4-tools_default" />
 

tests/Feature/Docker/TimeoutsTest.php

@@ -25,19 +25,12 @@ class TimeoutsTest extends TestCase
     // How long to sleep our test scripts that should not exceed timeout
     const SLEEP_NOT_EXCEED = 1;
 
-    /**
-     * Make sure we have a personal access client set up
-     */
-    public function setUpWithPersonalAccessClient()
-    {
-        $this->withPersonalAccessClient();
-    }
-
     /**
      * Run a test script and assert that the specified timeout is exceeded
      */
     private function assertTimeoutExceeded($data)
     {
+        $this->withPersonalAccessClient();
         Event::fake([
             ScriptResponseEvent::class,
         ]);
@@ -95,6 +88,7 @@ private function assertTimeoutNotExceeded($data)
      */
     public function testPhpScriptTimeoutExceeded()
     {
+        $this->withPersonalAccessClient();
         config(['simulate_timeout' => true]);
         $this->assertTimeoutExceeded([
             'language' => 'php',
@@ -107,6 +101,7 @@ public function testPhpScriptTimeoutExceeded()
      */
     public function testPhpScriptTimeoutNotExceeded()
     {
+        $this->withPersonalAccessClient();
         $this->assertTimeoutNotExceeded([
             'language' => 'php',
             'timeout' => self::TIMEOUT_LENGTH,

tests/Feature/RouteTest.php

@@ -3,6 +3,7 @@
 namespace Tests\Feature;
 
 use Illuminate\Routing\Router;
+use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 use Tests\Feature\Shared\RequestHelper;
 use Tests\TestCase;
 
@@ -14,6 +15,12 @@ class RouteTest extends TestCase
      * This this does some basic checks to make sure we converted routes
      * to the correct class-based routes as part of the Laravel 8 upgrade
      */
+    protected function setUp(): void
+    {
+        parent::setUp();
+        config(['queue.tenant_tracking_enabled' => true]);
+    }
+
     public function testIndexRoute()
     {
         $ethosRoutePath = base_path('vendor/processmaker/package-ellucian-ethos/routes/');

tests/Feature/Shared/LoggingHelper.php

@@ -118,7 +118,7 @@ public function assertLogIsEmpty()
     {
         $records = app('log')->getHandlers()[0]->getRecords();
 
-        return $this->assertEquals(0, count($records), 'Failed asserting that the log is empty.');
+        return $this->assertEquals(0, count($records), 'Failed asserting that the log is empty: ' . print_r($records, true));
     }
 
     /**