Skip to content

Use handler database for person referral lookup#2794

Open
cybercraftsolutionsllc wants to merge 1 commit into
stakwork:masterfrom
cybercraftsolutionsllc:codex/person-referral-db-1484
Open

Use handler database for person referral lookup#2794
cybercraftsolutionsllc wants to merge 1 commit into
stakwork:masterfrom
cybercraftsolutionsllc:codex/person-referral-db-1484

Conversation

@cybercraftsolutionsllc
Copy link
Copy Markdown

Summary

  • resolve referred_by UUIDs through the injected people handler database instead of the global database
  • add CreatePerson coverage proving a referral UUID stores the referrer person ID on the new person

Addresses #1484.

Validation

  • go test ./routes -run '^TestPeopleRoutes$' -count=1
  • git diff --check

Attempted but currently blocked by existing stale mocks:

  • go test ./handlers -run '^TestCreatePerson$/should_create_user_with_referred_by_person_id_from_referral_uuid$' -count=1
  • go test ./handlers -run '^$' -count=1

Both handler commands fail before running this test because mocks.Database is missing GetBountyByUnlockCode, matching the existing repo-wide handler test compile blocker.

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