Page MenuHomePhabricator

Error in "add link.link inspector can be used to accept/reject links and save an article."
Closed, InvalidPublic

Description

Example link: https://integration.wikimedia.org/ci/job/wmf-quibble-selenium-php74-docker/1005/console

14:08:17 [0-1] Error in "add link.link inspector can be used to accept/reject links and save an article."
14:09:01 Error: element (".mw-ge-postEditDrawer") still not existing after 30000ms
14:09:01     at /workspace/src/extensions/GrowthExperiments/node_modules/webdriverio/build/commands/browser/waitUntil.js:66:23
14:09:01     at async Element.wrapCommandFn (/workspace/src/extensions/GrowthExperiments/node_modules/@wdio/utils/build/shim.js:137:29)
14:09:01     at async Element.elementErrorHandlerCallbackFn (/workspace/src/extensions/GrowthExperiments/node_modules/webdriverio/build/middlewares.js:24:32)
14:09:01     at async Element.wrapCommandFn (/workspace/src/extensions/GrowthExperiments/node_modules/@wdio/utils/build/shim.js:137:29)
14:09:01     at async Element.wrapCommandFn (/workspace/src/extensions/GrowthExperiments/node_modules/@wdio/utils/build/shim.js:137:29)
14:09:01     at async Element.elementErrorHandlerCallbackFn (/workspace/src/extensions/GrowthExperiments/node_modules/webdriverio/build/middlewares.js:24:32)
14:09:01     at async Element.wrapCommandFn (/workspace/src/extensions/GrowthExperiments/node_modules/@wdio/utils/build/shim.js:137:29)
14:09:01     at async HomepagePage.waitForDisplayedAndClickable (/workspace/src/extensions/GrowthExperiments/tests/selenium/pageobjects/homepage.page.js:109:3)
14:09:01     at async HomepagePage.waitForPostEditDialog (/workspace/src/extensions/GrowthExperiments/tests/selenium/pageobjects/homepage.page.js:105:3)
14:09:01     at async Context.<anonymous> (/workspace/src/extensions/GrowthExperiments/tests/selenium/specs/addlink.js:49:3)
14:09:01 [0-1] RETRYING after 3s in chrome - /tests/selenium/specs/addlink.js

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

@xSavitar is it just that patch? I commented on it.

Thanks Kosta. There is a way forward in an attempt to fix this. I'll remove the CI tag now. Will update here if the issue gets fixed.

Cc @daniel

@xSavitar is it just that patch? I commented on it.

Seems it's just the patch, not the entire CI. We seem to have found the issue and trying to resolve now. Will resolve this task when we fix it. Thanks!

@xSavitar is it just that patch? I commented on it.

Seems it's just the patch, not the entire CI. We seem to have found the issue and trying to resolve now. Will resolve this task when we fix it. Thanks!

Then, I'm marking this task as Invalid as it seems like a success story where the test correctly caught a regression 🤗

As an update, we were able to resolve the issue. Thanks @kostajh for linking to the selenium test video. It helped us dig into the issue and do some findings.