Skip to content

Feat/dev 237 structure api test#1609

Merged
pikonha merged 21 commits intodevfrom
feat/DEV-237_structure_API_test
Feb 11, 2026
Merged

Feat/dev 237 structure api test#1609
pikonha merged 21 commits intodevfrom
feat/DEV-237_structure_API_test

Conversation

@LeonardoVieira1630
Copy link
Member

@LeonardoVieira1630 LeonardoVieira1630 commented Feb 6, 2026

Summary

  • Establish testing guidelines and rules in .claude/rules/testing.md
  • Add reference test implementations for treasury service and controller
  • Migrate Jest config to CommonJS format
  • Refactor delegation-percentage tests

What's included

  • Testing rules: Guidelines covering test pyramid, test doubles strategy (prefer stubs/fakes over mocks), AAA pattern, deterministic tests, and useful tools (Hono testClient, Testcontainers, MSW)
  • Treasury tests: Unit tests for service layer and integration tests for controller endpoints
  • Provider cache tests: Unit tests for the caching layer

LeonardoVieira1630 and others added 6 commits February 5, 2026 21:35
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@LeonardoVieira1630 LeonardoVieira1630 self-assigned this Feb 6, 2026
@vercel
Copy link

vercel bot commented Feb 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
anticapture Ready Ready Preview, Comment Feb 11, 2026 6:36pm
anticapture-storybook Ready Ready Preview, Comment Feb 11, 2026 6:36pm

Request Review

@pikonha pikonha merged commit 7827d8c into dev Feb 11, 2026
4 checks passed
@pikonha pikonha deleted the feat/DEV-237_structure_API_test branch February 11, 2026 18:49
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