Merge branch 'main' into feat/to-have-title-regex

Author: nunomaduro

.gitignore

@@ -10,7 +10,6 @@ coverage.xml
 *.swp
 *.swo
 .phpunit.cache
-
 # Playwright
 node_modules/
 /test-results/

.temp/e2e/.gitignore

@@ -0,0 +1,2 @@
+/*
+!.gitignore

.temp/e2e/.gitkeep

@@ -26,6 +26,7 @@
         ]
     },
     "require-dev": {
+        "laravel/pint": "^1.20",
         "pestphp/pest-dev-tools": "^3.4.0"
     },
     "minimum-stability": "dev",

.temp/e2e/runtime.spec.js

@@ -0,0 +1,58 @@
+{
+    "preset": "laravel",
+    "rules": {
+        "array_push": true,
+        "backtick_to_shell_exec": true,
+        "date_time_immutable": true,
+        "declare_strict_types": true,
+        "lowercase_keywords": true,
+        "lowercase_static_reference": true,
+        "final_class": true,
+        "final_internal_class": true,
+        "final_public_method_for_abstract_class": true,
+        "fully_qualified_strict_types": true,
+        "global_namespace_import": {
+            "import_classes": true,
+            "import_constants": true,
+            "import_functions": true
+        },
+        "mb_str_functions": true,
+        "modernize_types_casting": true,
+        "new_with_parentheses": false,
+        "no_superfluous_elseif": true,
+        "no_useless_else": true,
+        "no_multiple_statements_per_line": true,
+        "ordered_class_elements": {
+            "order": [
+                "use_trait",
+                "case",
+                "constant",
+                "constant_public",
+                "constant_protected",
+                "constant_private",
+                "property_public",
+                "property_protected",
+                "property_private",
+                "construct",
+                "destruct",
+                "magic",
+                "phpunit",
+                "method_abstract",
+                "method_public_static",
+                "method_public",
+                "method_protected_static",
+                "method_protected",
+                "method_private_static",
+                "method_private"
+            ],
+            "sort_algorithm": "none"
+        },
+        "ordered_interfaces": true,
+        "ordered_traits": true,
+        "protected_to_private": true,
+        "self_accessor": true,
+        "self_static_accessor": true,
+        "strict_comparison": true,
+        "visibility_required": true
+    }
+}

composer.json

@@ -20,7 +20,7 @@ final class Compiler
      * @param  array<int, Operation>  $operations
      */
     public function __construct(
-        private array $operations
+        private readonly array $operations
     ) {
         //
     }

pint.json

@@ -4,16 +4,28 @@
 
 namespace Pest\Browser;
 
+use Pest\Browser\Contracts\Operation;
+
 /**
  * @internal
  */
 final class PendingTest
 {
     /**
      * The pending operations.
+     *
+     * @var array<int, Operation>
      */
     private array $operations = [];
 
+    /**
+     * Ends the chain and builds the test result.
+     */
+    public function __destruct()
+    {
+        $this->compile();
+    }
+
     /**
      * Visits a URL.
      */
@@ -35,9 +47,9 @@ public function toHaveTitle(string $title): self
     }
 
     /**
-     * Build the test result.
+     * Compile the JavaScript test file.
      */
-    public function build(): void
+    public function compile(): void
     {
         $compiler = new Compiler($this->operations);
 
@@ -49,12 +61,4 @@ public function build(): void
 
         expect($result->ok())->toBeTrue();
     }
-
-    /**
-     * Ends the chain and builds the test result.
-     */
-    public function __destruct()
-    {
-        $this->build();
-    }
 }

src/Compiler.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 use function Pest\Browser\visit;
 
 it('may have a title at playwright', function () {