feat: add tests for handling non-existent elements in various locator methods

Author: MrPunyapal

playground/resources/views/test-pages/element-tests.blade.php

@@ -138,7 +138,7 @@
 
         <div class="form-group">
             <label for="comments">Comments</label>
-            <textarea id="comments" name="comments" placeholder="Enter your comments here"></textarea>
+            <textarea id="comments" name="comments" data-testid="textarea-input" placeholder="Enter your comments here"></textarea>
         </div>
 
         <div class="form-group">

tests/Browser/Playwright/Locator/BoundingBoxTest.php

@@ -38,3 +38,12 @@
     expect($boundingBox['width'])->toBeGreaterThan(0);
     expect($boundingBox['height'])->toBeGreaterThan(0);
 });
+
+it('returns null when boundingBox element is not found', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $locator = $page->locator('.non-existent-element');
+
+    $boundingBox = $locator->boundingBox();
+
+    expect($boundingBox)->toBeNull();
+});

tests/Browser/Playwright/Locator/ContentFrameTest.php

@@ -23,3 +23,11 @@
 
     expect($frame)->toBeNull();
 });
+
+it('throws RuntimeException when contentFrame element is not found', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $locator = $page->locator('.non-existent-element');
+
+    expect(fn() => $locator->contentFrame())
+        ->toThrow(RuntimeException::class, 'Element not found');
+});

tests/Browser/Playwright/Locator/FilterTest.php

@@ -80,3 +80,43 @@
 
     expect($filteredButtons)->toBeInstanceOf(Locator::class);
 });
+
+it('handles non-string hasText in filter options', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $buttons = $page->locator('button');
+
+    // Test with non-string hasText (this should be ignored based on the code)
+    $filteredButtons = $buttons->filter(['hasText' => 123]);
+
+    expect($filteredButtons)->toBeInstanceOf(Locator::class);
+});
+
+it('handles non-string hasNotText in filter options', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $buttons = $page->locator('button');
+
+    // Test with non-string hasNotText (this should be ignored based on the code)
+    $filteredButtons = $buttons->filter(['hasNotText' => 123]);
+
+    expect($filteredButtons)->toBeInstanceOf(Locator::class);
+});
+
+it('handles non-locator has in filter options', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $buttons = $page->locator('button');
+
+    // Test with non-locator has (this should be ignored based on the code)
+    $filteredButtons = $buttons->filter(['has' => 'not-a-locator']);
+
+    expect($filteredButtons)->toBeInstanceOf(Locator::class);
+});
+
+it('handles non-locator hasNot in filter options', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $buttons = $page->locator('button');
+
+    // Test with non-locator hasNot (this should be ignored based on the code)
+    $filteredButtons = $buttons->filter(['hasNot' => 'not-a-locator']);
+
+    expect($filteredButtons)->toBeInstanceOf(Locator::class);
+});

tests/Browser/Playwright/Locator/FrameAndWaitTest.php

@@ -68,20 +68,18 @@
     expect($button->isEnabled())->toBeTrue();
 });
 
-it('can wait for selector to appear relative to locator', function (): void {
+it('throws RuntimeException when waitForElementState element is not found', function (): void {
     $page = page()->goto('/test/element-tests');
-    $container = $page->getByTestId('profile-section');
+    $locator = $page->locator('.non-existent-element');
 
-    $childLocator = $container->waitForSelector('h2');
-
-    expect($childLocator)->toBeInstanceOf(Locator::class);
+    expect(fn() => $locator->waitForElementState('visible'))
+        ->toThrow(RuntimeException::class, 'Element not found');
 });
 
-it('returns null when waitForSelector times out', function (): void {
+it('throws RuntimeException when waitForState element is not found', function (): void {
     $page = page()->goto('/test/element-tests');
-    $container = $page->getByTestId('profile-section');
-
-    $result = $container->waitForSelector('.definitely-non-existent-element-12345', ['timeout' => 100]);
+    $locator = $page->locator('.non-existent-element');
 
-    expect($result)->toBeNull();
+    expect(fn() => $locator->waitForState('visible'))
+        ->toThrow(RuntimeException::class, 'Element not found');
 });

tests/Browser/Playwright/Locator/InputMethodsTest.php

@@ -26,3 +26,19 @@
     $checkbox->setChecked(true);
     expect($checkbox->isChecked())->toBeTrue();
 });
+
+it('throws RuntimeException when selectText element is not found', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $locator = $page->locator('.non-existent-element');
+
+    expect(fn() => $locator->selectText())
+        ->toThrow(RuntimeException::class, 'Element not found');
+});
+
+it('throws RuntimeException when setChecked element is not found', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $locator = $page->locator('.non-existent-element');
+
+    expect(fn() => $locator->setChecked(true))
+        ->toThrow(RuntimeException::class, 'Element not found');
+});

tests/Browser/Playwright/Locator/IsEditableTest.php

@@ -0,0 +1,39 @@
+<?php
+
+declare(strict_types=1);
+
+it('returns true for editable input elements', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $inputLocator = $page->getByTestId('text-input');
+
+    expect($inputLocator->isEditable())->toBeTrue();
+});
+
+it('returns true for textarea elements', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $textareaLocator = $page->getByTestId('textarea-input');
+
+    expect($textareaLocator->isEditable())->toBeTrue();
+});
+
+it('throws RuntimeException for non-editable elements', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $divLocator = $page->getByTestId('profile-section');
+
+    expect(fn() => $divLocator->isEditable())
+        ->toThrow(RuntimeException::class, 'Element is not an <input>, <textarea>, <select> or [contenteditable]');
+});
+
+it('returns false for disabled input elements', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $disabledInputLocator = $page->getByTestId('disabled-input');
+
+    expect($disabledInputLocator->isEditable())->toBeFalse();
+});
+
+it('returns false for readonly input elements', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $readonlyInputLocator = $page->getByTestId('readonly-input');
+
+    expect($readonlyInputLocator->isEditable())->toBeFalse();
+});

tests/Browser/Playwright/Locator/OwnerFrameTest.php

@@ -14,3 +14,11 @@
 
     expect($frame)->toBeNull();
 });
+
+it('throws RuntimeException when ownerFrame element is not found', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $locator = $page->locator('.non-existent-element');
+
+    expect(fn() => $locator->ownerFrame())
+        ->toThrow(RuntimeException::class, 'Element not found');
+});

tests/Browser/Playwright/Locator/SelectOptionTest.php

@@ -28,3 +28,11 @@
 
     expect($selected)->toBeArray();
 });
+
+it('throws RuntimeException when selectOption element is not found', function (): void {
+    $page = page()->goto('/test/element-tests');
+    $locator = $page->locator('.non-existent-element');
+
+    expect(fn() => $locator->selectOption('value'))
+        ->toThrow(RuntimeException::class, 'Element not found');
+});

tests/Browser/Playwright/Locator/TapTest.php

@@ -29,3 +29,11 @@
 
     expect($resultElement->textContent())->toBe('Button was clicked!');
 });
+
+it('throws RuntimeException when tap element is not found', function (): void {
+    $page = page(null, ['hasTouch' => true])->goto('/test/element-tests');
+    $locator = $page->locator('.non-existent-element');
+
+    expect(fn() => $locator->tap())
+        ->toThrow(RuntimeException::class, 'Element not found');
+});