Skip to content

Commit

Permalink
MDL-56210 behat: Test for disabled button later
Browse files Browse the repository at this point in the history
With phantomjs, dom is not updated by the time
next step is executed, so suffle it to ensure
this scenario pass
  • Loading branch information
Rajesh Taneja committed Oct 5, 2016
1 parent 77371e2 commit 86ebbed
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 14 deletions.
12 changes: 0 additions & 12 deletions lib/form/form.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,6 @@ if (typeof M.form.dependencyManager === 'undefined') {
this._hides = {};
this._dirty = {};

// Let behat know that JS is pending.
M.util.js_pending('mform-dependency-manager-' + this.get('form').get('id'));

// Setup event handlers.
Y.Object.each(this.get('dependencies'), function(value, i) {
var elements = this.elementsByName(i);
Expand Down Expand Up @@ -59,9 +56,6 @@ if (typeof M.form.dependencyManager === 'undefined') {
}, this);

this.updateAllDependencies();

// Let behat know that JS is complete.
M.util.js_complete('mform-dependency-manager-' + this.get('form').get('id'));
},

/**
Expand Down Expand Up @@ -595,12 +589,6 @@ M.form.initFormDependencies = function(Y, formid, dependencies) {
*/
M.form.updateFormState = function(formid) {
if (formid in M.form.dependencyManagers) {
// Let behat know that JS is pending.
M.util.js_pending('mform-dependency-manager-' + formid);

M.form.dependencyManagers[formid].updateAllDependencies();

// Let behat know that JS is complete.
M.util.js_complete('mform-dependency-manager-' + formid);
}
};
3 changes: 1 addition & 2 deletions mod/lti/tests/behat/addtool.feature
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,10 @@ Feature: Add tools
And I turn editing mode on
And I add a "Teaching Tool 1" to section "1"
# For tool that does not support Content-Item message type, the Select content button must be disabled.
And I wait until the page is ready
And the "Select content" "button" should be disabled
And I set the field "Activity name" to "Test tool activity 1"
And I click on "Show more..." "link"
And I set the field "Launch container" to "Embed"
And the "Select content" "button" should be disabled
And I press "Save and return to course"
And I open "Test tool activity 1" actions menu
And I choose "Edit settings" in the open action menu
Expand Down

0 comments on commit 86ebbed

Please sign in to comment.