zydronium/manage.get.gov
1
0
Fork 0
mirror of https://github.com/cisagov/manage.get.gov.git synced 2025-07-22 18:56:15 +02:00
Code Issues Projects Releases Packages Wiki Activity

added logic to not display blue info box in certain condition, and added test

Browse source
This commit is contained in:
David Kennedy 2024-11-18 13:51:39 -05:00
parent 9ee9e94405
commit 8f18a409ea
No known key found for this signature in database
GPG key ID: 6528A5386E66B96B
2 changed files with 37 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

30
src/registrar/templates/domain_detail.html
View file

@ -58,22 +58,24 @@
{% include "includes/domain_dates.html" %} {% include "includes/domain_dates.html" %}
{% if is_portfolio_admin and not is_domain_manager %} {% if analyst_action != 'edit' or analyst_action_location != domain.pk %}
<div class="usa-alert usa-alert--info usa-alert--slim"> {% if is_portfolio_admin and not is_domain_manager and not is_analyst_managing_domain %}
<div class="usa-alert__body"> <div class="usa-alert usa-alert--info usa-alert--slim">
<p class="usa-alert__text "> <div class="usa-alert__body">
To manage information for this domain, you must add yourself as a domain manager. <p class="usa-alert__text ">
</p> To manage information for this domain, you must add yourself as a domain manager.
</p>
</div>
</div> </div>
</div> {% elif is_portfolio_user and not is_domain_manager and not is_analyst_managing_domain %}
{% elif is_portfolio_user and not is_domain_manager %} <div class="usa-alert usa-alert--info usa-alert--slim">
<div class="usa-alert usa-alert--info usa-alert--slim"> <div class="usa-alert__body">
<div class="usa-alert__body"> <p class="usa-alert__text ">
<p class="usa-alert__text "> You don't have access to manage {{domain.name}}. If you need to make updates, contact one of the listed domain managers.
You don't have access to manage {{domain.name}}. If you need to make updates, contact one of the listed domain managers. </p>
</p> </div>
</div> </div>
</div> {% endif %}
{% endif %} {% endif %}

21
src/registrar/tests/test_views_domain.py
View file

@ -323,6 +323,27 @@ class TestDomainDetail(TestDomainOverview):
self.assertContains(detail_page, "noinformation.gov") self.assertContains(detail_page, "noinformation.gov")
self.assertContains(detail_page, "Domain missing domain information") self.assertContains(detail_page, "Domain missing domain information")
def test_domain_detail_with_analyst_managing_domain(self):
"""Test that domain management page returns 200 and does not display
blue error message when an analyst is managing the domain"""
with less_console_noise():
# have to use staff user for this test
staff_user = create_user()
# staff_user.save()
self.client.force_login(staff_user)
# need to set the analyst_action and analyst_action_location
# in the session to emulate user clicking Manage Domain
# in the admin interface
session = self.client.session
session["analyst_action"] = "edit"
session["analyst_action_location"] = self.domain.id
session.save()
detail_page = self.client.get(reverse("domain", kwargs={"pk": self.domain.id}))
self.assertNotContains(detail_page, "To manage information for this domain, you must add yourself as a domain manager.")
@less_console_noise_decorator @less_console_noise_decorator
@override_flag("organization_feature", active=True) @override_flag("organization_feature", active=True)
def test_domain_readonly_on_detail_page(self): def test_domain_readonly_on_detail_page(self):