test(agenda): a régua do agora tinha uma janela de um minuto em que a spec discordava do componente - #736
test(agenda): a régua do agora tinha uma janela de um minuto em que a spec discordava do componente#736melgarafael wants to merge 1 commit into
Conversation
Dois PRs de contribuidores diferentes (#717 e #718) ficaram vermelhos hoje num arquivo que nenhum dos dois tocou. O `e2e` dos dois começou às **22:00:12** e **22:00:24**, e é exatamente isso que explica o vermelho. `GradeDaAgenda.tsx` desenha a régua quando 0 <= (hora - 7) * 60 + minuto <= (21 - 7 + 1) * 60 — o limite de cima é `<= 900`, então ela ainda aparece às **22:00:59**. A spec comparava `hora >= 7 && hora <= 21` e exigia ausência a partir das 22h. Varri as duas réguas minuto a minuto: minutos em que a régua ANTIGA discordava do componente: 1 [(22, 0)] minutos em que a régua NOVA discorda: 0 **Um minuto em 1440**, e ele pegou dois PRs no mesmo dia — porque o CI enfileira e as execuções saem em lote, no mesmo instante. A raridade não protege quando os disparos são correlacionados. Não é defeito de produto: a régua às 22:00 em ponto é borda de desenho e não incomoda ninguém. O defeito era a spec duplicar a regra **por aproximação**. Continua duplicando (uma spec de Playwright não importa helper de componente cliente), mas agora duplica a conta, não o resumo dela — e a mensagem de falha passa a dizer a hora e o minuto medidos, que é o dado que faltava para alguém entender o vermelho sem reabrir o log. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
ECC Tools / Security EvidenceCommit: Security evidence gate passed (success) No security-sensitive scanner-evidence gap detected. Mode: enforce Scanned 1 changed file(s). No missing scanner-evidence signal was detected. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / PR Risk TaxonomyCommit: PR taxonomy review recommended (neutral) Detected 1 PR taxonomy bucket(s): CI/CD Recommendation. Scanned 1 changed file(s). Roadmap taxonomy buckets: CI/CD RecommendationCI, dependency, coverage, and contract signals should be routed into follow-up checks or verification work. Signals:
Paths:
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Reference Set ReadinessCommit: Reference set readiness gaps detected (neutral) Reference evidence present for 0/7 areas (0%) across 1 changed file(s). This check is based on files changed in this PR. Repository-level readiness is still reported by
Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
ECC Tools / Hosted Promotion ReadinessCommit: Hosted promotion readiness passed (success) No hosted promotion evidence gaps detected across 1 changed file(s); 0 corpus scenarios had matching evidence. This check compares PR file changes against the evaluator/RAG promotion corpus in No evaluator corpus scenarios matched this PR. Check publication was denied or unavailable. An app owner must enable Checks: read and write, and the installation owner must approve the updated permission. |
1 participant
Dois PRs de contribuidores (#717 e #718) ficaram vermelhos hoje num arquivo que nenhum dos dois tocou. Persegui até o fim em vez de chamar de instabilidade, e o resultado é uma aritmética simples.
A janela
GradeDaAgenda.tsxdesenha a régua quandoO limite de cima é
<= 900, então ela ainda aparece às 22:00:59. A spec comparavahora >= 7 && hora <= 21e exigia ausência a partir das 22h.Varri as duas réguas minuto a minuto:
E os dois
e2ecomeçaram às 22:00:12 e 22:00:24.Por que "um minuto em 1440" não é desprezível aqui
Porque os disparos são correlacionados: o CI enfileira e as execuções saem em lote, no mesmo instante. A raridade de um evento não protege quando as tentativas não são independentes — dois PRs entraram na mesma janela porque entraram na mesma fila.
O que NÃO é
Não é defeito de produto: a régua às 22:00 em ponto é borda de desenho e não incomoda ninguém. O defeito era a spec duplicar a regra por aproximação.
Ela continua duplicando — uma spec de Playwright não importa helper de componente cliente —, mas agora duplica a conta, não o resumo dela. E a mensagem de falha passa a dizer a hora e o minuto medidos, que é o dado que faltava para entender o vermelho sem reabrir o log.
NÃO MEDIDO
Não rodei a spec (precisa da stack de e2e local). O que está medido é a equivalência das duas réguas, minuto a minuto, e o
typecheckexit 0.