Skip to content

User-Register#21

Closed
relinar wants to merge 8 commits intotanjaq:mainfrom
relinar:main
Closed

User-Register#21
relinar wants to merge 8 commits intotanjaq:mainfrom
relinar:main

Conversation

@relinar
Copy link
Copy Markdown

@relinar relinar commented Mar 18, 2026

Exercise 1 vs Exercise 2:

Exercise 1 (real API): Tests the actual system end-to-end. Coverage <100% because some real service paths aren’t hit. Runs slower (~23 s) due to full validation logic.

Exercise 2 (mocked API): Mocks the email service, so all code paths are executed and coverage is 100%. Runs much faster (~1 s) since no real logic or external calls are processed.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 18, 2026

API & Mocked Tests and Coverage

Purpose: Validate README requirements for regular tests, mocked email tests, coverage, and runtime comparison.
Status: ✅ PASS
Regular tests (app.test.js): ✅ PASS (27381 ms)
Mocked tests (app.mock.test.js): ✅ PASS (1368 ms)
Coverage gate (both runs at 100%): ✅ PASS
Runtime gate (regular > mocked): ✅ PASS
Regular coverage: lines=100%, statements=100%, functions=100%, branches=100%
Mocked coverage: lines=100%, statements=100%, functions=100%, branches=100%
Run details: https://github.com/tanjaq/User-Register/actions/runs/23315721791
PR checks: https://github.com/tanjaq/User-Register/pull/21/checks

@tanjaq tanjaq closed this Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants