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

Functional tests fail intermittently on Travis #902

Closed
oliverroick opened this issue Nov 7, 2016 · 7 comments
Closed

Functional tests fail intermittently on Travis #902

oliverroick opened this issue Nov 7, 2016 · 7 comments
Assignees

Comments

@oliverroick
Copy link
Member

Some builds fail due to time out exceptions in functional tests. Different tests cause the exceptions, but they all seem to be related because the tests eventually all break in the same line. This needs further analysis; I'm collecting links to failed builds for the time being:

@seav
Copy link
Contributor

seav commented Nov 9, 2016

If I understand the errors correctly, I think this is caused by the alert box disappearing after a while which leads to the timeout because the tests are looking for the alert box's close button which has disappeared in the meantime. A similar issue was fixed by @ian-ross in PR #546 by changing the target element to be located from the close button to a unique element in the expected page.

@ian-ross
Copy link
Contributor

ian-ross commented Nov 9, 2016

That's weird, because only "success" alerts are automatically dismissed after some time, and these cases are supposed to fail, i.e. to produce a persistent alert that needs to be explicitly dismissed by clicking on its close button.

@oliverroick
Copy link
Member Author

oliverroick commented Nov 9, 2016

The exception is raised here

        try:
            WebDriverWait(self.browser, 10).until_not(
                EC.presence_of_element_located(wait)
            )
        except TimeoutException:
            self.get_screenshot('exception')
            raise

Doesn't that mean the exception is raised because the alter is still present even though the close is clicked?

@ian-ross
Copy link
Contributor

ian-ross commented Nov 9, 2016

Yeah, looks like it. Don't know why clicking the button wouldn't make the alert go away though.

@oliverroick oliverroick self-assigned this Nov 9, 2016
@amplifi
Copy link
Contributor

amplifi commented Nov 10, 2016

Decision from 9 Nov 2016 Dev Team call: temporary workaround - comment out functional tests causing intermittent failures, full solution target - Sprint 11.

@oliverroick
Copy link
Member Author

Can we close this in favour of #919 ?

@ian-ross
Copy link
Contributor

Yes: I was thinking we'd close this one once you'd disabled the failing tests.

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

4 participants