fix tests

SniperSister · SniperSister · commit afdadc17bc35 · 2025-05-19T09:48:44.000+02:00
diff --git a/tests/Unit/Jobs/UpdateSiteTest.php b/tests/Unit/Jobs/UpdateSiteTest.php
@@ -61,23 +61,17 @@ public function testJobQuitsIfNoUpdateIsAvailable()
 
     public function testJobQuitsIfRequirementsArentMet()
     {
+        $this->expectExceptionMessage("Site does not meet requirements");
+
         $site = $this->getSiteMock(
             [
                 'checkHealth' => $this->getHealthCheckMock(['update_requirement_state' => false])
             ]
         );
 
-        Log::spy();
-
         $object = new UpdateSite($site, "1.0.1");
         $object->handle();
 
-        Log::shouldHaveReceived('info')
-            ->once()
-            ->withArgs(function ($message) {
-                return str_contains($message, 'Site does not meet requirements, abort');
-            });
-
         $this->assertTrue(true);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -61,23 +61,17 @@ public function testJobQuitsIfNoUpdateIsAvailable()`
`61`	`61`
`62`	`62`	`public function testJobQuitsIfRequirementsArentMet()`
`63`	`63`	`{`
	`64`	`+ $this->expectExceptionMessage("Site does not meet requirements");`
	`65`	`+`
`64`	`66`	`$site = $this->getSiteMock(`
`65`	`67`	`[`
`66`	`68`	`'checkHealth' => $this->getHealthCheckMock(['update_requirement_state' => false])`
`67`	`69`	`]`
`68`	`70`	`);`
`69`	`71`
`70`		`- Log::spy();`
`71`		`-`
`72`	`72`	`$object = new UpdateSite($site, "1.0.1");`
`73`	`73`	`$object->handle();`
`74`	`74`
`75`		`- Log::shouldHaveReceived('info')`
`76`		`- ->once()`
`77`		`- ->withArgs(function ($message) {`
`78`		`- return str_contains($message, 'Site does not meet requirements, abort');`
`79`		`- });`
`80`		`-`
`81`	`75`	`$this->assertTrue(true);`
`82`	`76`	`}`
`83`	`77`