Skip to content

[stable21] Add missing waits and asserts in acceptance tests #25993

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
juliusknorr merged 9 commits into from
Mar 8, 2021
Merged

[stable21] Add missing waits and asserts in acceptance tests #25993

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
f3a4ec7
Add missing waits when finding elements in the acceptance tests
danxuliu Jan 17, 2021
d03bc3e
Verify WaitFor::element... results with an assertion
danxuliu Mar 5, 2021
34814ad
Fix delete user acceptance test
danxuliu Mar 6, 2021
4c98d29
Fix identation
danxuliu Mar 5, 2021
270db2f
Add locator for apps list
danxuliu Mar 5, 2021
1058abc
Find elements through the actor rather than the driver
danxuliu Mar 5, 2021
b086a0f
Add explicit locator for "Enable all" bundle button
danxuliu Mar 5, 2021
3a470c2
Assert also element visibility instead of just finding it
danxuliu Mar 5, 2021
8185c46
Find directly the label instead of falling back to it
danxuliu Mar 5, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add explicit locator for "Enable all" bundle button 
Instead of looking for the bundle button and then checking its value now
the expected value is included in the locator and the button is checked
similarly to other elements.

No "Disable all" locator was added as it was not currently needed
anywhere.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
  • Loading branch information
@danxuliu @backportbot
danxuliu authored and backportbot[bot] committed Mar 8, 2021
commit b086a0f007267be306ed9248b1d418fae527c648
12 changes: 7 additions & 5 deletions tests/acceptance/features/bootstrap/AppsManagementContext.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,10 +84,10 @@ public static function disableButtonForAnyApp() {
/**
* @return Locator
*/
public static function bundleButton($bundle) {
return Locator::forThe()->xpath("//div[@class='apps-header']/h2[normalize-space() = '$bundle']/input")->
public static function enableAllBundleButton($bundle) {
return Locator::forThe()->xpath("//div[@class='apps-header']/h2[normalize-space() = '$bundle']/input[@value='Enable all']")->
descendantOf(self::appsList())->
describedAs("Button to enable / disable bundles");
describedAs("Button to enable bundles");
}

/**
Expand Down Expand Up @@ -245,14 +245,16 @@ public function iSeeTheAppBundles() {
* @When /^I enable all apps from the "([^"]*)"$/
*/
public function iEnableAllAppsFromThe($bundle) {
$this->actor->find(self::bundleButton($bundle), 2)->click();
$this->actor->find(self::enableAllBundleButton($bundle), 2)->click();
}

/**
* @Given /^I see that the "([^"]*)" is disabled$/
*/
public function iSeeThatTheIsDisabled($bundle) {
PHPUnit\Framework\Assert::assertEquals('Enable all', $this->actor->find(self::bundleButton($bundle), 2)->getValue());
PHPUnit_Framework_Assert::assertTrue(
$this->actor->find(self::enableAllBundleButton($bundle), 2)->isVisible()
);
}

/**
Expand Down