test: add full test coverage for notes, ues, ue-modules, ajustements, enseignements, users #145

Merged

djalim merged 3 commits from PMPR-tests-coverage into develop 2026-04-26 17:10:39 +00:00

Conversation 0 Commits 3 Files Changed 27

+4077 -16

djalim commented 2026-04-26 16:26:21 +00:00

Collaborator

Copy Link

Copy Source

• Unit tests (mock DB + API) for all missing endpoints
• Integration tests (Drizzle direct) for all missing entities
• E2E tests (handler + real DB) for all missing endpoints
• Robustness tests: invalid inputs, SQL injection, type errors, business rule violations
• Seed helpers: seedNotes, seedUeModules, seedEnseignements, seedAjustements
• Add test:coverage and test:coverage:html tasks to deno.json

Tests expose known handler bugs (marked [BUG] in test names):

• ajustements PUT/DELETE: .where() without and() modifies all rows for student
• Missing try/catch in modules, users, enseignements handlers
• Whitespace accepted as valid string values
• No type or business rule validation (note bounds, coeff >= 0)

- Unit tests (mock DB + API) for all missing endpoints - Integration tests (Drizzle direct) for all missing entities - E2E tests (handler + real DB) for all missing endpoints - Robustness tests: invalid inputs, SQL injection, type errors, business rule violations - Seed helpers: seedNotes, seedUeModules, seedEnseignements, seedAjustements - Add test:coverage and test:coverage:html tasks to deno.json Tests expose known handler bugs (marked [BUG] in test names): - ajustements PUT/DELETE: .where() without and() modifies all rows for student - Missing try/catch in modules, users, enseignements handlers - Whitespace accepted as valid string values - No type or business rule validation (note bounds, coeff >= 0)

djalim added 1 commit 2026-04-26 16:26:21 +00:00

test: add full test coverage for notes, ues, ue-modules, ajustements, enseignements, users ... 2f4d8db1bf

- Unit tests (mock DB + API) for all missing endpoints
- Integration tests (Drizzle direct) for all missing entities
- E2E tests (handler + real DB) for all missing endpoints
- Robustness tests: invalid inputs, SQL injection, type errors, business rule violations
- Seed helpers: seedNotes, seedUeModules, seedEnseignements, seedAjustements
- Add test:coverage and test:coverage:html tasks to deno.json

Tests expose known handler bugs (marked [BUG] in test names):
- ajustements PUT/DELETE: .where() without and() modifies all rows for student
- Missing try/catch in modules, users, enseignements handlers
- Whitespace accepted as valid string values
- No type or business rule validation (note bounds, coeff >= 0)

djalim added 1 commit 2026-04-26 17:02:19 +00:00

fix: correct handler bugs exposed by test suite ... b0930b8da2

- ajustements [numEtud]/[idUE]: fix .where() missing and() — PUT/DELETE
  were applying only numEtud condition, modifying all rows for a student
- modules/users/enseignements POST: add try/catch, return 500 on invalid JSON
- modules/[idModule] PUT: add try/catch + type check on nom (string required)
- modules POST: add .trim() check to reject whitespace-only id/nom
- users POST: add .trim() check to reject whitespace-only id/nom/prenom
- ues POST: add .trim() check to reject whitespace-only nom
- notes POST: add type check (typeof number) and bounds check (0 ≤ note ≤ 20)
- ue-modules POST: add coeff >= 0 validation

Update robustness tests to reflect fixed behavior (remove [BUG] labels,
replace assertRejects with status code assertions).

djalim added 1 commit 2026-04-26 17:07:26 +00:00

chore: formated tests 714486f43c

djalim merged commit 714486f43c into develop 2026-04-26 17:10:39 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

anys clement djalim ibra admin

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: admin/PolyMPR#145