Skip to content

test(edge): matriz padronizada de status por endpoint#506

Closed
adm01-debug wants to merge 1 commit into
mainfrom
codex/standardize-status-table-by-endpoint
Closed

test(edge): matriz padronizada de status por endpoint#506
adm01-debug wants to merge 1 commit into
mainfrom
codex/standardize-status-table-by-endpoint

Conversation

@adm01-debug
Copy link
Copy Markdown
Owner

@adm01-debug adm01-debug commented May 29, 2026

Draft aberto durante limpeza de branches para resgatar trabalho não mergeado.

Conteúdo (1 commit)

  • Define matriz canônica de status (sucesso/validação/auth/404/conflito/500) por Edge Function em docs/API.md
  • Refatora edge_integration.test.ts para validar a matriz nos 4 endpoints principais (ai-recommendations, bitrix-sync, expert-chat, quote-approval)

⚠️ Atenção do revisor

Branch ~1.400 commits atrás do main. Verificar se os testes ainda batem com o comportamento atual das funções antes de rebase/merge.

https://claude.ai/code/session_01QpLctApdgHrT7hBAS3dk71


Generated by Claude Code


Summary by cubic

Padroniza a matriz de status HTTP por endpoint e garante essa consistência com testes de integração nas 4 Edge Functions principais. Documenta a matriz de referência em docs/API.md.

  • Refactors

    • Reestrutura supabase/functions/tests/edge_integration.test.ts com matriz de cenários e helper de asserção por endpoint.
    • Adiciona casos para auth ausente, validação e conflito/idempotência em bitrix-sync.
    • Remove testes de latência e suites de endpoints fora do escopo.
  • Migration

    • Rode a suíte e alinhe status codes dos endpoints que divergirem da matriz antes do merge.
    • A branch está ~1.4k commits atrás do main; valide o comportamento atual antes de rebase.

Written for commit edd59e2. Summary will update on new commits.

Review in cubic

@supabase
Copy link
Copy Markdown

supabase Bot commented May 29, 2026

This pull request has been ignored for the connected project doufsxqlfjyuvxuezpln due to reaching the limit of concurrent preview branches.
Go to Project Integrations Settings ↗︎ if you wish to update this limit.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 29, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 3146bca3-7ca6-47d6-8c0c-cab459ba0132

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Owner Author

Fechando após análise de viabilidade.

Motivo: branch sobre um main de histórico reescrito (sem ancestral comum → conflito total). Mais importante, o conteúdo está superado no main atual:

  1. A seção "Matriz padronizada de status por endpoint" já existe em docs/API.md (linha 91).
  2. Os testes de integração edge foram reorganizados numa estrutura modular (supabase/functions/tests/integration/edge/<função>/*_test.ts) cobrindo os mesmos endpoints (bitrix-sync, expert-chat, etc.). O edge_integration.test.ts virou apenas um agregador de imports. Trazer de volta o arquivo monolítico de 150 linhas desta branch seria regressão de arquitetura de testes.

Decisão de engenharia: fechar. Nada a salvar com segurança.


Generated by Claude Code

@adm01-debug adm01-debug deleted the codex/standardize-status-table-by-endpoint branch May 29, 2026 14:02
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.

1 participant