Last modified: 2013-11-19 14:21:52 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T58567, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 56567 - reasoning about javascript loading and test failures
reasoning about javascript loading and test failures
Status: RESOLVED WORKSFORME
Product: VisualEditor
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: James Forrester
: browser-test-bug
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-04 15:38 UTC by Chris McMahon
Modified: 2013-11-19 14:21 UTC (History)
6 users (show)

See Also:
Web browser: Google Chrome
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
abnormal load.php requests (82.27 KB, image/png)
2013-11-04 15:38 UTC, Chris McMahon
Details
gadget code in VE in normal operation (172.60 KB, image/png)
2013-11-04 15:38 UTC, Chris McMahon
Details

Description Chris McMahon 2013-11-04 15:38:09 UTC
Created attachment 13690 [details]
abnormal load.php requests

this is essentially the same problem as shown with UploadWizard in https://bugzilla.wikimedia.org/show_bug.cgi?id=56483

When run in the Sauce Labs environment, tests for VisualEditor for Chrome display many many calls to ResourceLoader that are not shown in normal operation, nor in a browser test in a local environment.  

At least some of these calls to load.php fail with javascript errors, thus causing VE to fail and the test to fail.  

One recent example is shown at https://wmf.ci.cloudbees.com/job/VisualEditor-en.wikipedia.beta.wmflabs.org-linux-chrome/117/testReport/junit/(root)/VisualEditor%20Links/Enter_non_existing_link/

Here we see two distinct calls for the same code at ext.echo.alert, the second of which fails, see screen shot.  In detail, the first call succeeds, the second fails:

[200 OK] GET http://bits.beta.wmflabs.org/en.wikipedia.beta.wmflabs.org/load.php?debug=false&lang=en&modules=ext.echo.alert%2Cbase%2Cdesktop%2Coverlay&skin=vector&version=20131021T193233Z&*

[200 OK] GET http://bits.beta.wmflabs.org/en.wikipedia.beta.wmflabs.org/load.php?debug=false&lang=en&modules=ext.echo.alert%2Cbase%2Cdesktop%2Coverlay&skin=vector&version=20131021T193233Z&*
Uncaught SyntaxError: Unexpected token ILLEGAL 

Also in this particular run of the test find VE loading up code for ArticleFeedback and for Gadget code for DelAutoDropdown.  VE loading gadget code in normal operation has been documented, see screen shot for invisible_iframe
Comment 1 Chris McMahon 2013-11-04 15:38:52 UTC
Created attachment 13691 [details]
gadget code in VE in normal operation
Comment 3 Roan Kattouw 2013-11-06 01:36:58 UTC
Unless you can reproduce this outside of SauceLabs, I'm inclined to believe this is simply Chrome in SauceLabs being broken.
Comment 4 Chris McMahon 2013-11-19 14:21:52 UTC
This is no longer happening.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links