Fix browser tests

hasanakg · hasanakg · commit fdfe5501348c · 2023-11-10T15:09:53.000+01:00
diff --git a/jest.config.js b/jest.config.js
@@ -42,7 +42,7 @@ module.exports = {
                 compatConfig: {
                     MODE: 3,
                     COMPILER_V_ON_NATIVE: true,
-                    COMPILER_V_BIND_SYNC: false
+                    COMPILER_V_BIND_SYNC: true
                 }
             }
         }
diff --git a/resources/js/Pages/Results.vue b/resources/js/Pages/Results.vue
@@ -109,7 +109,7 @@
                 </form>
             </section>
         </section>
-        <cdx-dialog class="confirmation-dialog"
+        <cdx-dialog id="results-confirmation-dialog"
             :title="$i18n('confirmation-dialog-title')"
             v-model:open="confirmationDialog"
             :primary-action="{
@@ -118,6 +118,7 @@
                 actionType: 'progressive'
 
             }"
+            @update:open="disableConfirmation = false"
             @primary="_handleConfirmation"
             close-button-label="X"
         >
diff --git a/tests/Browser/Components/DecisionDropdown.php b/tests/Browser/Components/DecisionDropdown.php
@@ -19,7 +19,7 @@ public function __construct(int $mismatchId)
      */
     public function selector()
     {
-        return "#mismatch-$this->mismatchId .wikit-Dropdown";
+        return "#mismatch-$this->mismatchId .cdx-select-vue";
     }
 
     /**
@@ -41,17 +41,17 @@ public function assert(Browser $browser)
     public function elements()
     {
         return [
-            '@select-menu' => '.wikit-Dropdown__select',
-            '@selected' => '.wikit-Dropdown__selectedOption',
-            '@menu-items' => '.wikit-Dropdown__menu'
+            '@select-menu' => '.cdx-select-vue__handle',
+            '@selected' => '.cdx-select-vue__handle',
+            '@menu-items' => '.cdx-menu__listbox'
         ];
     }
 
     public function selectPosition(Browser $browser, int $position, string $option)
     {
         $browser->click('@select-menu')
             ->within('@menu-items', function ($menu) use ($position, $option) {
-                $positionSelector = ".wikit-OptionsMenu__item:nth-child($position)";
+                $positionSelector = ".cdx-menu-item:nth-child($position)";
                 $menu->assertSeeIn($positionSelector, $option)
                     ->click($positionSelector);
             })
diff --git a/tests/Browser/ItemsFormTest.php b/tests/Browser/ItemsFormTest.php
@@ -48,7 +48,10 @@ public function test_empty_item_list_yields_warning()
                     ->press('.submit-ids')
                     ->assertSee('Please provide the Item identifiers that should be checked.');
 
-            $this->assertStringContainsString('--warning', $browser->attribute('@items-input', 'class'));
+            $this->assertStringContainsString(
+                '--warning',
+                $browser->attribute('@items-input-validation-message', 'class')
+            );
         });
     }
 
@@ -72,7 +75,10 @@ public function test_invalid_item_list_yields_error()
                     ->press('.submit-ids')
                     ->assertSee('One or more Item identifiers couldn\'t be processed.');
 
-            $this->assertStringContainsString('--error', $browser->attribute('@items-input', 'class'));
+            $this->assertStringContainsString(
+                '--error',
+                $browser->attribute('@items-input-validation-message', 'class')
+            );
         });
     }
 
diff --git a/tests/Browser/Pages/HomePage.php b/tests/Browser/Pages/HomePage.php
@@ -40,7 +40,8 @@ public function elements()
     {
         return [
             '@form' => '#items-form',
-            '@items-input' => '@form textarea'
+            '@items-input' => '@form textarea',
+            '@items-input-validation-message' => '@form .wikit-TextArea .wikit-ValidationMessage'
         ];
     }
 }
diff --git a/tests/Browser/Pages/ResultsPage.php b/tests/Browser/Pages/ResultsPage.php
@@ -47,7 +47,7 @@ public function elements()
     {
         return [
             '@back-button' => '.back-button',
-            '@confirmation-dialog' => '.confirmation-dialog',
+            '@confirmation-dialog' => '#results-confirmation-dialog',
             '@disable-confirmation' => '.disable-confirmation',
             '@error-section' => '#error-section',
             '@disable-confirmation-label' => '.disable-confirmation>.wikit-checkbox__label'
diff --git a/tests/Browser/ResultsTest.php b/tests/Browser/ResultsTest.php
@@ -343,7 +343,7 @@ public function test_can_disable_confirmation_dialog()
                     'label' => 'Wrong data on external source'
                 ])
                 ->waitFor('@confirmation-dialog')
-                ->pause(250)
+                ->pause(self::ANIMATION_WAIT_MS)
                 ->within('@confirmation-dialog', function ($dialog) {
                     $dialog->assertSee('Do not show again')
                         ->assertVue('checked', false, '@disable-confirmation')
@@ -392,7 +392,7 @@ public function test_disable_confirmation_checkbox_resets()
                     $dialog->assertSee('Do not show again')
                         ->assertVue('checked', false, '@disable-confirmation')
                         ->click('@disable-confirmation-label')
-                        ->click('.wikit-Dialog__close');
+                        ->click('.cdx-dialog__header__close-button');
                 })
                 ->waitUntilMissing('@confirmation-dialog')
                 ->decideAndApply($mismatch, [
diff --git a/webpack.mix.js b/webpack.mix.js
@@ -32,7 +32,7 @@ mix.ts('resources/js/app.ts', 'public/js')
                             compatConfig: {
                                 MODE: 3,
                                 COMPILER_V_ON_NATIVE: true,
-                                COMPILER_V_BIND_SYNC: false
+                                COMPILER_V_BIND_SYNC: true
                             }
                         }
                     }

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ module.exports = {`
`42`	`42`	`compatConfig: {`
`43`	`43`	`MODE: 3,`
`44`	`44`	`COMPILER_V_ON_NATIVE: true,`
`45`		`- COMPILER_V_BIND_SYNC: false`
	`45`	`+ COMPILER_V_BIND_SYNC: true`
`46`	`46`	`}`
`47`	`47`	`}`
`48`	`48`	`}`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ public function __construct(int $mismatchId)`
`19`	`19`	`*/`
`20`	`20`	`public function selector()`
`21`	`21`	`{`
`22`		`- return "#mismatch-$this->mismatchId .wikit-Dropdown";`
	`22`	`+ return "#mismatch-$this->mismatchId .cdx-select-vue";`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`/**`
`@@ -41,17 +41,17 @@ public function assert(Browser $browser)`
`41`	`41`	`public function elements()`
`42`	`42`	`{`
`43`	`43`	`return [`
`44`		`- '@select-menu' => '.wikit-Dropdown__select',`
`45`		`- '@selected' => '.wikit-Dropdown__selectedOption',`
`46`		`- '@menu-items' => '.wikit-Dropdown__menu'`
	`44`	`+ '@select-menu' => '.cdx-select-vue__handle',`
	`45`	`+ '@selected' => '.cdx-select-vue__handle',`
	`46`	`+ '@menu-items' => '.cdx-menu__listbox'`
`47`	`47`	`];`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`public function selectPosition(Browser $browser, int $position, string $option)`
`51`	`51`	`{`
`52`	`52`	`$browser->click('@select-menu')`
`53`	`53`	`->within('@menu-items', function ($menu) use ($position, $option) {`
`54`		`- $positionSelector = ".wikit-OptionsMenu__item:nth-child($position)";`
	`54`	`+ $positionSelector = ".cdx-menu-item:nth-child($position)";`
`55`	`55`	`$menu->assertSeeIn($positionSelector, $option)`
`56`	`56`	`->click($positionSelector);`
`57`	`57`	`})`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,8 @@ public function elements()`
`40`	`40`	`{`
`41`	`41`	`return [`
`42`	`42`	`'@form' => '#items-form',`
`43`		`- '@items-input' => '@form textarea'`
	`43`	`+ '@items-input' => '@form textarea',`
	`44`	`+ '@items-input-validation-message' => '@form .wikit-TextArea .wikit-ValidationMessage'`
`44`	`45`	`];`
`45`	`46`	`}`
`46`	`47`	`}`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public function elements()`
`47`	`47`	`{`
`48`	`48`	`return [`
`49`	`49`	`'@back-button' => '.back-button',`
`50`		`- '@confirmation-dialog' => '.confirmation-dialog',`
	`50`	`+ '@confirmation-dialog' => '#results-confirmation-dialog',`
`51`	`51`	`'@disable-confirmation' => '.disable-confirmation',`
`52`	`52`	`'@error-section' => '#error-section',`
`53`	`53`	`'@disable-confirmation-label' => '.disable-confirmation>.wikit-checkbox__label'`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ mix.ts('resources/js/app.ts', 'public/js')`
`32`	`32`	`compatConfig: {`
`33`	`33`	`MODE: 3,`
`34`	`34`	`COMPILER_V_ON_NATIVE: true,`
`35`		`- COMPILER_V_BIND_SYNC: false`
	`35`	`+ COMPILER_V_BIND_SYNC: true`
`36`	`36`	`}`
`37`	`37`	`}`
`38`	`38`	`}`