Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The tests are plagued by heisenbugs #8

Open
regebro opened this issue Aug 2, 2012 · 2 comments
Open

The tests are plagued by heisenbugs #8

regebro opened this issue Aug 2, 2012 · 2 comments

Comments

@regebro
Copy link
Contributor

regebro commented Aug 2, 2012

Many tests will not work reliably. Some tests will always fail if you reload the page. Some tests will fail every second reload, and pass every second reload. All tests will however run if run in isolation. Trying to debug the issue (in any case including alerts or having a debugger) will make the tests pass.

I suspect timing issues, but adding delays seem to make no difference.

I suspect a JS/jQuery/qUnit guru is needed to look at this.

@thet
Copy link
Member

thet commented Oct 29, 2013

i tried to fix this, but without any success...
i assumed, that the cause of the heisenbugs are the async ajax calls of the widget (writing this, i think, maybe we should use synchronous ajax calls for the tests...).
my experiments are in this branch:
https://github.com/collective/jquery.recurrenceinput.js/commits/fix-heisenbugs

@plone-foundation
Copy link

Quoting Johannes Raggam (2013-10-29 11:18:36)

i tried to fix this, but without any success...
i assumed, that the cause of the heisenbugs are the async ajax calls of the
widget (writing this, i think, maybe we should use synchronous ajax calls for
the tests...).
my experiments are in this branch:
https://github.com/collective/jquery.recurrenceinput.js/commits/fix-heisenbugs

switch to mocha and use sinon library to mock ajax requests. we do this already
in many places in mockup.

Rok Garbas - http://www.garbas.si

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants