From 97673ae30baa68be7d8d26e96bc8481ba73b65fc Mon Sep 17 00:00:00 2001
From: W0rma <beck.worma@gmail.com>
Date: Tue, 9 Dec 2025 14:44:50 +0100
Subject: [PATCH] Fix type error for element without form

---
 src/Codeception/Lib/InnerBrowser.php          |  4 ++--
 tests/data/app/view/form/field_values.php     |  1 +
 tests/unit/Codeception/Module/TestsForWeb.php | 21 ++++++++++++++++++-
 3 files changed, 23 insertions(+), 3 deletions(-)

diff --git a/src/Codeception/Lib/InnerBrowser.php b/src/Codeception/Lib/InnerBrowser.php
index dc11858..22dd243 100644
--- a/src/Codeception/Lib/InnerBrowser.php
+++ b/src/Codeception/Lib/InnerBrowser.php
@@ -802,7 +802,7 @@ protected function getValueAndTextFromField(Crawler $nodes): array|string
             return $values;
         }
 
-        $this->fail("Element {$nodes} is not a form field or does not contain a form field");
+        $this->fail("Element {$nodes->getNode(0)?->getNodePath()} is not a form field or does not contain a form field");
     }
 
     /**
@@ -1539,7 +1539,7 @@ public function grabValueFrom($field): mixed
             return $values;
         }
 
-        $this->fail("Element {$nodes} is not a form field or does not contain a form field");
+        $this->fail("Element {$nodes->getNode(0)?->getNodePath()} is not a form field or does not contain a form field");
     }
 
     public function setCookie($name, $val, $params = [])
diff --git a/tests/data/app/view/form/field_values.php b/tests/data/app/view/form/field_values.php
index 8c99c29..df89ba3 100644
--- a/tests/data/app/view/form/field_values.php
+++ b/tests/data/app/view/form/field_values.php
@@ -54,5 +54,6 @@
         
         <input type="submit" name="submit" value="Submit" />
     </form>
+    <button type="submit" name="button_without_form" value="yes" />
 </body>
 </html>
diff --git a/tests/unit/Codeception/Module/TestsForWeb.php b/tests/unit/Codeception/Module/TestsForWeb.php
index 10a3868..acea18e 100644
--- a/tests/unit/Codeception/Module/TestsForWeb.php
+++ b/tests/unit/Codeception/Module/TestsForWeb.php
@@ -633,6 +633,14 @@ public function testDontSeeInFieldOnTextarea()
         $this->module->dontSeeInField('textarea', 'sunset');
         $this->module->dontSeeInField('descendant-or-self::textarea[@id="description"]', 'sunset');
     }
+    
+    public function testSeeInFieldOnButtonWithoutFormFails()
+    {
+        $this->shouldFail('Element /html/body/button is not a form field or does not contain a form field');
+        
+        $this->module->amOnPage('/form/field_values');
+        $this->module->seeInField(['name' => 'button_without_form'], 'yes');
+    }
 
     public function testSeeInFormFields()
     {
@@ -784,6 +792,14 @@ public function testGrabValueFromWithFillField()
         $this->assertSame('new value', $result);
     }
 
+    public function testGrabValueFromOnButtonWithoutFormFails()
+    {
+        $this->shouldFail('Element /html/body/button is not a form field or does not contain a form field');
+
+        $this->module->amOnPage('/form/field_values');
+        $this->module->grabValueFrom(['name' => 'button_without_form']);
+    }
+
     public function testGrabAttributeFrom()
     {
         $this->module->amOnPage('/search');
@@ -1251,9 +1267,12 @@ public function testSelectTwoSubmitsByCSS()
         $this->assertSame('2', $form['sandwich_select']);
     }
 
-    protected function shouldFail()
+    protected function shouldFail(?string $message = null)
     {
         $this->expectException(AssertionFailedError::class);
+        if (null !== $message) {
+            $this->expectExceptionMessage($message);
+        }
     }
 
     /**