Opened 9 years ago

Closed 9 years ago

#11287 closed defect (fixed)

Dijit test suites failing due to missing timeouts

Reported by: haysmark Owned by: Douglas Hays
Priority: high Milestone: 1.5
Component: Dijit Version: 1.5.0rc1
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by haysmark)

While I was testing in Safari 5 over archive, I noticed the following tests would time out in the middle of loading:

  • Widget-placeAt
  • _Templated-widgetsInTemplate.html
  • dijit/tests/Tree.html
  • dijit/tests/Tree_with_JRS.html

It seems these tests do not have timeouts specified in their module.js files. All dijit entries in module.js files should have timeout 99999999 because the Web server can take arbitrarily long to send all of the requested data that dijits need.

Change History (3)

comment:1 Changed 9 years ago by haysmark

Description: modified (diff)

I did a quick grep and here is the list of tests that are in danger of timing out:

./i18n/module.js:		doh.registerUrl("dijit.tests.i18n.currency", dojo.moduleUrl("dijit", "tests/i18n/currency.html"));
./i18n/module.js:		doh.registerUrl("dijit.tests.i18n.date", dojo.moduleUrl("dijit", "tests/i18n/date.html"));
./i18n/module.js:		doh.registerUrl("dijit.tests.i18n.number", dojo.moduleUrl("dijit", "tests/i18n/number.html"));
./i18n/module.js:		doh.registerUrl("dijit.tests.i18n.textbox", dojo.moduleUrl("dijit", "tests/i18n/textbox.html"));
./i18n/module.js:		doh.registerUrl("dijit.tests.i18n.time", dojo.moduleUrl("dijit", "tests/i18n/time.html"));
./i18n/module.js:		doh.registerUrl("dijit.tests.i18n.digit", dojo.moduleUrl("dijit", "tests/i18n/digit.html"));
./infrastructure-module.js:	doh.registerUrl("dijit.tests._Widget-lifecycle", dojo.moduleUrl("dijit", "tests/_Widget-lifecycle.html"));
./infrastructure-module.js:	doh.registerUrl("dijit.tests._Widget-attr", dojo.moduleUrl("dijit", "tests/_Widget-attr.html"));
./infrastructure-module.js:	doh.registerUrl("dijit.tests._Widget-subscribe", dojo.moduleUrl("dijit", "tests/_Widget-subscribe.html"));
./infrastructure-module.js:	doh.registerUrl("dijit.tests.Widget-placeAt", dojo.moduleUrl("dijit", "tests/Widget-placeAt.html"));
./infrastructure-module.js:	doh.registerUrl("dijit.tests._Templated", dojo.moduleUrl("dijit", "tests/_Templated.html"));
./infrastructure-module.js:	doh.registerUrl("dijit.tests._Templated-widgetsInTemplate", dojo.moduleUrl("dijit", "tests/_Templated-widgetsInTemplate.html"));
./infrastructure-module.js:	doh.registerUrl("dijit.tests._Container", dojo.moduleUrl("dijit", "tests/_Container.html"));
./layout/module.js:	doh.registerUrl("dijit.tests.layout.StackContainer", dojo.moduleUrl("dijit", "tests/layout/nestedStack.html")); 
./tree/module.js:	doh.registerUrl("dijit.tests.Tree", dojo.moduleUrl("dijit", "tests/Tree.html"));
./tree/module.js:	doh.registerUrl("dijit.tests.Tree_with_JRS", dojo.moduleUrl("dijit", "tests/Tree_with_JRS.html"));
./_base/module.js:	doh.registerUrl("dijit.tests._base.manager", dojo.moduleUrl("dijit", "tests/_base/manager.html"));
./_base/module.js:	doh.registerUrl("dijit.tests._base.tabindex", dojo.moduleUrl("dijit", "tests/_base/tabindex.html"));
./_base/module.js:	doh.registerUrl("dijit.tests._base.wai", dojo.moduleUrl("dijit", "tests/_base/wai.html"));
./_base/module.js:	doh.registerUrl("dijit.tests._base.place", dojo.moduleUrl("dijit", "tests/_base/place.html"));

comment:2 Changed 9 years ago by Douglas Hays

Milestone: tbd1.5
Owner: set to Douglas Hays
Status: newassigned
Version: 1.5.0b21.5.0rc1

comment:3 Changed 9 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed

(In [22316]) Fixes #11287. Add explicit test timeouts to registerUrl calls in dijit that were using the default.

Note: See TracTickets for help on using tickets.