Increased code test coverage.

drevops · Nov 4, 2024 · 16bca42 · 16bca42
1 parent 02b10a1
commit 16bca42
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 0 deletions.
diff --git a/src/DrevOps/BehatScreenshotExtension/Context/Initializer/ScreenshotContextInitializer.php b/src/DrevOps/BehatScreenshotExtension/Context/Initializer/ScreenshotContextInitializer.php
@@ -32,6 +32,8 @@ class ScreenshotContextInitializer implements ContextInitializer {
    *   File name pattern failed.
    * @param bool $needsPurging
    *   Check if need to actually purge.
+   *
+   * @codeCoverageIgnore
    */
   public function __construct(protected string $dir, protected bool $fail, private readonly string $failPrefix, protected bool $purge, protected string $filenamePattern, protected string $filenamePatternFailed, protected bool $needsPurging = TRUE) {
   }

diff --git a/src/DrevOps/BehatScreenshotExtension/Context/ScreenshotContext.php b/src/DrevOps/BehatScreenshotExtension/Context/ScreenshotContext.php
@@ -173,10 +173,12 @@ public function iSaveScreenshot(bool $fail = FALSE, ?string $filename = NULL): v
       $fileName = $this->makeFileName('png', $filename, $fail);
       $this->saveScreenshotData($fileName, $data);
     }
+    // @codeCoverageIgnoreStart
     catch (UnsupportedDriverActionException) {
       // Nothing to do here - drivers without support for screenshots
       // simply do not have them created.
     }
+    // @codeCoverageIgnoreEnd
   }
 
   /**
@@ -233,6 +235,8 @@ public function getBeforeStepScope(): BeforeStepScope {
    *
    * @return int
    *   Current timestamp.
+   *
+   * @codeCoverageIgnore
    */
   public function getCurrentTime(): int {
     return time();
@@ -303,10 +307,12 @@ protected function makeFileName(string $ext, ?string $filename = NULL, bool $fai
     }
 
     if (!empty($url) && !empty(getenv('BEHAT_SCREENSHOT_TOKEN_HOST'))) {
+      // @codeCoverageIgnoreStart
       $host = parse_url($url, PHP_URL_HOST);
       if ($host) {
         $url = str_replace($host, getenv('BEHAT_SCREENSHOT_TOKEN_HOST'), $url);
       }
+      // @codeCoverageIgnoreEnd
     }
 
     $data = [

diff --git a/src/DrevOps/BehatScreenshotExtension/ServiceContainer/BehatScreenshotExtension.php b/src/DrevOps/BehatScreenshotExtension/ServiceContainer/BehatScreenshotExtension.php
@@ -24,6 +24,8 @@ class BehatScreenshotExtension implements ExtensionInterface {
 
   /**
    * {@inheritdoc}
+   *
+   * @codeCoverageIgnore
    */
   public function process(ContainerBuilder $container): void {
   }
@@ -37,6 +39,8 @@ public function getConfigKey(): string {
 
   /**
    * {@inheritdoc}
+   *
+   * @codeCoverageIgnore
    */
   public function initialize(ExtensionManager $extensionManager): void {
   }

diff --git a/tests/phpunit/Unit/Tokenizer/TokenizerTest.php b/tests/phpunit/Unit/Tokenizer/TokenizerTest.php
@@ -300,6 +300,16 @@ public static function dataProviderReplaceTokens(): array {
         $data,
         '20240312_045703.foo-fail_foo-file.feature_6_Foo_step_name.png',
       ],
+      [
+        '{url}.{ext}',
+        $data,
+        'http%3A%2F%2Fexample.com%2Ffoo%3Ffoo%3Dfoo-value%23hello-fragment.png',
+      ],
+      [
+        '{nontoken}.{ext}',
+        $data,
+        '{nontoken}.png',
+      ],
     ];
   }