Cleaned up code

.github/workflows/javascript.yaml

@@ -16,7 +16,7 @@ name: JavaScript
 on:
   pull_request:
     paths: &paths
-      - "tests/**/*.{js,ts}"
+      - "tests/**/*.ts"
       - "docker-compose.yml"
   push:
     branches:

.github/workflows/module.yaml

@@ -53,7 +53,7 @@ jobs:
       - uses: go-task/setup-task@v2
 
       - name: Install Drupal site
-        run: task drupal:site:install
+        run: task drupal:site:install --yes
 
       - name: Extract translations
         run: task translations:extract

modules/os2forms_test_playwright/src/VcrOperator.php

@@ -33,6 +33,9 @@ public function __construct(
    *
    * @param array<string, mixed> $configuration
    *   The configuration.
+   *
+   * @return array<string, mixed>
+   *   The complete and validated configuration.
    */
   public function configure(array $configuration): array {
     $conf = $this->state->get(self::CONFIGURATION);
@@ -147,6 +150,9 @@ public static function getSubscribedEvents(): array {
 
   /**
    * Validate configuration.
+   *
+   * @return array<string, mixed>
+   *   The complete and validated configuration.
    */
   private function validateConfiguration(): array {
     $configuration = $this->state->get(self::CONFIGURATION);

phpstan.neon

@@ -7,6 +7,8 @@ parameters:
         - message: '#Method Drupal\\os2forms_f2\\.+\(\) return type has no value type specified in iterable type array.#'
         - message: '#Method Drupal\\os2forms_f2\\.+\(\) has parameter \$.+ with no value type specified in iterable type array.#'
         - message: '#Function .+\(\) return type has no value type specified in iterable type array.#'
+        - message: '#Call to an undefined method Symfony\\Contracts\\EventDispatcher\\EventDispatcherInterface::addListener\(\).#'
+          path: modules/os2forms_test_playwright/src/VcrOperator.php
 
 # Local Variables:
 # mode: yaml

src/Helper/WebformHelperF2.php

@@ -131,7 +131,7 @@ public function createJob(WebformSubmissionInterface $webformSubmission, Webform
       $queue->enqueueJob($job);
       $context['@queue'] = $queue->id();
       $this->notice('F2 job (archive on @archive_target) added to the queue @queue.', $context + [
-        '@archive_target' => $handlerSettings->archive?->archiveTarget?->value ?? '',
+        '@archive_target' => $handlerSettings->archive->archiveTarget->value,
       ]);
 
       return $job;

tests/Playwright/tests/user-manual-images.spec.ts

@@ -391,7 +391,6 @@ screenshooting ${selector}
   await vcrEject();
 });
 
-
 test("webform submission", async ({ page }) => {
   const context = "webform-submission";
   const locale = "da";