Skip to content

Stub leaderboard RPCs in check-cost (no live submits, no flaky cert error) - #75

Merged
IlianP merged 1 commit into
mainfrom
claude/evaluate-design-zip-c7koxm
Oct 3, 2026
Merged

IlianP merged 1 commit into
mainfrom
claude/evaluate-design-zip-c7koxm

Conversation

@IlianP

@IlianP IlianP commented Oct 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • tests/browser/check-cost.mjs clicks "Eintragen" on the win card but never stubbed submit_score, so every run posted a real score to the live Supabase project. Where a proxy breaks the certificate (this sandbox) the request failed with ERR_CERT_AUTHORITY_INVALID, which landed in the page's console errors and made no console errors fail intermittently (also reproducible on an unchanged main).
  • The test now answers every rpc/* leaderboard call locally (submit_score → rank 1 of 1, everything else → []), and lets bump_stat fall through to openGame's own counter stub.
  • Checked the other tests that reference win-submit: only check-cost actually clicked it unstubbed (win-feedback and leaderboard-retry already stub submit_score; i18n-layout only reads selectors).

Test plan

  • check-cost: 6 consecutive runs green (previously failing roughly 1 run in 2–4)
  • win-feedback still green

🤖 Generated with Claude Code

https://claude.ai/code/session_01HjTnVkUHVtQ9PjK9JyR5pE


Generated by Claude Code

…ve project

The test clicks "Eintragen" without a submit_score stub, so every run posted a
real score to Supabase (and failed intermittently with ERR_CERT_AUTHORITY_INVALID
where a proxy breaks the certificate). Answer every leaderboard RPC locally;
the play counters keep openGame's own stub.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HjTnVkUHVtQ9PjK9JyR5pE
@IlianP
IlianP merged commit 34a085f into main Oct 3, 2026
1 check passed
@IlianP
IlianP deleted the claude/evaluate-design-zip-c7koxm branch October 3, 2026 09:43
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