Skip to content

Commit

Permalink
Another adjustment to handle timing issue on tests
Browse files Browse the repository at this point in the history
ref #1702
-timing issue with the test still exists in #2729 since
 .description is added to the dom after the fact via js
  • Loading branch information
jaswilli committed May 11, 2014
1 parent e4799e3 commit ef1bc05
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions core/test/functional/admin/settings_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -159,15 +159,15 @@ CasperTest.begin('Ensure image upload modals display correctly', 6, function sui
this.click('#general .js-modal-logo');
}, casper.failOnTimeout(test, 'waitForOpaque #general timed out'));

casper.waitForSelector('#modal-container .modal-content .js-drop-zone', assertImageUploaderModalThenClose,
casper.waitForSelector('#modal-container .modal-content .js-drop-zone .description', assertImageUploaderModalThenClose,
casper.failOnTimeout(test, 'No upload logo modal container appeared'));

// Test Blog Cover Upload Button
casper.then(function () {
this.click('#general .js-modal-cover');
});

casper.waitForSelector('#modal-container .modal-content .js-drop-zone', assertImageUploaderModalThenClose,
casper.waitForSelector('#modal-container .modal-content .js-drop-zone .description', assertImageUploaderModalThenClose,
casper.failOnTimeout(test, 'No upload cover modal container appeared'));
});

Expand Down

0 comments on commit ef1bc05

Please sign in to comment.