Merge remote-tracking branch 'origin/bug/book-no-more-than-12-places'

Author: laurence

server.py

@@ -1,6 +1,5 @@
 import json
 from datetime import datetime
-
 from flask import Flask,render_template,request,redirect,flash,url_for
 
 
@@ -48,6 +47,9 @@ def purchasePlaces():
     competition = [c for c in competitions if c['name'] == request.form['competition']][0]
     club = [c for c in clubs if c['name'] == request.form['club']][0]
     placesRequired = int(request.form['places'])
+    if placesRequired > 12:
+        flash("you can not book more than 12 places")
+        return render_template('welcome.html', club=club, competitions=competitions)
     if datetime.strptime(competition['date'], "%Y-%m-%d %H:%M:%S") < datetime.now():
         flash("You cannot book place in past competition")
         return render_template('welcome.html', club=club, competitions=competitions)

templates/booking.html

@@ -10,7 +10,7 @@ <h2>{{competition['name']}}</h2>
     <form action="/purchasePlaces" method="post">
         <input type="hidden" name="club" value="{{club['name']}}">
         <input type="hidden" name="competition" value="{{competition['name']}}">
-        <label for="places">How many places?</label><input type="number" name="places" id=""/>
+        <label for="places">How many places?</label><input type="number" min="0" max="12" name="places" id=""/>
         <button type="submit">Book</button>
     </form>
 </body>

test/tests_unitaires/test_server.py

@@ -6,7 +6,7 @@
 @pytest.fixture
 def client():
     app.config['TESTING'] = True
-    app.secret_key = "test"  # Nécessaire pour flash
+    app.secret_key = "test"
     with app.test_client() as client:
         yield client
 
@@ -16,7 +16,6 @@ def test_booking_past_competition(client, monkeypatch):
     past_date = (datetime.now() - timedelta(days=1)).strftime("%Y-%m-%d %H:%M:%S")
     test_competition = {"name": "Past Competition", "numberOfPlaces": "10", "date": past_date}
     test_club = {"name": "Test Club", "email": "[email protected]", "points": "15"}
-
     monkeypatch.setattr("server.clubs", [test_club])
     monkeypatch.setattr("server.competitions", [test_competition])
 
@@ -35,6 +34,26 @@ def test_booking_past_competition(client, monkeypatch):
     # Vérifie que les points du club n'ont pas changé
     assert test_club["points"] == "15"
 
+
+def test_booking_more_than_12_places(client, monkeypatch):
+    test_club = {"name": "Test Club", "email": "[email protected]", "points": "50"}
+    test_competition = {"name": "Test Competition", "numberOfPlaces": "25", "date": "2025-12-12 10:00:00"}
+    monkeypatch.setattr("server.clubs", [test_club])
+    monkeypatch.setattr("server.competitions", [test_competition])
+
+    response = client.post("/purchasePlaces", data={
+        "competition": "Test Competition",
+        "club": "Test Club",
+        "places": "13"  # > 12 → doit déclencher le bloc
+    }, follow_redirects=True)
+
+    # Vérifie que le message flash est bien là
+    assert b"you can not book more than 12 places" in response.data
+    # Vérifie que les données n'ont pas été modifiées
+    assert test_competition["numberOfPlaces"] == "25"
+    assert test_club["points"] == "50"
+
+
 def test_competition_places_are_decreased(client, monkeypatch):
     # Club avec assez de points
     test_club = {"name": "Test Club", "email": "[email protected]", "points": "20"}
@@ -53,4 +72,5 @@ def test_competition_places_are_decreased(client, monkeypatch):
     }, follow_redirects=True)
 
     #  Vérifie que la compétition a bien été mise à jour
-    assert test_competition["numberOfPlaces"] == 7  # 10 - 3
+    assert test_competition["numberOfPlaces"] == 7  # 10 - 3
+