Skip to content

[Behat] IBX-11133 Added enabled option for Help center#1808

Open
pawlakadrian wants to merge 3 commits into4.6from
IBX-11133-help-center-user-settings
Open

[Behat] IBX-11133 Added enabled option for Help center#1808
pawlakadrian wants to merge 3 commits into4.6from
IBX-11133-help-center-user-settings

Conversation

@pawlakadrian
Copy link
Contributor

@pawlakadrian pawlakadrian commented Dec 30, 2025

🎫 Issue IBX-11133

Related PRs:

Description:

Added additional options (enable/disable) for help center

For QA:

Documentation:

Copy link
Contributor

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some nitpicks :)

public function enableHelpCenter(): void
{
$this->contentActionsMenu->verifyIsLoaded();
$this->getHTMLPage()->find($this->getLocator('helpCenterValueDropdown'))->click();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would add timeout here - just to be sure.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested multiple times with success. Before I had some problems with locators here, but after adding $this->ibexaDropdown->verifyIsLoaded(); test is stable.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, im just thinking about a rare case when $this->getLocator('helpCenterValueDropdown') is not found.

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants