Merge pull request #3 from Imed7223/bug/correction-mise-a-jour-points

Correction bug 3: Mise à jour correction des points après réservation…

Author: Imed7223

server.py

@@ -26,7 +26,7 @@ def index():
 
 @app.route('/showSummary',methods=['POST'])
 def showSummary():
-    # Correction bug1 "email"
+    # Correction BUG 1 "email"
     email = request.form.get('email')
     club = next((club for club in clubs if club['email'] == email), None)
 
@@ -50,15 +50,29 @@ def book(competition,club):
 
 @app.route('/purchasePlaces',methods=['POST'])
 def purchasePlaces():
+    # Trouver la compétition et le club
     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'])
     competition['numberOfPlaces'] = int(competition['numberOfPlaces'])-placesRequired
+
+    # Validation des points disponibles
+    if placesRequired > int(club['points']):
+        flash("Your club doesn't have enough points")
+        return render_template('welcome.html', club=club, competitions=competitions), 400
+
+    competition_places = int(competition['numberOfPlaces'])
+    club_points = int(club['points'])
+
+    # Correction BUG 3: Mise à jour des points du club
+    competition['numberOfPlaces'] = competition_places - placesRequired
+    club['points'] = club_points - placesRequired
+
     flash('Great-booking complete!')
     return render_template('welcome.html', club=club, competitions=competitions)
 
 
-# FONCTIONNALITÉ 2: Tableau d'affichage des points
+# FONCTIONNALITÉ BUG 2: Tableau d'affichage des points
 @app.route('/pointsDisplay')
 def pointsDisplay():
     return render_template('points_display.html', clubs=clubs)

tests/unit/test_points_update.py

@@ -0,0 +1,64 @@
+import sys
+import os
+sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), '..', '..')))
+import pytest
+from server import app, clubs, competitions
+import json
+
+@pytest.fixture
+def client():
+    app.config['TESTING'] = True
+    with app.test_client() as client:
+        yield client
+
+def test_points_deduction(client):
+    """Test que les points sont bien déduits après réservation"""
+    # Données de test
+    club_name = "Simply Lift"
+    competition_name = "Spring Festival"
+    places_required = 2
+    
+    # Points initiaux
+    initial_points = int(next(c['points'] for c in clubs if c['name'] == club_name))
+    
+    # Faire la requête de réservation
+    response = client.post('/purchasePlaces', data={
+        'competition': competition_name,
+        'club': club_name,
+        'places': str(places_required)
+    })
+    
+    # Vérifications
+    assert response.status_code == 200
+    assert b"Great-booking complete!" in response.data
+    
+    # Vérifier la mise à jour des points
+    updated_points = int(next(c['points'] for c in clubs if c['name'] == club_name))
+    assert updated_points == initial_points - places_required
+
+def test_insufficient_points(client):
+    """Test que les points ne sont pas déduits si réservation impossible"""
+    # Données de test
+    club_name = "Iron Temple"  # 4 points initialement
+    competition_name = "Spring Festival"
+    places_required = 5  # Plus que les points disponibles
+    
+    # Points initiaux
+    initial_points = int(next(c['points'] for c in clubs if c['name'] == club_name))
+    
+    # Faire la requête de réservation
+    response = client.post('/purchasePlaces', data={
+        'competition': competition_name,
+        'club': club_name,
+        'places': str(places_required)
+    })
+    
+    # Vérifications
+    assert response.status_code == 400
+    # Vérifie soit la version encodée HTML soit décodée
+    assert (b"Your club doesn't have enough points" in response.data or 
+            b"Your club doesn't have enough points" in response.data)
+    
+    # Vérifier que les points n'ont pas changé
+    current_points = int(next(c['points'] for c in clubs if c['name'] == club_name))
+    assert current_points == initial_points