Skip to content

Commit

Permalink
Merge pull request #7187 from LeeDr/tryToFixVisTests
Browse files Browse the repository at this point in the history
Add try in 2 visualize page methods that have failed.
  • Loading branch information
spalger committed May 12, 2016
2 parents b93578d + 670e878 commit cb52c70
Showing 1 changed file with 11 additions and 6 deletions.
17 changes: 11 additions & 6 deletions test/support/pages/visualize_page.js
Original file line number Diff line number Diff line change
Expand Up @@ -226,11 +226,14 @@ export default (function () {
},

selectField: function selectField(fieldValue) {
return this.remote
.setFindTimeout(defaultFindTimeout)
// the css below should be more selective
.findByCssSelector('option[label="' + fieldValue + '"]')
.click();
var self = this;
return common.try(function () {
return self.remote
.setFindTimeout(defaultFindTimeout)
// the css below should be more selective
.findByCssSelector('option[label="' + fieldValue + '"]')
.click();
});
},

orderBy: function orderBy(fieldValue) {
Expand Down Expand Up @@ -341,10 +344,12 @@ export default (function () {
var self = this;
common.debug('clickVisualizationByLinkText(' + vizName + ')');

return this.remote
return common.try(function () {
return self.remote
.setFindTimeout(defaultFindTimeout)
.findByLinkText(vizName)
.click();
});
},

// this starts by clicking the Load Saved Viz button, not from the
Expand Down

0 comments on commit cb52c70

Please sign in to comment.