Skip to content

Make tab nav page object more robust - #2472

Merged
wedaly merged 1 commit into
masterfrom
will/tab-nav-page-object-fix
Feb 5, 2014
Merged

Make tab nav page object more robust#2472
wedaly merged 1 commit into
masterfrom
will/tab-nav-page-object-fix

Conversation

@wedaly

@wedaly wedaly commented Feb 5, 2014

Copy link
Copy Markdown
Contributor

There was a failure in the bok-choy test suite this morning: https://jenkins.testeng.edx.org/job/edx-bok-choy-auto-master/39/console

From the traceback, it looks like the top-level tabs hadn't loaded before the test tried to click "Courseware". This fix adds a promise that at least one tab name is loaded before attempting to go to a particular tab.

@jzoldak

@jzoldak

jzoldak commented Feb 5, 2014

Copy link
Copy Markdown
Contributor

lgtm 👍
Curious why tab_names is a private property, seems like it might be useful to call from test code?

wedaly pushed a commit that referenced this pull request Feb 5, 2014
Make tab nav page object more robust
@wedaly
wedaly merged commit 0cd7a48 into master Feb 5, 2014
@wedaly
wedaly deleted the will/tab-nav-page-object-fix branch February 5, 2014 16:04
@wedaly

wedaly commented Feb 5, 2014

Copy link
Copy Markdown
Contributor Author

Yeah, it could be public. But that would be easy enough to change if anyone of the tests need to use it.

jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Mar 23, 2018
* Add ORA2 video upload option openedx#2375 (openedx#2417)

* Modified courseware page look and feel openedx#2377 (openedx#2409)

* Modified about page openedx#2379 (openedx#2413)

* Fix biz bugs. openedx#2404 (openedx#2406)

* Fix display width of popup. (openedx#2384)

(cherry picked from commit a43c935a575d15fcf629f0edeb81446c778cda95)

* Validate duplicate url-code.

* Fix bug, when course not found.

* Fix order of course as CourseOverview.

* Remove additional-info count from contract grid. openedx#2419 (openedx#2437)

* Fix order global course. openedx#2420 (openedx#2421)

* Add additional info register. openedx#2419 (openedx#2433)

* fix survey csv character encode problem openedx#2380 (openedx#2434)

* Fix bokchoy for LoginCodeEnabledBizSurveyTest. (openedx#2457)

* Fix register students confirm message. (openedx#2461)

* Add command to check playback_log. openedx#2438 (openedx#2445)

* Fix courseware page lookandfeel (openedx#2446, openedx#2439, openedx#2452, openedx#2453)

* Fix box-shadow of sequence-nav-button. openedx#2453 (openedx#2467)

* Fix bugs. openedx#2462 openedx#2463 (openedx#2464)

* Fix password message in register students page.

* Fix glass pane of processing when register additional item.

* Fix display width of popup. (openedx#2466)

* Fix min-width of sequence-nav. openedx#2468 (openedx#2469)

* Fix isRegistered javascript in about page openedx#2470 (openedx#2471)
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.

2 participants