Files
DeskcommCRM/tests/sonda-painel-inbox.ts
jmpoandClaude Opus 5 f691b9c8f8 merge: 198 commits da main — duas convergências e um desfecho a corrigir
O fork estava 198 commits atrás. A `main` já tinha absorvido o PR #200
(canal Zernio) e renumerado as migrations que vieram daqui, então boa
parte do conflito era o MESMO trabalho com dois números.

─── O que era duplicata ───

`0117_canal_zernio_vocabulario` e `0118_zernio_envio` diferem de `0131` e
`0132` em UMA linha: o número no comentário do cabeçalho. São as minhas,
renumeradas lá. Ficam as de lá — numeração é verdade compartilhada, e
manter as duas deixaria o mesmo schema com dois nomes.

`0119_contato_lookup_telefone` e `0122_telefone_do_lid` PARECIAM a mesma
convergência e não são: uma acrescenta `phone_lookup_at` (fila do cron de
backfill), a outra `wa_lid` (coluna gerada + dedup). Convivem.

─── O defeito que o merge criaria, e que ninguém veria ───

A `0139_kind_check_completo` reconstrói `agent_inbox_items_kind_check`, e
a minha `0120` também. Pelo timestamp a minha roda DEPOIS — e a lista
dela não conhecia `midia_nao_lida` nem `contact_proposal_expired`.
Aplicar as duas em ordem, num clone atualizado, APAGARIA dois kinds em
uso: quem reconstrói por último é quem manda, e uma lista incompleta não
é "aditiva", é uma remoção silenciosa. A `0120` passou a trazer a lista
inteira.

No `baseline.sql` continua UM bloco só para essa constraint, agora com os
kinds dos dois lados — a regra do #159, que o `update.sh` de todo clone
depende.

─── Onde discordei da main, e por quê ───

`zernio.ts` — `isConfigured()` fica `true`, e não
`zernioCredsFromEnv() !== null`.

A preocupação de lá é real: `isConfigured() === true` não pode implicar
"sem credencial", porque o handler grava `sent` quando `send()` não
lança. Só que quem fecha esse par mudou de lugar — `send()` LANÇA
`zernio_not_configured`, e o caso descrito naquele comentário ("devolve
`{externalId: null}` SEM lançar") não existe mais no arquivo.

Exigir env reintroduziria um defeito medido: `resolveZernioCreds` procura
primeiro na SESSÃO e só depois no ambiente. Uma instalação que conectou
pelo botão tem a credencial no banco e nada no `.env` — a checagem por
env diria "não configurado" para um canal conectado e funcionando, e toda
mensagem ficaria em `queued` sem tentar sair.

Mas o teste de lá estava certo no DESFECHO: sem credencial nenhuma, o
certo é `queued` com o motivo, não `failed`. Isso não estava atendido de
nenhum dos dois lados — o `catch` do handler marcava `failed`. Agora
`notConfigured` é traduzido para `queued`, ao lado do
`storage_sign_failed` que já tinha tratamento próprio: falta de
credencial não é falha DESTA mensagem, é canal ainda não conectado, e
marcar `failed` faria o follow-up desistir de algo que sai sozinho assim
que alguém conectar.

Os dez casos de `messages-handler-canal-intermediado` passam.

─── O que fica vermelho, e não é daqui ───

`lib/ai/dispatcher/rate-limit.test.ts` — 5 casos estourando o timeout de
15s. Falha também com os arquivos da main trocados um a um, então não
vem desta resolução; o CI de lá está verde, então é diferença de
ambiente. Fica registrado, não escondido.

3647 casos passam. typecheck, lint:channels e build limpos.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-10 11:57:33 -03:00

125 lines
5.4 KiB
TypeScript

/**
* Sonda do bloco 2.8 — o painel do inbox parou de mentir?
*
* Prova nos DOIS sentidos, que é o padrão que passei a exigir do time depois do
* teste de mutação do @DevVivo: não basta ver a tela certa quando tudo funciona.
* Um painel que mostra dados no caminho feliz e diz "Sem leads." quando a
* leitura falha continua mentindo — só que mais raramente.
*
* A) CAMINHO FELIZ — o contato tem lead e atividade, e a tela mostra os dois,
* com o motivo humano e sem UUID.
* B) FALHA INJETADA — a rota devolve 500 e a tela precisa dizer que NÃO
* CONSEGUIU LER. Se disser "Sem leads.", reprova: isso é
* afirmação sobre o negócio em cima de um erro.
*
* Run: E2E_PORT=3020 npx tsx tests/sonda-painel-inbox.ts
*/
import { chromium, type Page } from "@playwright/test";
import { BASE, EVIDENCE, login, shotPage } from "./qa-helpers";
const CONTATO = "Ana Souza LGPD E2E";
const ROTA = "**/crm-summary**";
const resultados: { nome: string; ok: boolean; detalhe: string }[] = [];
function record(nome: string, ok: boolean, detalhe: string): void {
resultados.push({ nome, ok, detalhe });
console.info(`${ok ? "PASS " : "FALHA"} ${nome} — ${detalhe}`);
}
/** Abre o inbox e CLICA na conversa — passar id na URL não seleciona nada. */
async function abrirConversa(page: Page): Promise<void> {
await page.goto(`${BASE}/app/inbox`, { waitUntil: "networkidle" });
// Espera pela RESOLUÇÃO, não por um título. Esperar "Leads recentes" me deu
// um falso verde: o cabeçalho existe ANTES dos dados, então as asserções
// negativas ("não diz Sem leads") passaram por ausência — verde vácuo, a
// mesma armadilha que virou lei no §7.4 e que eu quebrei no meu próprio teste.
// A resolução verdadeira é a resposta da rota, que também dispara quando ela
// está interceptada com 500.
const resposta = page.waitForResponse((r) => r.url().includes("/crm-summary"), {
timeout: 20_000,
});
await page.getByText(CONTATO).first().click();
await resposta;
// E depois, o DOM assentado: alguma das TRÊS saídas possíveis da seção.
const painel = page.getByRole("complementary").last();
await painel
.getByText(/Sem leads\.|Não consegui ler|R\$|Movido de/)
.first()
.waitFor({ state: "visible", timeout: 15_000 });
}
async function main(): Promise<void> {
const browser = await chromium.launch();
const ctx = await browser.newContext({ viewport: { width: 1440, height: 900 } });
const page = await ctx.newPage();
await login(page, "admin");
// ---------- A) caminho feliz ----------
await abrirConversa(page);
// `.last()` porque HÁ DOIS <aside>: a navegação lateral e o painel. Contar
// com "o único" foi o erro que eu mesmo cataloguei três vezes hoje.
const painel = page.getByRole("complementary").last();
const textoOk = await painel.innerText();
// POSITIVA, não negativa. "Não diz 'Sem leads'" é trivialmente verdadeiro numa
// seção vazia — foi assim que este mesmo teste me deu falso verde na 1ª rodada.
// Agora exige ver a coisa: o título do lead e um rótulo de atividade.
const temLead = /Titular LGPD E2E/.test(textoOk);
const temAtividade = /Mudou de estágio|Anotação|Atendimento da IA/.test(textoOk);
record(
"2.8.a-dados",
temLead && temAtividade,
`lead ${temLead ? "visível" : "AUSENTE"} · atividade ${temAtividade ? "visível" : "AUSENTE"}`,
);
record(
"2.8.b-motivo",
/Movido de .+ para .+/.test(textoOk),
/Movido de .+ para .+/.test(textoOk)
? "o motivo humano aparece na 2ª superfície"
: "o motivo humano NÃO aparece",
);
const uuid = textoOk.match(/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}/i);
record("2.8.c-uuid", !uuid, uuid ? `UUID visível: ${uuid[0]}` : "nenhum UUID visível");
await shotPage(page, "wave-3-2.8-painel-ok.png");
// ---------- B) falha injetada ----------
// Injetada de propósito, não dependendo do bug: o gate continua valendo
// depois que a causa raiz for embora, e reprova se alguém reintroduzir
// "erro vira lista vazia" daqui a seis meses.
await page.route(ROTA, (r) =>
r.fulfill({ status: 500, contentType: "application/json", body: '{"error":{"code":"boom"}}' }),
);
await abrirConversa(page);
const textoFalha = await painel.innerText();
// "Nenhuma demanda aberta." entra na lista pelo MESMO motivo das outras
// três: com a leitura falhando, é uma afirmação sobre o negócio feita em
// cima de um erro. A seção nasceu depois da sonda; se não entrasse aqui, o
// painel poderia voltar a mentir por um caminho que ninguém vigia.
const mente = /Sem leads\.|Sem atividade\.|Sem pedidos\.|Nenhuma demanda aberta\./.test(
textoFalha,
);
const confessa = /Não consegui ler/i.test(textoFalha);
record(
"2.8.d-falha-confessa",
confessa && !mente,
mente
? "MENTE: com a leitura falhando, afirma 'Sem leads/atividade' — erro disfarçado de fato"
: confessa
? "diz que não conseguiu ler, e oferece tentar de novo"
: "não mente, mas também não diz nada",
);
await shotPage(page, "wave-3-2.8-painel-falha.png");
await browser.close();
const reprovados = resultados.filter((r) => !r.ok);
console.info(`\n=== ${resultados.length - reprovados.length}/${resultados.length} ===`);
console.info(`prints em ${EVIDENCE}`);
if (reprovados.length) process.exitCode = 1;
}
void main();