test: assert HTTP status code is set on exception code in failure tests

Author: HarshMN2345

tests/VCS/Adapter/GitHubTest.php

@@ -237,8 +237,12 @@ public function testDeleteRepository(): void
 
     public function testDeleteNonExistingRepositoryFails(): void
     {
-        $this->expectException(\Exception::class);
-        $this->vcsAdapter->deleteRepository(static::$owner, 'non-existing-repo-' . \uniqid());
+        try {
+            $this->vcsAdapter->deleteRepository(static::$owner, 'non-existing-repo-' . \uniqid());
+            $this->fail('Expected exception not thrown');
+        } catch (\Exception $e) {
+            $this->assertGreaterThanOrEqual(400, $e->getCode());
+        }
     }
 
     public function testGetRepositoryName(): void

tests/VCS/Adapter/GitLabTest.php

@@ -1198,14 +1198,22 @@ public function testDeleteRepositoryTwiceFails(): void
         $this->vcsAdapter->createRepository(static::$owner, $repositoryName, false);
         $this->vcsAdapter->deleteRepository(static::$owner, $repositoryName);
 
-        $this->expectException(\Exception::class);
-        $this->vcsAdapter->deleteRepository(static::$owner, $repositoryName);
+        try {
+            $this->vcsAdapter->deleteRepository(static::$owner, $repositoryName);
+            $this->fail('Expected exception not thrown');
+        } catch (\Exception $e) {
+            $this->assertGreaterThanOrEqual(400, $e->getCode());
+        }
     }
 
     public function testDeleteNonExistingRepositoryFails(): void
     {
-        $this->expectException(\Exception::class);
-        $this->vcsAdapter->deleteRepository(static::$owner, 'non-existing-repo-' . \uniqid());
+        try {
+            $this->vcsAdapter->deleteRepository(static::$owner, 'non-existing-repo-' . \uniqid());
+            $this->fail('Expected exception not thrown');
+        } catch (\Exception $e) {
+            $this->assertGreaterThanOrEqual(400, $e->getCode());
+        }
     }
 
     public function testGetPullRequestFromBranchNoPR(): void

tests/VCS/Adapter/GiteaTest.php

@@ -1203,14 +1203,22 @@ public function testDeleteRepositoryTwiceFails(): void
         $this->vcsAdapter->createRepository(static::$owner, $repositoryName, false);
         $this->vcsAdapter->deleteRepository(static::$owner, $repositoryName);
 
-        $this->expectException(\Exception::class);
-        $this->vcsAdapter->deleteRepository(static::$owner, $repositoryName);
+        try {
+            $this->vcsAdapter->deleteRepository(static::$owner, $repositoryName);
+            $this->fail('Expected exception not thrown');
+        } catch (\Exception $e) {
+            $this->assertGreaterThanOrEqual(400, $e->getCode());
+        }
     }
 
     public function testDeleteNonExistingRepositoryFails(): void
     {
-        $this->expectException(\Exception::class);
-        $this->vcsAdapter->deleteRepository(static::$owner, 'non-existing-repo-' . \uniqid());
+        try {
+            $this->vcsAdapter->deleteRepository(static::$owner, 'non-existing-repo-' . \uniqid());
+            $this->fail('Expected exception not thrown');
+        } catch (\Exception $e) {
+            $this->assertGreaterThanOrEqual(400, $e->getCode());
+        }
     }
 
     public function testGetOwnerName(): void