test : added unit tests for mergeContributionDays helper#801
test : added unit tests for mergeContributionDays helper#801tmdeveloper007 wants to merge 2 commits into
Conversation
|
@TESTPERSONAL is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Tests must import from source — not reimplement the function locally.
The test file re-implements the function being tested inside the test itself. This defeats the purpose of testing — changes to the real implementation won't fail these tests.
Fix: import the actual function from its source file and test that import. Example:
import { safeCompare } from '../src/lib/crypto'
// then test safeCompare directlyAlso fix:
- Add
"test": "vitest run"toscriptsinpackage.json - Add
vitest.config.tswithresolve.alias: { '@': path.resolve(__dirname, 'src') } - Add EOF newline to test file
20f271a to
c784d65
Compare
|
Deployment failed with the following error: |
Closes Priyanshu-byte-coder#781 Added test/mergeContributionDays.test.ts covering: - Empty input returns empty object - Single account: all dates preserved - Two accounts: sums overlapping dates, preserves non-overlapping - Date keys stay in YYYY-MM-DD format - Three+ accounts: correct sum accumulation - No overlap between accounts handled - Original maps are not mutated - Correct total across all dates Impact: All 10 tests pass. Multi-account contribution aggregation validated.
…test for type checking
|
This pull request is fully up-to-date with the latest upstream merges, all review items are addressed, local tests are passing cleanly, and it is fully ready to be merged! 🚀 |
Closes #781.
Summary of What Has Been Done:
Added test/mergeContributionDays.test.ts with 10 vitest tests covering the mergeContributionDays function from src/app/api/metrics/contributions/route.ts.
Changes Made:
New file: test/mergeContributionDays.test.ts
Test coverage:
Impact it Made:
All 10 tests pass. Multi-account contribution aggregation validated.