Skip to content
This repository was archived by the owner on May 10, 2019. It is now read-only.
This repository was archived by the owner on May 10, 2019. It is now read-only.

Add an indicator that the loading screen has been displayed for testability #4227

@davehunt

Description

@davehunt

We have a number of test automation suites that complete sign-in via Persona, which have started to fail in Firefox when using Selenium 2.48. This is due to us not waiting for the loading screen to be hidden before attempting to interact with the content. It means that clicks are received by the loading screen instead of the submit button, for example. Usually we would wait for the loading screen to be hidden, but the loading screen is not initially visible after the previous interaction so our wait condition would be met immediately. We could first wait for the loading screen to be displayed, but as it can flash up for a fraction of a second we might miss it and time out (depending on how you run your tests, there can be network latency).

I think an ideal solution would be to indicate in the document that the loading screen has been displayed but is no longer displayed. This could be done most simply by adding a class to an element. I'm open to other suggestions. For now, I will add a sleep in our automation.

This is the change in Selenium that caused this: SeleniumHQ/selenium@0eec81d
The change has caused others to see similar issues: SeleniumHQ/selenium#1202

Note that this is not a bug in Selenium, it is an enhancement that has highlighted an issue with our Persona automation.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions