You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
On e2e sites, we usually have license data key and email already defined in wp-config, so if the zip has a problem with the license, e2e will never catch that now.
usually, when running on branch zip, we won't have license data in zip and will depend on wp-config.
Proposed solution:
=> Having a
test to remove the key/email from wp-config in precondition
=> Test run to install, activate WPR and access the settings page ==> no problem is there (thus no license issue and if license issue is there log error and fail if fields are empty, o.w click validate button on license validation page)
=> Test run only when tag @prerelease is used
=> When tag @prerelease is used, we need to run this test + smoke test with single cmd i.e npm run test:prerelease
=> After the test, we need to add back the key/email that was removed (so any run on PR works smoothly)
The text was updated successfully, but these errors were encountered:
On e2e sites, we usually have license data key and email already defined in wp-config, so if the zip has a problem with the license, e2e will never catch that now.
usually, when running on branch zip, we won't have license data in zip and will depend on wp-config.
Proposed solution:
=> Having a
test to remove the key/email from wp-config in precondition
=> Test run to install, activate WPR and access the settings page ==> no problem is there (thus no license issue and if license issue is there log error and fail if fields are empty, o.w click validate button on license validation page)
=> Test run only when tag @prerelease is used
=> When tag @prerelease is used, we need to run this test + smoke test with single cmd i.e npm run test:prerelease
=> After the test, we need to add back the key/email that was removed (so any run on PR works smoothly)
The text was updated successfully, but these errors were encountered: