Merge pull request #87 from aik099/frame-switching-by-index

aik099 · web-flow · commit f9d36b63480c · 2024-02-27T17:01:16.000+02:00
Added tests to verify, that iframe switching by an id works
diff --git a/tests/Basic/IFrameTest.php b/tests/Basic/IFrameTest.php
@@ -6,22 +6,36 @@
 
 final class IFrameTest extends TestCase
 {
-    public function testIFrame(): void
+    /**
+     * @dataProvider iFrameDataProvider
+     */
+    public function testIFrame(string $iframeIdentifier, string $elementSelector, string $elementContent): void
     {
         $this->getSession()->visit($this->pathTo('/iframe.html'));
         $webAssert = $this->getAssertSession();
 
         $el = $webAssert->elementExists('css', '#text');
         $this->assertSame('Main window div text', $el->getText());
 
-        $this->getSession()->switchToIFrame('subframe');
+        $this->getSession()->switchToIFrame($iframeIdentifier);
 
-        $el = $webAssert->elementExists('css', '#text');
-        $this->assertSame('iFrame div text', $el->getText());
+        $el = $webAssert->elementExists('css', $elementSelector);
+        $this->assertSame($elementContent, $el->getText());
 
         $this->getSession()->switchToIFrame();
 
         $el = $webAssert->elementExists('css', '#text');
         $this->assertSame('Main window div text', $el->getText());
     }
+
+    /**
+     * @return array
+     */
+    public static function iFrameDataProvider()
+    {
+        return array(
+            'by name' => array('subframe_by_name', '#text', 'iFrame div text'),
+            'by id' => array('subframe_by_id', '#foobar', 'Some accentués characters'),
+        );
+    }
 }
diff --git a/web-fixtures/iframe.html b/web-fixtures/iframe.html
@@ -2,7 +2,9 @@
 <html>
 <body>
 
-<iframe src="iframe_inner.html" name="subframe"></iframe>
+<iframe src="iframe_inner.html" name="subframe_by_name"></iframe>
+
+<iframe src="issue131.html" name="subframe_by_id"></iframe>
 
 <div id="text">
     Main window div text
