From ac2a802b72c7c8714cda8beecdaf33b0bd4cb7ec Mon Sep 17 00:00:00 2001
From: sahil raja <s.raja@imperial.ac.uk>
Date: Wed, 9 Oct 2024 16:07:49 +0100
Subject: [PATCH] adding tests and removing dropdown

---
 main/templates/main/navbar.html | 20 +++++---------------
 tests/main/test_views.py        |  8 ++++++++
 2 files changed, 13 insertions(+), 15 deletions(-)

diff --git a/main/templates/main/navbar.html b/main/templates/main/navbar.html
index 445a98f..9470c58 100644
--- a/main/templates/main/navbar.html
+++ b/main/templates/main/navbar.html
@@ -24,23 +24,13 @@
       </ul>
       <ul class="navbar-nav">
         {% if user.is_authenticated %}
-          <li class="nav-item dropdown">
-            <a class="nav-link dropdown-toggle"
-               href="#"
-               id="navbarDropdownMenuLink"
-               role="button"
-               data-bs-toggle="dropdown"
-               aria-expanded="false">{{ user.username }}</a>
-            <ul class="dropdown-menu" aria-labelledby="navbarDropdownMenuLink">
-              <li>
-                <a class="dropdown-item" href="{% url 'main:logout' %}">Logout</a>
-              </li>
-            </ul>
+          <li class="nav-item">
+            <a class="nav-link">Welcome: {{ user.username }}</a>
+          </li>
+          <li class="nav-item">
+            <a class="nav-link" href="{% url 'main:help' %}">Help</a>
           </li>
         {% endif %}
-        <li class="nav-item">
-          <a class="nav-link" href="{% url 'main:help' %}">Help</a>
-        </li>
       </ul>
     </div>
   </div>
diff --git a/tests/main/test_views.py b/tests/main/test_views.py
index efa8274..a10f4cc 100644
--- a/tests/main/test_views.py
+++ b/tests/main/test_views.py
@@ -38,3 +38,11 @@ def test_index_view_authenticated(self, auth_client):
             response,
             f'<a href="{reverse("controller:index")}" class="btn btn-secondary">Controller</a>',  # noqa: E501
         )
+
+    def test_help_view(self, auth_client):
+        """Test that the help view is rendered."""
+        response = auth_client.get(reverse("main:help"))
+        assert response.status_code == HTTPStatus.OK
+        assertTemplateUsed(response, "main/help.html")
+        assertContains(response, "<h1>Help</h1>")
+        assertContains(response, "This is the help page.")