fix: resolve competition place deduction crashes in purchasePlaces by SiRipo92 · Pull Request #364 · OpenClassrooms-Student-Center/Python_Testing

chore: set up test folder structure, conftest and pytest config

SiRipo92

committed

chore: configure app entry point and clean up .gitignore

SiRipo92

committed

chore: add test fixtures and mock data for full test isolation

SiRipo92

committed

Added coverage reports to each testing

SiRipo92

committed

chore: wrote happy and sad test cases for login

SiRipo92

committed

fix: switched from index based search for email to next() with default, used condition to set flash message, set 200 status code and stay on index.html

SiRipo92

committed

improvement: added flash message to set failed login attempts into ui without crashing

SiRipo92

committed

TDD: failed test case for an edge case (no email submission)

SiRipo92

committed

test fix: removed contraction to escape string literals and match server.py

SiRipo92

committed

Merge pull request #11 from SiRipo92/fix/login-unknown-email-crash

SiRipo92

authored

refactor: rename client_with_mock_data to mock_client for brevity

SiRipo92

committed

chore: defined test cases to run for happy + sad paths + edge cases

SiRipo92

committed

test: add happy path for competition place deduction - issue #6

SiRipo92

committed

test: add happy path tests for competition place deduction - issue #6

SiRipo92

committed

test: add sad path for unknown competition - issue #6

SiRipo92

committed

test: add sad path for unknown club - issue #6

SiRipo92

committed

fix: handle unknown club with next() and guard clause - issue #6

SiRipo92

committed

test: add characterisation test for numberOfPlaces type consistency - issue #6

SiRipo92

committed

refactor: PEP8 fixes in purchasePlaces - snake_case and spacing

SiRipo92

committed

Merge pull request #12 from SiRipo92/fix/competition-places-not-deducted

SiRipo92

authored

refactor: PEP8 fixes in show_summary - snake_case and spacing

SiRipo92

committed

Merge pull request #13 from SiRipo92/fix/login-unknown-email-crash

SiRipo92

authored

refactor: PEP8 fixes in purchasePlaces - snake_case in function name

SiRipo92

committed

Merge pull request #14 from SiRipo92/fix/competition-places-not-deducted

SiRipo92

authored

refactor: use pytest.mark.parametrize for sad path tests - issue #2

SiRipo92

committed

refactor: added missing annotation for sad paths/paramaterize

SiRipo92

committed

Merge pull request #15 from SiRipo92/fix/login-unknown-email-crash

SiRipo92

authored

refactor: use pytest.mark.parametrize for sad path tests - issue #6

SiRipo92

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: resolve competition place deduction crashes in purchasePlaces#364

fix: resolve competition place deduction crashes in purchasePlaces#364
SiRipo92 wants to merge 28 commits into
OpenClassrooms-Student-Center:masterfrom
SiRipo92:fix/competition-places-not-deducted

Commits on Mar 13, 2026

chore: set up test folder structure, conftest and pytest config

chore: configure app entry point and clean up .gitignore

chore: add test fixtures and mock data for full test isolation

Added coverage reports to each testing

chore: wrote happy and sad test cases for login

fix: switched from index based search for email to next() with default, used condition to set flash message, set 200 status code and stay on index.html

improvement: added flash message to set failed login attempts into ui without crashing

TDD: failed test case for an edge case (no email submission)

test fix: removed contraction to escape string literals and match server.py

Merge pull request #11 from SiRipo92/fix/login-unknown-email-crash

refactor: rename client_with_mock_data to mock_client for brevity

chore: defined test cases to run for happy + sad paths + edge cases

test: add happy path for competition place deduction - issue #6

test: add happy path tests for competition place deduction - issue #6

test: add sad path for unknown competition - issue #6

test: add sad path for unknown club - issue #6

fix: handle unknown club with next() and guard clause - issue #6

test: add characterisation test for numberOfPlaces type consistency - issue #6

refactor: PEP8 fixes in purchasePlaces - snake_case and spacing

Merge pull request #12 from SiRipo92/fix/competition-places-not-deducted

refactor: PEP8 fixes in show_summary - snake_case and spacing

Merge pull request #13 from SiRipo92/fix/login-unknown-email-crash

refactor: PEP8 fixes in purchasePlaces - snake_case in function name

Merge pull request #14 from SiRipo92/fix/competition-places-not-deducted

refactor: use pytest.mark.parametrize for sad path tests - issue #2

refactor: added missing annotation for sad paths/paramaterize

Merge pull request #15 from SiRipo92/fix/login-unknown-email-crash

refactor: use pytest.mark.parametrize for sad path tests - issue #6