From d8223cb2973d4bd8c74c83df7e8f7590a5994bec Mon Sep 17 00:00:00 2001
From: Peter Heijstek <p.j.heijstek@student.tue.nl>
Date: Thu, 3 Apr 2025 14:15:43 +0200
Subject: [PATCH] add tests and remove unused css class

---
 ihatemoney/static/css/main.css |  5 -----
 ihatemoney/templates/home.html |  4 ++--
 ihatemoney/tests/api_test.py   | 39 +++++++++++++++++++++++++++++-----
 3 files changed, 36 insertions(+), 12 deletions(-)

diff --git a/ihatemoney/static/css/main.css b/ihatemoney/static/css/main.css
index 2249cabf..bbe8f15e 100644
--- a/ihatemoney/static/css/main.css
+++ b/ihatemoney/static/css/main.css
@@ -645,8 +645,3 @@ footer .icon svg {
 .edit-project .custom-file {
   margin-bottom: 2em;
 }
-
-.connected-projects {
-  a {
-  }
-}
diff --git a/ihatemoney/templates/home.html b/ihatemoney/templates/home.html
index b27d7c0b..f748f9f5 100644
--- a/ihatemoney/templates/home.html
+++ b/ihatemoney/templates/home.html
@@ -36,10 +36,10 @@
   {% if 'projects' in session %}
     <div class="card">
       <div class="card-header">
-        {{ _("Open a project in your current session") }}
+        {{ _("Open a connected project") }}
       </div>
       <div class="card-body">
-        <ul class="connected-projects">
+        <ul>
           {% for id, name in session['projects'].items() %}
             <li><a href="{{ url_for("main.list_bills", project_id=id )}}">{{name}}</a>
           {% endfor %}
diff --git a/ihatemoney/tests/api_test.py b/ihatemoney/tests/api_test.py
index d9f655ed..6e5d6b3b 100644
--- a/ihatemoney/tests/api_test.py
+++ b/ihatemoney/tests/api_test.py
@@ -6,7 +6,7 @@ import pytest
 
 from ihatemoney.tests.common.help_functions import em_surround
 from ihatemoney.tests.common.ihatemoney_testcase import IhatemoneyTestCase
-from flask import session
+from flask import url_for
 
 
 class TestAPI(IhatemoneyTestCase):
@@ -1081,6 +1081,29 @@ class TestAPI(IhatemoneyTestCase):
         got = json.loads(req.data.decode("utf-8"))
         assert got["bill_type"] == "Expense"
 
+    def test_project_list_redirection(self):
+        self.post_project("project1", default_currency="USD")
+        self.post_project("project2", default_currency="EUR")
+
+        # Step 2: Log into these projects (simulate a user accessing them)
+        self.login("project1")
+        self.login("project2")
+
+        # Step 3: Access the homepage where the project list should be displayed
+        response = self.client.get("/")
+        self.assertStatus(200, response)
+        page_content = response.data.decode("utf-8")
+
+        # Check that both project names appear in the list
+        assert "project1" in page_content
+        assert "project2" in page_content
+
+        # Step 4: Simulate clicking on "project1" by visiting its link
+        response = self.client.get("/project1/")
+        self.assertStatus(200, response)  # Should load the project page
+        
+        assert "project1" in response.data.decode("utf-8")  # Project content should be visible
+
     def test_get_auth(self):
         """
         Redirects to logged in projects
@@ -1088,21 +1111,27 @@ class TestAPI(IhatemoneyTestCase):
         self.create_project("test-project")
         self.login("test-project")
 
+        req = self.client.get(url_for('main.list_bills', project_id='test-project'))
+        self.assertStatus(200, req)
+
     def test_post_auth_wrong_password(self):
         """
         Rejects wrong passwords for projects
         in the session
         """
         self.create_project("project1", password="a")
-        self.login("project1", "a")
+        req = self.login("project1", "b")
 
-        print(session["projects"])
-        
+
+        assert req.request.path == '/authenticate'
 
     def test_post_auth_correct_password(self):
         """
         Accepts correct passwords for projects
         in the session
         """
-        pass
+        self.create_project("project1", password="a")
+        req = self.login("project1", "a")
+        
+        assert req.request.path == '/project1/'