Skip to content

Commit

Permalink
Fixed waiting for event/procedure/trigger drop
Browse files Browse the repository at this point in the history
The nothing2display element is always present, sometimes it is just
hidden. Wait for AJAX completion instead.

Signed-off-by: Michal Čihař <michal@cihar.com>
  • Loading branch information
nijel committed Sep 7, 2017
1 parent 037dde0 commit c3f8315
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
3 changes: 1 addition & 2 deletions test/selenium/PmaSeleniumDbEventsTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -247,9 +247,8 @@ public function testDropEvent()
"byClassName", "submitOK"
)->click();

$this->waitForElement("byId", "nothing2display");
$this->waitAjax();

sleep(1);
$result = $this->dbQuery(
"SHOW EVENTS WHERE Db='" . $this->database_name
. "' AND Name='test_event'"
Expand Down
3 changes: 1 addition & 2 deletions test/selenium/PmaSeleniumDbProceduresTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -179,9 +179,8 @@ public function testDropProcedure()
"byCssSelector", "button.submitOK"
)->click();

$this->waitForElement("byId", "nothing2display");
$this->waitAjax();

sleep(1);
$result = $this->dbQuery(
"SHOW PROCEDURE STATUS WHERE Db='" . $this->database_name . "'"
);
Expand Down
3 changes: 1 addition & 2 deletions test/selenium/PmaSeleniumDbTriggersTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -201,8 +201,7 @@ public function testDropTrigger()
"byCssSelector", "button.submitOK"
)->click();

$this->waitForElement("byId", "nothing2display");
sleep(1);
$this->waitAjax();

// test trigger
$this->dbQuery("INSERT INTO `test_table` (val) VALUES (1);");
Expand Down

0 comments on commit c3f8315

Please sign in to comment.