Skip to content

Fix activity tests again - #22291

Merged
rullzer merged 1 commit into
masterfrom
bugfix/22117/fix-activity-tests-again
Aug 18, 2020
Merged

Fix activity tests again#22291
rullzer merged 1 commit into
masterfrom
bugfix/22117/fix-activity-tests-again

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Regression from #22117

Signed-off-by: Joas Schilling <coding@schilljs.com>
@faily-bot

faily-botBot commented Aug 18, 2020

Copy link
Copy Markdown

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 31937: failure

acceptance-header

  • tests/acceptance/features/header.feature:31
Show full log
 Scenario: users from other groups are not seen in the contacts menu when autocompletion is restricted within the same group # /drone/src/tests/acceptance/features/header.feature:31
Given I am logged in as the admin # LoginPageContext::iAmLoggedInAsTheAdmin()
And I visit the settings page # SettingsMenuContext::iVisitTheSettingsPage()
And I open the "Sharing" section of the "Administration" group # AppNavigationContext::iOpenTheSectionOf()
And I enable restricting username autocompletion to groups # SettingsContext::iEnableRestrictingUsernameAutocompletionToGroups()
And I see that username autocompletion is restricted to groups # SettingsContext::iSeeThatUsernameAutocompletionIsRestrictedToGroups()
When I open the Contacts menu # ContactsMenuContext::iOpenTheContactsMenu()
Then I see that the Contacts menu is shown # ContactsMenuContext::iSeeThatTheContactsMenuIsShown()
And I see that the contact "user0" in the Contacts menu is not shown # ContactsMenuContext::iSeeThatTheContactInTheContactsMenuIsNotShown()
Failed asserting that true is false.
And I see that the contact "admin" in the Contacts menu is not shown # ContactsMenuContext::iSeeThatTheContactInTheContactsMenuIsNotShown()

@rullzer
rullzer merged commit 5d8242c into masterAug 18, 2020
@rullzer
rullzer deleted the bugfix/22117/fix-activity-tests-again branch August 18, 2020 18:00
Sign up for freeto 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.

3 participants

@nickvergessen@rullzer@ChristophWurst