E2E testing and make synchronization between old and new settings page#2927
Merged
akzmoudud merged 102 commits intoenhance/map-all-the-settings-storage-key-based-on-settings-mapperfrom Dec 18, 2025
Conversation
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…rage-key-based-on-settings-mapper' into enhance/new-admin-settings-fixing-and-e2e-testing the commit.
…d fixing this page bugs
…xing-and-e2e-testing
649416b to
4ddbf12
Compare
> marketplace settings
…ttings tests - Add new 'checkbox' field type to adminSettingsPageNew.ts to handle DOM elements with 'enabled' class - Update selector logic to click on visible label while checking state from hidden input element - Add Enable Single Seller Mode field to old dataset in generalMarketplace.spec.ts - Update test values and selectors for marketplace settings synchronization testing - Add debug logging for checkbox state verification
- Fix async iteration by replacing forEach with for...of loop to properly await operations - Increase element visibility timeout from 10s to 15s for old admin page - Add explicit wait for element attachment before scrolling - Improve page stabilization with longer waits for old admin interface - Add conditional navigation check to prevent unnecessary operations Resolves timeout errors: - TimeoutError on #dokan_settings_general selector - Test ended error due to unwaited async operations
- Add reloadUrl method to AdminSettingsPageNew for forced page reload - Fix missing await statements in updateSettings method - Remove unnecessary waitForTimeout calls to improve test speed - Restructure General settings migration test with two separate tests: * Test 1: New settings to old settings synchronization * Test 2: Old settings to new settings synchronization - Optimize element visibility checks and navigation timing This improves test reliability and reduces overall test execution time.
…ngs-mapper' into enhance/new-admin-settings-fixing-and-e2e-testing
…led and fix option names
* in progress * Ai Content generation page e2e
* e2e test for generalMarketplace page * Vendor Onboarding Page * pages of vendor section * Set ID of showHide fields * Social Onboarding Page * Vendor Capabilities Page * Vendor Subscription Page * Single Product Milti-vendor Page * Moderation RMA Page * Updated VendorSubscription Page * Moderation pages templets * Moderation Store Support Page * Moderation Live Chat Page * Moderation Abuse Report Page * Implement mapping of delivery time page between new and old page * Implement mapping of new shipment settings page with old setting page * Shipment settings Page * Add SMS gateway and Email vendor verification settings mapper * Update mapper for change Ui design --------- Co-authored-by: Moudud Ahmed <abdulkader225704@gmail.com> Co-authored-by: Kamruzzaman <kzamanbn@gmail.com>
… enhance/general-setting-page-e2e-testing
…2e-testing Enhance/general setting page e2e testing
* Add SMS gateway and Email vendor verification settings mapper * EmailAndSmsGatewayPagee2eTest
* Add SMS gateway and Email vendor verification settings mapper * EmailAndSmsGatewayPagee2eTest * e2etesting in progress * appearance_tab_e2e_test
* Vendor verification page e2e test * Vendor Verification page e2e test
* Appearance > Color Customizer mapper * New ui doc link updated In Ai Assist page
892007d to
39364cc
Compare
cf013b7
into
enhance/map-all-the-settings-storage-key-based-on-settings-mapper
1 check failed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All Submissions:
Changes proposed in this Pull Request:
Related Pull Request(s)
Closes
How to test the changes in this Pull Request:
Changelog entry
Title
Detailed Description of the pull request. What was previous behaviour
and what will be changed in this PR.
Before Changes
Describe the issue before changes with screenshots(s).
After Changes
Describe the issue after changes with screenshot(s).
Feature Video (optional)
Link of detailed video if this PR is for a feature.
PR Self Review Checklist:
FOR PR REVIEWER ONLY: