X Tutup
Skip to content

Makes acceptance test more stable with Travis#756

Merged
aaltat merged 3 commits intorobotframework:masterfrom
aaltat:make_acceptance_test_more_stable
Feb 8, 2017
Merged

Makes acceptance test more stable with Travis#756
aaltat merged 3 commits intorobotframework:masterfrom
aaltat:make_acceptance_test_more_stable

Conversation

@aaltat
Copy link
Contributor

@aaltat aaltat commented Feb 7, 2017

There is quite often a random failures with test that open new window.
It seem that with Travis is takes little bit of time to open a window
and this fix wait the windows to open before test is performed.

Partially fixes #734

There is quite often a random failures with test that open new window.
It seem that with Travis is takes little bit of time to open a window
and this fix wait the windows to open before test is performed.

Partially fixes #734
Wait Untill Window Is Open
${titles} = Get Window Titles
${status} = Evaluate len(${titles}) > 1
Should Be True ${status}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait Until Window Is Open sounds like a keyword that could be in the library itself. Also, there's only one L in Until.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@aaltat
Copy link
Contributor Author

aaltat commented Feb 8, 2017

There has been some complaints from users too, example #750 and we should build such Wait* keyword for window selection too.

I will fix the typo later today.

@aaltat
Copy link
Contributor Author

aaltat commented Feb 8, 2017

Created #757 to create the new keyword

@aaltat aaltat merged commit b40a2c7 into robotframework:master Feb 8, 2017
@aaltat aaltat deleted the make_acceptance_test_more_stable branch February 18, 2017 22:04
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

Successfully merging this pull request may close these issues.

There are random failures with acceptance test when running in CI + Chrome

2 participants

X Tutup