Clean up impossible test conditions from the RequestFetcherTest class

Author: ste93cry

tests/Integration/RequestFetcherTest.php

@@ -4,7 +4,6 @@
 
 namespace Sentry\SentryBundle\Tests\Integration;
 
-use GuzzleHttp\Psr7\ServerRequest;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Message\ServerRequestInterface;
@@ -37,24 +36,36 @@ protected function setUp(): void
         $this->requestFetcher = new RequestFetcher($this->requestStack, $this->httpMessageFactory);
     }
 
-    /**
-     * @dataProvider fetchRequestDataProvider
-     */
-    public function testFetchRequest(?Request $request, ?ServerRequestInterface $expectedRequest): void
+    public function testFetchRequest(): void
     {
+        $request = Request::create('https://www.example.com');
+        $expectedRequest = $this->createMock(ServerRequestInterface::class);
+
         $this->requestStack->expects($this->once())
             ->method('getCurrentRequest')
             ->willReturn($request);
 
-        $this->httpMessageFactory->expects(null !== $expectedRequest ? $this->once() : $this->never())
+        $this->httpMessageFactory->expects($this->once())
             ->method('createRequest')
             ->with($request)
             ->willReturn($expectedRequest);
 
         $this->assertSame($expectedRequest, $this->requestFetcher->fetchRequest());
     }
 
-    public function testFetchRequestFailsSilently(): void
+    public function testFetchRequestReturnsNullIfTheRequestStackIsEmpty(): void
+    {
+        $this->requestStack->expects($this->once())
+            ->method('getCurrentRequest')
+            ->willReturn(null);
+
+        $this->httpMessageFactory->expects($this->never())
+            ->method('createRequest');
+
+        $this->assertNull($this->requestFetcher->fetchRequest());
+    }
+
+    public function testFetchRequestReturnsNullIfTheRequestFactoryThrowsAnException(): void
     {
         $this->requestStack->expects($this->once())
             ->method('getCurrentRequest')
@@ -66,20 +77,4 @@ public function testFetchRequestFailsSilently(): void
 
         $this->assertNull($this->requestFetcher->fetchRequest());
     }
-
-    /**
-     * @return \Generator<array{Request|null,ServerRequest|null}>
-     */
-    public function fetchRequestDataProvider(): \Generator
-    {
-        yield [
-            null,
-            null,
-        ];
-
-        yield [
-            Request::create('http://www.example.com'),
-            new ServerRequest('GET', 'http://www.example.com'),
-        ];
-    }
 }