Skip to content

Commit

Permalink
Merge pull request #148 from plone/drop-empty-keyword
Browse files Browse the repository at this point in the history
Drop empty keyword
  • Loading branch information
mauritsvanrees authored Jun 28, 2023
2 parents 78bb80c + f69a791 commit 56a1288
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 10 deletions.
2 changes: 2 additions & 0 deletions news/147.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Remove unused and empty keyword that was displaying an error.
[gforcada]
10 changes: 0 additions & 10 deletions src/plone/app/robotframework/keywords.robot
Original file line number Diff line number Diff line change
Expand Up @@ -81,16 +81,6 @@ Log in as site owner
... with all the rights and privileges of that user.
Log in ${SITE_OWNER_NAME} ${SITE_OWNER_PASSWORD}

Log in as test user with role
[Arguments] ${usrid} ${role}

# We need a generic way to login with a user that has one or more roles.

# Do we need to be able to assign multiple roles at once?

# Do we need to assign roles to arbitray users or is it sufficient if we
# always assign those roles to the test user?

Log out
Go to ${PLONE_URL}/logout
Page Should Contain Element css=#login-form
Expand Down

0 comments on commit 56a1288

Please sign in to comment.