Merge pull request #715 from cisagov/sspj/remove-whoami

Remove whoami page

src/.pa11yci

@@ -1,14 +1,11 @@
 {
     "defaults": {
         "concurrency": 1,
-        "timeout": 30000,
-        "hideElements": "a[href='/whoami/']"
-
+        "timeout": 30000
     },
     "urls": [
         "http://localhost:8080/",
         "http://localhost:8080/health/",
-        "http://localhost:8080/whoami/",
         "http://localhost:8080/register/",
         "http://localhost:8080/register/organization/",
         "http://localhost:8080/register/org_federal/",

src/registrar/config/urls.py

@@ -45,7 +45,6 @@ for step, view in [
 
 urlpatterns = [
     path("", views.index, name="home"),
-    path("whoami/", views.whoami, name="whoami"),
     path("admin/", admin.site.urls),
     path(
         "application/<id>/edit/",

src/registrar/templates/base.html

@@ -152,7 +152,7 @@
         <ul class="usa-nav__primary usa-accordion">
           <li class="usa-nav__primary-item">
             {% if user.is_authenticated %}
-            <a href="{% url 'whoami' %}"><span>{{ user.email }}</span></a>
+            <span>{{ user.email }}</span>
           </li>
           <li class="usa-nav__primary-item display-flex flex-align-center">
             <span class="text-base"> | </span>

src/registrar/templates/whoami.html

@@ -1,10 +0,0 @@
-{% extends 'base.html' %}
-
-{% block title %} Hello {% endblock %}
-{% block content %}
-<main id="main-content" class="grid-container">
-  <p> Hello {{ user.last_name|default:"No last name given" }}, {{ user.first_name|default:"No first name given" }} &lt;{{ user.email }}&gt;!  </p>
-
-  <p><a href="{% url 'logout' %}">Click here to log out</a></p>        
-</main>
-{% endblock %}

src/registrar/tests/test_views.py

@@ -40,12 +40,6 @@ class TestViews(TestCase):
         response = self.client.get("/")
         self.assertEqual(response.status_code, 302)
 
-    def test_whoami_page_no_user(self):
-        """Whoami page not accessible without a logged-in user."""
-        response = self.client.get("/whoami/")
-        self.assertEqual(response.status_code, 302)
-        self.assertIn("?next=/whoami/", response.headers["Location"])
-
     def test_application_form_not_logged_in(self):
         """Application form not accessible without a logged-in user."""
         response = self.client.get("/register/")
@@ -100,13 +94,6 @@ class LoggedInTests(TestWithUser):
         # clean up
         role.delete()
 
-    def test_whoami_page(self):
-        """User information appears on the whoami page."""
-        response = self.client.get("/whoami/")
-        self.assertContains(response, self.user.first_name)
-        self.assertContains(response, self.user.last_name)
-        self.assertContains(response, self.user.email)
-
     def test_application_form_view(self):
         response = self.client.get("/register/", follow=True)
         self.assertContains(

src/registrar/views/__init__.py

@@ -12,4 +12,3 @@ from .domain import (
 )
 from .health import *
 from .index import *
-from .whoami import *

src/registrar/views/whoami.py

@@ -1,8 +0,0 @@
-from django.shortcuts import render
-from django.contrib.auth.decorators import login_required
-
-
-@login_required
-def whoami(request):
-    """This is the first page someone goes to after logging in."""
-    return render(request, "whoami.html")