Merge pull request #30 from chrisguitarguy/fix_substr

Move the Empty Check on $cassetteName Up

Author: renatomefi

src/VCRTestListener.php

@@ -45,15 +45,15 @@ public function startTest(Test $test): void
         $parsed = self::parseDocBlock($docBlock, '@vcr');
         $cassetteName = array_pop($parsed);
 
+        if (empty($cassetteName)) {
+            return;
+        }
+
         // If the cassette name ends in .json, then use the JSON storage format
         if (substr($cassetteName, -5) === '.json') {
             VCR::configure()->setStorage('json');
         }
 
-        if (empty($cassetteName)) {
-            return;
-        }
-
         VCR::turnOn();
         VCR::insertCassette($cassetteName);
     }

tests/VCRTestListenerTest.php

@@ -40,6 +40,14 @@ public function testInterceptsWithAnnotationsWhenUsingDataProvider(int $dummyVal
         $this->assertEquals('This is a annotation test dummy.', $result, 'Call was not intercepted (using annotations with data provider).');
     }
 
+    /**
+     * @group https://github.com/php-vcr/phpunit-testlistener-vcr/issues/29
+     */
+    public function testNoVcrAnnotationRunsSuccessfulAndDoesNotProduceWarnings()
+    {
+        $this->assertTrue(true, 'just adding an assertion here');
+    }
+
     public function dummyDataProvider(): array
     {
         return [