mirror of
https://github.com/cisagov/manage.get.gov.git
synced 2025-08-16 22:44:11 +02:00
fix broken tests
This commit is contained in:
parent
fd6ca004f2
commit
6856da38f1
1 changed files with 8 additions and 12 deletions
|
@ -915,9 +915,9 @@ class TestPortfolio(WebTest):
|
||||||
# Assert text within the page is correct
|
# Assert text within the page is correct
|
||||||
self.assertContains(response, "First Last")
|
self.assertContains(response, "First Last")
|
||||||
self.assertContains(response, self.user.email)
|
self.assertContains(response, self.user.email)
|
||||||
self.assertContains(response, "Basic access")
|
self.assertContains(response, "Basic")
|
||||||
self.assertContains(response, "No access")
|
self.assertContains(response, "No access")
|
||||||
self.assertContains(response, "View all members")
|
self.assertContains(response, "Viewer")
|
||||||
self.assertContains(response, "This member does not manage any domains.")
|
self.assertContains(response, "This member does not manage any domains.")
|
||||||
|
|
||||||
# Assert buttons and links within the page are correct
|
# Assert buttons and links within the page are correct
|
||||||
|
@ -933,15 +933,11 @@ class TestPortfolio(WebTest):
|
||||||
"""Test that user can access the member page with edit_members permission"""
|
"""Test that user can access the member page with edit_members permission"""
|
||||||
|
|
||||||
# Arrange
|
# Arrange
|
||||||
# give user permissions to view AND manage members
|
# give user admin role, which includes edit_members
|
||||||
permission_obj, _ = UserPortfolioPermission.objects.get_or_create(
|
permission_obj, _ = UserPortfolioPermission.objects.get_or_create(
|
||||||
user=self.user,
|
user=self.user,
|
||||||
portfolio=self.portfolio,
|
portfolio=self.portfolio,
|
||||||
roles=[UserPortfolioRoleChoices.ORGANIZATION_ADMIN],
|
roles=[UserPortfolioRoleChoices.ORGANIZATION_ADMIN],
|
||||||
additional_permissions=[
|
|
||||||
UserPortfolioPermissionChoices.EDIT_REQUESTS,
|
|
||||||
UserPortfolioPermissionChoices.EDIT_MEMBERS,
|
|
||||||
],
|
|
||||||
)
|
)
|
||||||
|
|
||||||
# Verify the page can be accessed
|
# Verify the page can be accessed
|
||||||
|
@ -952,9 +948,9 @@ class TestPortfolio(WebTest):
|
||||||
# Assert text within the page is correct
|
# Assert text within the page is correct
|
||||||
self.assertContains(response, "First Last")
|
self.assertContains(response, "First Last")
|
||||||
self.assertContains(response, self.user.email)
|
self.assertContains(response, self.user.email)
|
||||||
self.assertContains(response, "Admin access")
|
self.assertContains(response, "Admin")
|
||||||
self.assertContains(response, "View all requests plus create requests")
|
self.assertContains(response, "Creator")
|
||||||
self.assertContains(response, "View all members plus manage members")
|
self.assertContains(response, "Manager")
|
||||||
self.assertContains(
|
self.assertContains(
|
||||||
response, 'This member does not manage any domains. To assign this member a domain, click "Manage"'
|
response, 'This member does not manage any domains. To assign this member a domain, click "Manage"'
|
||||||
)
|
)
|
||||||
|
@ -1028,9 +1024,9 @@ class TestPortfolio(WebTest):
|
||||||
# Assert text within the page is correct
|
# Assert text within the page is correct
|
||||||
self.assertContains(response, "Invited")
|
self.assertContains(response, "Invited")
|
||||||
self.assertContains(response, portfolio_invitation.email)
|
self.assertContains(response, portfolio_invitation.email)
|
||||||
self.assertContains(response, "Basic access")
|
self.assertContains(response, "Basic")
|
||||||
self.assertContains(response, "No access")
|
self.assertContains(response, "No access")
|
||||||
self.assertContains(response, "View all members")
|
self.assertContains(response, "Viewer")
|
||||||
self.assertContains(response, "This member does not manage any domains.")
|
self.assertContains(response, "This member does not manage any domains.")
|
||||||
|
|
||||||
# Assert buttons and links within the page are correct
|
# Assert buttons and links within the page are correct
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue