Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: Clubs should not be able to use more than their points allowed #2

Open
dompatmore opened this issue Jul 3, 2020 · 0 comments · May be fixed by #226
Open

BUG: Clubs should not be able to use more than their points allowed #2

dompatmore opened this issue Jul 3, 2020 · 0 comments · May be fixed by #226

Comments

@dompatmore
Copy link
Collaborator

dompatmore commented Jul 3, 2020

When:

A secretary redeems more points than they have available, which would leave them in the negative

Then:

They receive a confirmation message

Expected:

They should not be able to redeem more points than available; this should be done within the UI. 
The redeemed points should be correctly deducted from the club's total.  
PabloLec added a commit to PabloLec/oc_gudlift that referenced this issue Jul 6, 2021
mjeammet referenced this issue in mjeammet/DAPY-P11_gudlft Dec 10, 2021
mjeammet referenced this issue in mjeammet/DAPY-P11_gudlft Dec 17, 2021
RafaRemote pushed a commit to RafaRemote/oc_python_p11_testing that referenced this issue Dec 22, 2021
…le - fix and tests

- fix: ui: booking.html: html input attribute min set to 1 and max set to club["points"]
- fix: server.py: purchase_places(): conditional structure implemented, checks redeemed points amount against competition places available

- tests: class TestPurchase created: method: test_should_not_use_more_points_than_have
- tests: class TestPurchase: method: test_should_deduct_points_from_club
RafaRemote pushed a commit to RafaRemote/oc_python_p11_testing that referenced this issue Dec 23, 2021
…le - Tests Improvement

- tests: conftest.py: config fixture to avoid repetition in test_server.py
- UI: welcome.html: added a span to localize more efficiently the 'club_points_left'
RafaRemote pushed a commit to RafaRemote/oc_python_p11_testing that referenced this issue Dec 23, 2021
… info

- fix: bug already fixed when fixing issue [OpenClassrooms-Student-Center#2] as it was specified: 'The redeemed points should be correctly deducted from the club's total.'

-UI: welcome.html: implementation of a table to better show the points redemption and update after a booking
MilhouseVP added a commit to MilhouseVP/Python_Testing that referenced this issue Jan 3, 2022
found another bug during testing : it was possible to buy more places than available, fixed too.
Madscientiste added a commit to Madscientiste/OpenClassrooms_P11 that referenced this issue Feb 17, 2022
ribandgel pushed a commit to ribandgel/Python_Testing that referenced this issue Mar 17, 2022
bdibon added a commit to bdibon/Python_Testing that referenced this issue Mar 21, 2022
zionhigt referenced this issue in zionhigt/Python_Testing Mar 27, 2022
zionhigt referenced this issue in zionhigt/Python_Testing Mar 27, 2022
This was referenced Mar 29, 2022
josayko added a commit to josayko-courses/p11_gudlft that referenced this issue Apr 19, 2022
Pascal273 added a commit to Pascal273/Python_Testing that referenced this issue May 8, 2022
-BUG: Clubs should not be able to use more than their points allowed- fixed.
Booking Form won't allow number greater than points available.
Pascal273 added a commit to Pascal273/Python_Testing that referenced this issue May 11, 2022
…se to book places

equals the available points of the club
Luc-Aka-Evy added a commit to Luc-Aka-Evy/Gudlift that referenced this issue May 19, 2022
Aponovi added a commit to Aponovi/Python_Testing that referenced this issue May 30, 2022
SelHel added a commit to SelHel/Python_Testing that referenced this issue Jun 9, 2022
SelHel added a commit to SelHel/Python_Testing that referenced this issue Jun 9, 2022
SelHel added a commit to SelHel/Python_Testing that referenced this issue Jun 10, 2022
SelHel added a commit to SelHel/Python_Testing that referenced this issue Jun 10, 2022
jogubo pushed a commit to jogubo/oc-python-testing that referenced this issue Jun 13, 2022
SelHel added a commit to SelHel/Python_Testing that referenced this issue Jun 14, 2022
jogubo pushed a commit to jogubo/oc-python-testing that referenced this issue Jun 14, 2022
Deadjuju added a commit to Deadjuju/P11_OC_Python_Testing that referenced this issue Jul 11, 2022
A club can no longer spend more points than it has

Issue OpenClassrooms-Student-Center#2
Rossignol-h added a commit to Rossignol-h/P11_Python_Testing that referenced this issue Nov 13, 2022
* feat(board.html): add template for points display board

* feat(server): add route for points display board

* feat(board.html): complete template

* test(test_points_board): start testing happy path

* feat(board.html): add condition if no clubs in json

* test(test_points_board): add sad path to test

* test(units/): add test_unlimited_points.py

* fix(server): add condition to limit purchase

* fix(booking.html): add display of flash message

* test(units/): test_unlimited_points.py passed

* test(units/): change docstrings in test_unlimited_points.py

* test(units/): done test_unlimited_points.py

* fix(issue): [OpenClassrooms-Student-Center#2]

* test(units/): add test_points_not_update.py

with TDD approch added test and failed

* fix(server): add substraction points after book places

* test(units/): run test_points_not_update.py and passed

* fix(issue): [OpenClassrooms-Student-Center#6]

* test(units/): add test_book_under_12.py

with TDD method: started with failed test

* fix(server): add condition if book more 12 places

* test(units/): complete test_book_under_12.py and run tests passed

* fix(issue): [OpenClassrooms-Student-Center#4]

* test(functional/): add & complete test_route.py

* fix(404.html): add 404 handler and template

* fix(500.html): add 500 handler and template

* test(functional/): add & complete test_load_json.py

* test(integration/): add & complete test of app

* fix(server): issue book more than places available

* test(integration/): cleanup test_app

* test(coverage/): make changes to have 100% test coverage

* test(requirements.txt): install Locust

* test(performance/): configure locust

* test(performance/): add README

* refactor(app): flake8 html report 100% clean

* refactor(app): remove unused fixtures

* fix(welcome.hmtl): adde operator = to fix bug
Rossignol-h added a commit to Rossignol-h/P11_Python_Testing that referenced this issue Nov 13, 2022
* feat(board.html): add template for points display board

* feat(server): add route for points display board

* feat(board.html): complete template

* test(test_points_board): start testing happy path

* feat(board.html): add condition if no clubs in json

* test(test_points_board): add sad path to test

* test(units/): add test_unlimited_points.py

* fix(server): add condition to limit purchase

* fix(booking.html): add display of flash message

* test(units/): test_unlimited_points.py passed

* test(units/): change docstrings in test_unlimited_points.py

* test(units/): done test_unlimited_points.py

* fix(issue): [OpenClassrooms-Student-Center#2]

* test(units/): add test_points_not_update.py

with TDD approch added test and failed

* fix(server): add substraction points after book places

* test(units/): run test_points_not_update.py and passed

* fix(issue): [OpenClassrooms-Student-Center#6]

* test(units/): add test_book_under_12.py

with TDD method: started with failed test

* fix(server): add condition if book more 12 places

* test(units/): complete test_book_under_12.py and run tests passed

* fix(issue): [OpenClassrooms-Student-Center#4]

* test(functional/): add & complete test_route.py

* fix(404.html): add 404 handler and template

* fix(500.html): add 500 handler and template

* test(functional/): add & complete test_load_json.py

* test(integration/): add & complete test of app

* fix(server): issue book more than places available

* test(integration/): cleanup test_app

* test(coverage/): make changes to have 100% test coverage

* test(requirements.txt): install Locust

* test(performance/): configure locust

* test(performance/): add README

* refactor(app): flake8 html report 100% clean

* refactor(app): remove unused fixtures

* fix(welcome.hmtl): adde operator = to fix bug

* style(app): add css
Mattgris6 added a commit to Mattgris6/OC_DAPython_Projet11_Gudlft that referenced this issue Nov 16, 2022
Mattgris6 added a commit to Mattgris6/OC_DAPython_Projet11_Gudlft that referenced this issue Nov 17, 2022
selaouni added a commit to selaouni/Projet11 that referenced this issue Dec 8, 2022
Bimarsto added a commit to Bimarsto/Python_Testing that referenced this issue Jan 20, 2023
Issue OpenClassrooms-Student-Center#2 fixed + tests created
VisualDev-FR added a commit to VisualDev-FR/openclassrooms-P11 that referenced this issue Mar 31, 2023
VisualDev-FR added a commit to VisualDev-FR/openclassrooms-P11 that referenced this issue Apr 21, 2023
JLenseele added a commit to JLenseele/Project_11 that referenced this issue Apr 26, 2023
…put places / purchasePlaces-add club substration points
Nemesix493 added a commit to Nemesix493/P11_Python_Testing that referenced this issue May 9, 2023
kivtorsm added a commit to kivtorsm/OCP11 that referenced this issue Jun 21, 2023
kivtorsm added a commit to kivtorsm/OCP11 that referenced this issue Jun 22, 2023
CeeG33 referenced this issue in CeeG33/oc-cg-p11 Sep 11, 2023
CeeG33 referenced this issue in CeeG33/oc-cg-p11 Sep 11, 2023
Thomas-Savelli added a commit to Thomas-Savelli/Python_Testing that referenced this issue Nov 28, 2023
…ed the bug by introducing conditional checks and appropriate flash messages. Fixes OpenClassrooms-Student-Center#2.
DomninBenoit added a commit to DomninBenoit/gudlift that referenced this issue Dec 11, 2023
ErnestoAquino added a commit to ErnestoAquino/Python_Testing that referenced this issue Dec 29, 2023
AlexBotswana added a commit to AlexBotswana/Python_Testing that referenced this issue Feb 20, 2024
geo1310 added a commit to geo1310/Python_Testing that referenced this issue Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant