From 20736bd00156fad2e6e315a451c8b3e5bb0fb667 Mon Sep 17 00:00:00 2001 From: Imenbr Date: Wed, 29 May 2024 17:32:43 +0200 Subject: [PATCH 1/2] (fix) handle unknown email in login --- .coverage | Bin 0 -> 53248 bytes .coveragerc | 6 ++ .flake8 | 12 +++ .gitignore | 5 +- pyproject.toml | 2 + pytest.ini | 3 + requirements.txt | 3 + server.py | 81 +++++++++----- templates/index.html | 13 ++- templates/welcome.html | 6 +- tests/__init__.py | 0 tests/integration_tests/__init__.py | 0 tests/integration_tests/conftest.py | 48 +++++++++ tests/integration_tests/test_unknown_email.py | 24 +++++ tests/unit_tests/__init__.py | 0 tests/unit_tests/conftest.py | 8 ++ tests/unit_tests/test_unknown_email.py | 101 ++++++++++++++++++ tests/unit_tests/utils.py | 23 ++++ 18 files changed, 301 insertions(+), 34 deletions(-) create mode 100644 .coverage create mode 100644 .coveragerc create mode 100644 .flake8 create mode 100644 pyproject.toml create mode 100644 pytest.ini create mode 100644 tests/__init__.py create mode 100644 tests/integration_tests/__init__.py create mode 100644 tests/integration_tests/conftest.py create mode 100644 tests/integration_tests/test_unknown_email.py create mode 100644 tests/unit_tests/__init__.py create mode 100644 tests/unit_tests/conftest.py create mode 100644 tests/unit_tests/test_unknown_email.py create mode 100644 tests/unit_tests/utils.py diff --git a/.coverage b/.coverage new file mode 100644 index 0000000000000000000000000000000000000000..e361004f7f162b57579d9fcceca1f3ab6d62707d GIT binary patch literal 53248 zcmeI)+i%-c90zdQr%RSJ!9&xCs%rEI3EHY{LN&25co+rxFfnNx6kgECoaChOi0#aF z*5(0C-H;|B@dtnq{3-kkpz%KO0(+UzB$$wfRDM5Sa@m@#+RL=&YvtnD=NzB&Ikz|$ ze{%7h9V*%Me8&pq3GIlc>)Lx#YMPd#dy($xlBJcY^aVZZL+j&Kb6WYzFGce+t&sUv zGruUFH&+Tj6yKQswczBwnf@_rP5R!J>?mJ0?Y5G(+pz0aXuF#-Y^gMKH&6|p z=x7^FbsVr9#pkuE-Jn>Z+N2~pzU^3kPkyO-%Ta*jXrbB;;|-LEvNv50As3S9i_%w3 z^v(<7y!4TGYbZBjyY&u?({-BNMoSmM8!io{b(srvIgXhe zYe;7hd5@S$@Fa4RGUs|J>mVW@nj%MecDqiE$ggyr4p$Dvhz}fk<6@4B8+u7$_gy!8 z)a9VOBh{IbbB)Hax~_e-tNN9{EC(@{qhh$;vgATg3wV7xQ#yTA*Qiyh?ci#g-fvZ{ zZsLxxbv0@z%u}t|VbEX6jWnA6<}po&_m6Kk+)G9q3HPFQ zTYAl?%W!NNR}Qu2;XYGIF)W`un)H%@l+)+h+Bmb62U%--wb8Z~wU|Ir2cfYb4c6?s zb?zzwm!a-mQ@*vS%E{h%Sa{wIiJH!8E>l{c8`pIDO`y8kU(N5#WU{5%S^Y*XYDP(I zB~RIrCY09X9>Zw5%TZR)_0xgmNrPk>V=zjleI(gK#aYQ_N^7&@ij#z^t|lvocd}F2 z(y?RuPTV7ST;ltokwWm6K^J&rBG`ZSJ670|kLR5?Q^w~HWlHZJ8<%m;LUolNGdDjs zqAcH0uX=z1Rwwb2tWV=5P$##AOL}95jZJ? z7F0eB=voGqD_)N-pVCm33%#)Axz!C7@I{I$p>jHHy2e0IiLMD#ZiP`h~^M73b4{a0z5P$##AOHafKmY;| zfB*y_aG(VQ{Z63B-~a38e;VDeKmY;|fB*y_009U<00Izz00ba#00o37=HmbV-{$N8 z-!X3=z(S%(5P$##AOHafKmY;|fB*y_0D*%kkQY;W?iKN1clT%gC4<&qH6C35_l&l? zdtI3NR4(`0AH9=*{&c6HWwb*4|NncM`Oy5^ymv4`P$~#O00Izz00bZa0SG_<0uX?} zL`+x1oL@A)K5P$##AOHafKmY;|fB*y_0D*}U;P3x&{y%Zm zjsimf0uX=z1Rwwb2tWV=5P-mh3B>>Zzt6w_e_-C9u)?Fb5P$##AOHafKmY;|fB*y_ T0D*%lAOv0YUl{zr*Z=/') -def book(competition,club): - foundClub = [c for c in clubs if c['name'] == club][0] - foundCompetition = [c for c in competitions if c['name'] == competition][0] +@app.route("/book//") +def book(competition, club): + foundClub = [c for c in clubs if c["name"] == club][0] + foundCompetition = [c for c in competitions if c["name"] == competition][0] if foundClub and foundCompetition: - return render_template('booking.html',club=foundClub,competition=foundCompetition) + return render_template( + "booking.html", club=foundClub, competition=foundCompetition + ) else: flash("Something went wrong-please try again") - return render_template('welcome.html', club=club, competitions=competitions) + return render_template( + "welcome.html", club=club, competitions=competitions + ) -@app.route('/purchasePlaces',methods=['POST']) +@app.route("/purchasePlaces", methods=["POST"]) 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']) - competition['numberOfPlaces'] = int(competition['numberOfPlaces'])-placesRequired - flash('Great-booking complete!') - return render_template('welcome.html', club=club, competitions=competitions) + 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 + ) + flash("Great-booking complete!") + return render_template("welcome.html", club=club, competitions=competitions) # TODO: Add route for points display -@app.route('/logout') +@app.route("/logout") def logout(): - return redirect(url_for('index')) \ No newline at end of file + return redirect(url_for("index")) diff --git a/templates/index.html b/templates/index.html index 926526b7d..0d9e52c9f 100644 --- a/templates/index.html +++ b/templates/index.html @@ -1,3 +1,4 @@ + @@ -6,10 +7,20 @@

Welcome to the GUDLFT Registration Portal!

+ {% with messages = get_flashed_messages() %} + {% if messages %} +
    + {% for message in messages %} +
  • {{message}}
    • + {% endfor %} +
+ {% endif %} + {% endwith %} + Please enter your secretary email to continue:
- +
diff --git a/templates/welcome.html b/templates/welcome.html index ff6b261a2..db7c1e47e 100644 --- a/templates/welcome.html +++ b/templates/welcome.html @@ -7,7 +7,7 @@

Welcome, {{club['email']}}

Logout - {% with messages = get_flashed_messages()%} + {% with messages = get_flashed_messages() %} {% if messages %}
    {% for message in messages %} @@ -15,6 +15,7 @@

    Welcome, {{club['email']}}

    Logout {% endfor %}
{% endif%} + Points available: {{club['points']}}

Competitions:

    @@ -30,7 +31,6 @@

    Competitions:

    {% endfor %}
- {%endwith%} - + {% endwith %} \ No newline at end of file diff --git a/tests/__init__.py b/tests/__init__.py new file mode 100644 index 000000000..e69de29bb diff --git a/tests/integration_tests/__init__.py b/tests/integration_tests/__init__.py new file mode 100644 index 000000000..e69de29bb diff --git a/tests/integration_tests/conftest.py b/tests/integration_tests/conftest.py new file mode 100644 index 000000000..4cf8514f0 --- /dev/null +++ b/tests/integration_tests/conftest.py @@ -0,0 +1,48 @@ +import json +import pytest +from unittest.mock import mock_open, patch + +# Mock data for clubs.json and competitions.json +mock_clubs_json = json.dumps( + { + "clubs": [ + {"name": "Club 1", "email": "club1@example.com"}, + {"name": "Club 2", "email": "club2@example.com"}, + ] + } +) + +mock_competitions_json = json.dumps( + { + "competitions": [ + {"name": "Competition 1", "numberOfPlaces": "25"}, + {"name": "Competition 2", "numberOfPlaces": "15"}, + ] + } +) + + +def mocked_open(file, *args, **kwargs): + """ + Mock open function to return mock data for + clubs.json and competitions.json + """ + if file == "clubs.json": + return mock_open(read_data=mock_clubs_json)() + elif file == "competitions.json": + return mock_open(read_data=mock_competitions_json)() + else: + raise FileNotFoundError(f"File {file} not found") + + +# Patch open before importing the app to ensure clubs +# and competitions are loaded with mock data +with patch("builtins.open", side_effect=mocked_open): + from server import app # Import app after patching + + +@pytest.fixture +def client(): + app.config["TESTING"] = True + with app.test_client() as client: + yield client diff --git a/tests/integration_tests/test_unknown_email.py b/tests/integration_tests/test_unknown_email.py new file mode 100644 index 000000000..f77e75899 --- /dev/null +++ b/tests/integration_tests/test_unknown_email.py @@ -0,0 +1,24 @@ + +# Integration test for valid email +def test_showSummary_valid_email_integration(client): + # Simulate POST request with a valid email + response = client.post("/showSummary", data={"email": "club1@example.com"}) + + # Check if the welcome page is rendered with the correct club data + assert response.status_code == 200 + assert b"Welcome" in response.data + + +# Integration test for invalid email +def test_showSummary_invalid_email_integration(client): + # Simulate POST request with an invalid email + response = client.post( + "/showSummary", data={"email": "invalid@example.com"} + ) + + # Validate that the response redirects to the index page + assert response.status_code == 302 + response = client.get(response.headers["Location"]) # Follow the redirect + + # Check if the flash message appears in the redirected page + assert b"Sorry, that email wasn't found." in response.data diff --git a/tests/unit_tests/__init__.py b/tests/unit_tests/__init__.py new file mode 100644 index 000000000..e69de29bb diff --git a/tests/unit_tests/conftest.py b/tests/unit_tests/conftest.py new file mode 100644 index 000000000..1c3ff9b09 --- /dev/null +++ b/tests/unit_tests/conftest.py @@ -0,0 +1,8 @@ +import pytest +from server import app + + +@pytest.fixture +def client(): + with app.test_client() as client: + yield client diff --git a/tests/unit_tests/test_unknown_email.py b/tests/unit_tests/test_unknown_email.py new file mode 100644 index 000000000..d0bd5446e --- /dev/null +++ b/tests/unit_tests/test_unknown_email.py @@ -0,0 +1,101 @@ +import json +from unittest.mock import mock_open, patch + +from server import get_club_from_email, loadClubs, loadCompetitions +from .utils import ( + mock_clubs_list, + mock_competitions_list, + mock_clubs_json, + mock_competitions_json, +) + + +# Test for loadClubs function +@patch("builtins.open", new_callable=mock_open, read_data=mock_clubs_json) +@patch("server.json.load") +def test_loadClubs(mock_json_load, mock_file): + # Mock json.load to return the parsed data + mock_json_load.return_value = json.loads(mock_clubs_json) + + # Call loadClubs and check if it returns the correct data + clubs = loadClubs() + assert len(clubs) == 2 + assert clubs[0]["name"] == "Club 1" + assert clubs[1]["email"] == "club2@example.com" + + +# Test for loadCompetitions function +@patch( + "builtins.open", new_callable=mock_open, read_data=mock_competitions_json +) +@patch("server.json.load") +def test_loadCompetitions(mock_json_load, mock_file): + # Mock json.load to return the parsed data + mock_json_load.return_value = json.loads(mock_competitions_json) + + # Call loadCompetitions and check if it returns the correct data + competitions = loadCompetitions() + assert len(competitions) == 2 + assert competitions[0]["name"] == "Competition 1" + assert competitions[1]["numberOfPlaces"] == "15" + + +@patch("server.clubs", mock_clubs_list) +def test_get_club_from_email_valid(): + club = get_club_from_email("club1@example.com") + assert club is not None + assert club["name"] == "Club 1" + + +@patch("server.clubs", mock_clubs_list) +def test_get_club_from_email_invalid(): + club = get_club_from_email("invalid@example.com") + assert club is None + + +# Unit test for showSummary with valid email +@patch( + "server.competitions", mock_competitions_list +) # Directly patch the value of competitions +@patch("server.get_club_from_email") +@patch("server.render_template") +def test_showSummary_valid_email(mock_render_template, mock_get_club, client): + # Mock get_club_from_email to return a valid club + mock_get_club.return_value = mock_clubs_list[0] + + # Simulate POST request with a valid email + response = client.post("/showSummary", data={"email": "club1@example.com"}) + + # Check if render_template was called with the correct arguments + mock_render_template.assert_called_once_with( + "welcome.html", + club=mock_clubs_list[0], + competitions=mock_competitions_list, + ) + assert response.status_code == 200 + + +# Unit test for showSummary with invalid email +@patch("server.get_club_from_email") +@patch("server.flash") +@patch("server.redirect") +@patch("server.url_for") +def test_showSummary_invalid_email( + mock_url_for, mock_redirect, mock_flash, mock_get_club, client +): + # Mock get_club_from_email to return None (simulating an invalid email) + mock_get_club.return_value = None + + # Mock url_for to return a URL for the index page + mock_url_for.return_value = "/" + + # Simulate POST request with an invalid email + client.post( + "/showSummary", data={"email": "invalid@example.com"} + ) + + # Check if flash was called with the correct message + mock_flash.assert_called_once_with("Sorry, that email wasn't found.") + + # Check if redirect was called with the correct arguments + mock_redirect.assert_called_once_with("/") diff --git a/tests/unit_tests/utils.py b/tests/unit_tests/utils.py new file mode 100644 index 000000000..3a095deab --- /dev/null +++ b/tests/unit_tests/utils.py @@ -0,0 +1,23 @@ + +import json + + +# Mock data for clubs and competitions +mock_clubs_list = [ + {"name": "Club 1", "email": "club1@example.com"}, + {"name": "Club 2", "email": "club2@example.com"} +] + +mock_competitions_list = [ + {"name": "Competition 1", "numberOfPlaces": "25"}, + {"name": "Competition 2", "numberOfPlaces": "15"} +] + +# Mock data for clubs.json and competitions.json +mock_clubs_json = json.dumps({ + "clubs": mock_clubs_list +}) + +mock_competitions_json = json.dumps({ + "competitions": mock_competitions_list +}) From 247bb57d4fcf8a3b0c8e66ca9affa1d5368df0cb Mon Sep 17 00:00:00 2001 From: Imenbr Date: Fri, 31 May 2024 00:45:47 +0200 Subject: [PATCH 2/2] (fix) Handle redeeming more points than available --- .coverage | Bin 53248 -> 53248 bytes requirements.txt | 2 +- server.py | 67 ++++- templates/booking.html | 12 +- tests/integration_tests/conftest.py | 32 ++- ...s_should_not_use_more_than_their_points.py | 44 ++++ ...s_should_not_use_more_than_their_points.py | 229 ++++++++++++++++++ tests/unit_tests/utils.py | 26 +- 8 files changed, 370 insertions(+), 42 deletions(-) create mode 100644 tests/integration_tests/test_clubs_should_not_use_more_than_their_points.py create mode 100644 tests/unit_tests/test_clubs_should_not_use_more_than_their_points.py diff --git a/.coverage b/.coverage index e361004f7f162b57579d9fcceca1f3ab6d62707d..5c65d975222eea263bfde8c47648bae7c1596e07 100644 GIT binary patch delta 259 zcmZozz}&Eac>{|B4-3CK1OH|Inf&qm>YD`x#Q9mxSeO}#C(rZOoZRov&B#1?w!aR0 zQht7RW?uT_DgFgO9U2V$SNIe7HTY)%bx80}PKq~YWwB%~X9Vejsogv!zCZz}_#gxS zEB;&jXZa5T6>sGC=VD=D

B1_W%EWMs8*xTY?$L{{LT^j|s?;WBOMA1H`Ro7GY%J zZwh7Xt$W3;#0){@?sx_}}tB0~&gbpPdb;m63&I^V|92 hAUofHq`&ch{|B7c+k%1OH|Inf!^H1qFinS{VECnv<~ZayAws{oX~%)tMi|33d^p!89G zYc6IMMouy2Z~y=AXXIl7vgMe*)&H<(`2WA2nV*q`lT+zgrP=fO_jwrD7^Xai-S!42on6p{}Cwon4g~+=tf> int(club["points"]): + return "Places required exceed club's total points" + + +def take_places(places, club, competition): + try: + competition["numberOfPlaces"] = \ + int(competition["numberOfPlaces"]) - places + club["points"] = int(club["points"]) - places + return True + except Exception: + return False + + @app.route("/purchasePlaces", methods=["POST"]) 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] + competition = get_competition_from_name(request.form["competition"]) + club = get_club_from_name(request.form["club"]) + + error_message = check_places(request.form["places"], club) + if error_message: + flash(error_message) + return redirect( + url_for("book", competition=competition["name"], club=club["name"]) + ) placesRequired = int(request.form["places"]) - competition["numberOfPlaces"] = ( - int(competition["numberOfPlaces"]) - placesRequired - ) - flash("Great-booking complete!") - return render_template("welcome.html", club=club, competitions=competitions) - -# TODO: Add route for points display + if take_places(placesRequired, club, competition): + flash("Great-booking complete!") + return render_template("welcome.html", club=club, + competitions=competitions) + else: + flash("Something went wrong-please try again") + return redirect( + url_for("book", competition=competition["name"], club=club["name"]) + ) @app.route("/logout") diff --git a/templates/booking.html b/templates/booking.html index 06ae1156c..5037049c1 100644 --- a/templates/booking.html +++ b/templates/booking.html @@ -1,3 +1,4 @@ + @@ -6,11 +7,20 @@

{{competition['name']}}

+ {% with messages = get_flashed_messages()%} + {% if messages %} +
    + {% for message in messages %} +
  • {{message}}
    • + {% endfor %} +
+ {% endif%} + {% endwith %} Places available: {{competition['numberOfPlaces']}}
- +
diff --git a/tests/integration_tests/conftest.py b/tests/integration_tests/conftest.py index 4cf8514f0..1c4b1477c 100644 --- a/tests/integration_tests/conftest.py +++ b/tests/integration_tests/conftest.py @@ -3,23 +3,21 @@ from unittest.mock import mock_open, patch # Mock data for clubs.json and competitions.json -mock_clubs_json = json.dumps( - { - "clubs": [ - {"name": "Club 1", "email": "club1@example.com"}, - {"name": "Club 2", "email": "club2@example.com"}, - ] - } -) - -mock_competitions_json = json.dumps( - { - "competitions": [ - {"name": "Competition 1", "numberOfPlaces": "25"}, - {"name": "Competition 2", "numberOfPlaces": "15"}, - ] - } -) +mock_clubs_json = json.dumps({ + "clubs": [ + {"name": "Club 1", "email": "club1@example.com", "points": "10"}, + {"name": "Club 2", "email": "club2@example.com", "points": "15"} + ] +}) + +mock_competitions_json = json.dumps({ + "competitions": [ + {"name": "Competition 1", "date": "2023-03-27 10:00:00", + "numberOfPlaces": "25"}, + {"name": "Competition 2", "date": "2023-10-22 13:30:00", + "numberOfPlaces": "15"} + ] +}) def mocked_open(file, *args, **kwargs): diff --git a/tests/integration_tests/test_clubs_should_not_use_more_than_their_points.py b/tests/integration_tests/test_clubs_should_not_use_more_than_their_points.py new file mode 100644 index 000000000..985ae9115 --- /dev/null +++ b/tests/integration_tests/test_clubs_should_not_use_more_than_their_points.py @@ -0,0 +1,44 @@ + + +def test_purchase_places_valid(client): + """ + Test purchasing places with valid club points and valid competition. + """ + response = client.post("/purchasePlaces", data={ + "competition": "Competition 1", + "club": "Club 1", + "places": "5" + }) + + assert response.status_code == 200 + assert b"Great-booking complete!" in response.data + + +def test_purchase_places_insufficient_points(client): + """ + Test purchasing more places than the club's available points. + """ + response = client.post("/purchasePlaces", data={ + "competition": "Competition 1", + "club": "Club 1", + "places": "20" # Club 1 has only 10 points + }) + + assert response.status_code == 302 # Redirect expected + response = client.get(response.headers["Location"]) # Follow the redirect + assert b"Places required exceed club's total points" in response.data + + +def test_purchase_places_zero_places(client): + """ + Test purchasing zero places, which is an invalid input. + """ + response = client.post("/purchasePlaces", data={ + "competition": "Competition 1", + "club": "Club 1", + "places": "0" + }) + + assert response.status_code == 302 # Redirect expected + response = client.get(response.headers["Location"]) # Follow the redirect + assert b"Places required must be a positive integer" in response.data diff --git a/tests/unit_tests/test_clubs_should_not_use_more_than_their_points.py b/tests/unit_tests/test_clubs_should_not_use_more_than_their_points.py new file mode 100644 index 000000000..5170fd669 --- /dev/null +++ b/tests/unit_tests/test_clubs_should_not_use_more_than_their_points.py @@ -0,0 +1,229 @@ +from unittest.mock import patch +from .utils import mock_clubs_list, mock_competitions_list + + +# Unit Tests for get_competition_from_name +@patch("server.competitions", mock_competitions_list) +def test_get_competition_from_name_valid(): + from server import get_competition_from_name + + competition = get_competition_from_name("Competition 1") + assert competition is not None + assert competition["name"] == "Competition 1" + + +@patch("server.competitions", mock_competitions_list) +def test_get_competition_from_name_invalid(): + from server import get_competition_from_name + + competition = get_competition_from_name("Invalid Competition") + assert competition is None + + +# Unit Tests for get_club_from_name ### +@patch("server.clubs", mock_clubs_list) +def test_get_club_from_name_valid(): + from server import get_club_from_name + + club = get_club_from_name("Club 1") + assert club is not None + assert club["name"] == "Club 1" + + +@patch("server.clubs", mock_clubs_list) +def test_get_club_from_name_invalid(): + from server import get_club_from_name + + club = get_club_from_name("Invalid Club") + assert club is None + + +# Unit Tests for check_places ### +def test_check_places_valid(): + from server import check_places + + error = check_places("5", mock_clubs_list[0]) + assert error is None + + +def test_check_places_invalid_zero(): + from server import check_places + + error = check_places("0", mock_clubs_list[0]) + assert error == "Places required must be a positive integer" + + +def test_check_places_invalid_negative(): + from server import check_places + + error = check_places("-3", mock_clubs_list[0]) + assert error == "Places required must be a positive integer" + + +def test_check_places_exceeds_points(): + from server import check_places + + error = check_places("20", mock_clubs_list[0]) + assert error == "Places required exceed club's total points" + + +# Unit Tests for take_places ### +def test_take_places_valid(): + from server import take_places + + result = take_places(5, mock_clubs_list[0], mock_competitions_list[0]) + assert result is True + assert mock_clubs_list[0]["points"] == 5 # 10 - 5 + assert mock_competitions_list[0]["numberOfPlaces"] == 20 # 25 - 5 + + +def test_take_places_invalid(): + from server import take_places + + result = take_places( + "invalid", mock_clubs_list[0], mock_competitions_list[0] + ) + assert result is False + + +# Unit test for valid booking +@patch("server.competitions", mock_competitions_list) +@patch("server.render_template") +@patch("server.take_places") +@patch("server.check_places") +@patch("server.get_club_from_name") +@patch("server.get_competition_from_name") +@patch("server.flash") +def test_purchase_places_valid( + mock_flash, + mock_get_competition, + mock_get_club, + mock_check_places, + mock_take_places, + mock_render_template, + client, +): + # Mock the functions to return valid data + mock_get_competition.return_value = mock_competitions_list[0] + mock_get_club.return_value = mock_clubs_list[0] + mock_check_places.return_value = None # No error message + mock_take_places.return_value = True # Simulate successful booking + + # Simulate POST request to the route + response = client.post( + "/purchasePlaces", + data={"competition": "Competition 1", "club": "Club 1", "places": "5"}, + ) + + # Assert that the necessary functions are called with the correct parameters + mock_get_competition.assert_called_once_with("Competition 1") + mock_get_club.assert_called_once_with("Club 1") + mock_check_places.assert_called_once_with("5", mock_clubs_list[0]) + mock_take_places.assert_called_once_with( + 5, mock_clubs_list[0], mock_competitions_list[0] + ) + mock_flash.assert_called_once_with("Great-booking complete!") + mock_render_template.assert_called_once_with( + "welcome.html", + club=mock_clubs_list[0], + competitions=mock_competitions_list, + ) + + # Check response status code + assert response.status_code == 200 + + +# Unit test for booking with error in places +@patch("server.redirect") +@patch("server.url_for") +@patch("server.flash") +@patch("server.check_places") +@patch("server.get_club_from_name") +@patch("server.get_competition_from_name") +def test_purchase_places_invalid_places( + mock_get_competition, + mock_get_club, + mock_check_places, + mock_flash, + mock_url_for, + mock_redirect, + client, +): + # Mock the functions to simulate an error in places + mock_get_competition.return_value = mock_competitions_list[0] + mock_get_club.return_value = mock_clubs_list[0] + mock_check_places.return_value = ( + "Places required exceed club's total points" + ) + + # Mock the redirect and url_for to return a redirect response + mock_url_for.return_value = "/book" + + # Simulate POST request to the route + client.post( + "/purchasePlaces", + data={"competition": "Competition 1", "club": "Club 1", "places": "20"}, + ) + + # Assert that the necessary functions are called with the correct parameters + mock_get_competition.assert_called_once_with("Competition 1") + mock_get_club.assert_called_once_with("Club 1") + mock_check_places.assert_called_once_with("20", mock_clubs_list[0]) + mock_flash.assert_called_once_with( + "Places required exceed club's total points" + ) + mock_url_for.assert_called_once_with( + "book", + competition=mock_competitions_list[0]["name"], + club=mock_clubs_list[0]["name"], + ) + mock_redirect.assert_called_once_with("/book") + + +# Unit test for failed booking due to take_places failure +@patch("server.redirect") +@patch("server.url_for") +@patch("server.flash") +@patch("server.take_places") +@patch("server.check_places") +@patch("server.get_club_from_name") +@patch("server.get_competition_from_name") +def test_purchase_places_failed_take_places( + mock_get_competition, + mock_get_club, + mock_check_places, + mock_take_places, + mock_flash, + mock_url_for, + mock_redirect, + client, +): + # Mock the functions to simulate an error in take_places + mock_get_competition.return_value = mock_competitions_list[0] + mock_get_club.return_value = mock_clubs_list[0] + mock_check_places.return_value = None # No error message + mock_take_places.return_value = False # Simulate failure in taking places + + # Mock the redirect and url_for to return a redirect response + mock_url_for.return_value = "/book" + + # Simulate POST request to the route + client.post( + "/purchasePlaces", + data={"competition": "Competition 1", "club": "Club 1", "places": "5"}, + ) + + # Assert that the necessary functions are called with the correct parameters + mock_get_competition.assert_called_once_with("Competition 1") + mock_get_club.assert_called_once_with("Club 1") + mock_check_places.assert_called_once_with("5", mock_clubs_list[0]) + mock_take_places.assert_called_once_with( + 5, mock_clubs_list[0], mock_competitions_list[0] + ) + mock_flash.assert_called_once_with("Something went wrong-please try again") + mock_url_for.assert_called_once_with( + "book", + competition=mock_competitions_list[0]["name"], + club=mock_clubs_list[0]["name"], + ) + mock_redirect.assert_called_once_with("/book") diff --git a/tests/unit_tests/utils.py b/tests/unit_tests/utils.py index 3a095deab..233e88226 100644 --- a/tests/unit_tests/utils.py +++ b/tests/unit_tests/utils.py @@ -1,23 +1,25 @@ - import json # Mock data for clubs and competitions mock_clubs_list = [ - {"name": "Club 1", "email": "club1@example.com"}, - {"name": "Club 2", "email": "club2@example.com"} + {"name": "Club 1", "email": "club1@example.com", "points": "10"}, + {"name": "Club 2", "email": "club2@example.com", "points": "15"}, ] mock_competitions_list = [ - {"name": "Competition 1", "numberOfPlaces": "25"}, - {"name": "Competition 2", "numberOfPlaces": "15"} + { + "name": "Competition 1", + "date": "2023-03-27 10:00:00", + "numberOfPlaces": "25", + }, + { + "name": "Competition 2", + "date": "2023-10-22 13:30:00", + "numberOfPlaces": "15", + }, ] - # Mock data for clubs.json and competitions.json -mock_clubs_json = json.dumps({ - "clubs": mock_clubs_list -}) +mock_clubs_json = json.dumps({"clubs": mock_clubs_list}) -mock_competitions_json = json.dumps({ - "competitions": mock_competitions_list -}) +mock_competitions_json = json.dumps({"competitions": mock_competitions_list})