Better message when skipping tests

Add info about the env vars and config filename when skipping tests. Refs #81
samwilson · Dec 18, 2024 · c679cfa · c679cfa
1 parent 70f4ff9
commit c679cfa
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/tests/TestCase.php b/tests/TestCase.php
@@ -30,12 +30,13 @@ public function getFlickr(bool $authenticate = false): PhpFlickr
         $apiSecret = getenv('FLICKR_API_SECRET');
         $accessToken = getenv('FLICKR_ACCESS_TOKEN');
         $accessTokenSecret = getenv('FLICKR_ACCESS_SECRET');
-        if (empty($apiKey) && file_exists(__DIR__ . '/config.php')) {
-            require __DIR__ . '/config.php';
+        $configFile = __DIR__ . '/config.php';
+        if (empty($apiKey) && file_exists($configFile)) {
+            require $configFile;
         }
         if (empty($apiKey)) {
             // Skip if no key found, so PRs from forks can still be run in CI.
-            static::markTestSkipped('No Flickr API key set.');
+            static::markTestSkipped('No Flickr API key set (in either the FLICKR_* env vars or ' . $configFile . ')');
         }
         try {
             $this->flickrs[$authed] = new PhpFlickr($apiKey, $apiSecret);