` com `text-sm text-muted-foreground` (precedentes: `Nenhum resultado.` no shell e `emptyState` do `SuggestionsPanel` E11). Componente novo seria camada sem volatilidade. + +### Componentes / mudanças + +- **`HOME_SEARCH_SUGGEST_EMPTY_MESSAGE`** (`src/lib/campaignHomeSearchMessages.ts`): nova constante de copy — "Nenhuma sugestão ainda — priorize municípios na planilha de projeção." (texto recomendado A da intenção, que diz o que é, por quê e o que desbloqueia). Morar ao lado de `HOME_SEARCH_GENERIC_ERROR_MESSAGE`/`HOME_SEARCH_STAFF_ONLY_MESSAGE`, que já são a casa das strings do search. +- **`HomeSearchResultsShell`** (`src/components/campaign/dashboard/HomeSearchResultsShell.tsx`): nova condição `showSuggestEmpty` — `resultKind === 'suggest' && results.status === 'success' && !homeSearchHasAnyHits(results.data)` — renderizando `
` com a mensagem, no mesmo lugar e estilo do `Nenhum resultado.` do modo busca (simetria byte a byte com o padrão existente). O modo busca (`query.isActive`) e o estado com hits ficam intocados.
+- **Migration:** nenhuma — sem schema.
+- **Access / Consent:** nenhum — UI pura; nenhum dado novo exposto.
+- **UI:** Impeccable A — só copy em superfície existente, sem rearranjo (a intenção já classifica como B mas restringe a "só copy/label"; o canvas diz N/A). Sem shape/craft/polish.
+
+### Dados → forma (se aplicável)
+
+N/A — estado vazio de UI, sem métrica nova (pergunta 3 de data-presentation não dispara).
+
+## Decisões de engenharia
+
+**D1 — Onde vive o empty (A vs B vs C):** ver "Opções consideradas" acima. Custo de reverter: nulo (uma condição no shell).
+
+**D2 — Texto único compartilhado vs copy por papel:**
+
+- **Opções:** A) mensagem única para todos os papéis | B) copy específica por papel (coordenador vs assessor).
+- **Recomendação:** A — a regra real do ranking (`rankHomeSearchSuggestMunicipalities`) mostra que **ambos** os papéis podem nascer com lista vazia: unrestricted filtra `priority: alta` (vazio em banco novo — o caso da intenção); advisor rankeia a carteira, que é vazia se o coordenador ainda não atribuiu municípios (caso raro mas real). A intenção recomendava C ("assessor não tem empty") para a pergunta em aberto, mas a regra real contradiz a premissa — a carteira só "sempre lista algo" quando o assessor já tem atribuição. Com os dois papéis capazes de vazio, uma mensagem única honesta é mais simples que plumbing de role até o cliente para um edge raro.
+- **Rejeitadas:** B (role-aware) porque exigiria expor o role no client state do search para um caso de canto; o núcleo honesto da mensagem ("não há sugestões ainda") vale para ambos.
+
+**D3 — Estratégia do e2e (intenção: "asserção para o vazio ou expectativa tolerante a ambos, sem enfraquecer o contrato"):**
+
+- **Opções:** A) estado determinístico por teste — o B126 existente ganha um município `alta` explícito (contrato original preservado: região `Sugestões` com dados) + dois testes novos de vazio (FAB e Início) | B) expectativa tolerante (`region 'Sugestões'` OR mensagem de vazio) no B126.
+- **Recomendação:** A — cada estado vira contrato explícito e verificado, em qualquer ambiente. O banco de teste e2e nasce sem `alta` (migrações não seedam prioridade) e o cleanup do fixture reseta `priority: 'normal'` dos municípios tocados (campaignE2EFixtures.ts:431).
+- **Refinamento pós-simplify (race paralelo):** a config roda `fullyParallel` com workers ≥2 num banco compartilhado — o município `alta` pinado pelo B126 fica vivo do `payload.update` ao cleanup e vazaria para testes zero-alta rodando em paralelo (a classe do miss #73). Os dois testes OPS29 novos usam então um **advisor sem municípios administrados**: o scope do suggest é a carteira (vazia), imune a qualquer `alta` global. O caso unrestricted (coordenador/candidato) continua coberto pelos unit tests do shell, que não ramificam por papel. O teste do FAB também aguarda o `suggestResponse` antes das asserções (o POST acontece no focus).
+- **Rejeitadas:** B — OR tolerante é mais fraco: verifica "algum estado honesto" sem nunca provar nenhum dos dois por extenso; e o contrato da intenção ("passa em qualquer estado") já é atendido por A com asserções mais fortes. A variante "zero-alta via fixture default" sem o advisor foi rejeitada pelo race acima.
+
+**D4 — Edge de exclusão de contexto (FAB em página de detalhe):** `filterHomeSearchResponseForContext` pode zerar a lista filtrada mesmo com hits no servidor (único `alta` = o município que você está vendo; FAB em `/campanha/municipios/
Nenhum resultado.
: null} + {showSuggestEmpty ? ( +{HOME_SEARCH_SUGGEST_EMPTY_MESSAGE}
+ ) : null} ) } diff --git a/src/lib/campaignHomeSearchMessages.ts b/src/lib/campaignHomeSearchMessages.ts index 4084fbf8..7009f579 100644 --- a/src/lib/campaignHomeSearchMessages.ts +++ b/src/lib/campaignHomeSearchMessages.ts @@ -3,3 +3,6 @@ export const HOME_SEARCH_STAFF_ONLY_MESSAGE = 'Somente a equipe de campanha pode usar a busca do Início.' export const HOME_SEARCH_GENERIC_ERROR_MESSAGE = 'Não foi possível buscar. Tente novamente.' + +export const HOME_SEARCH_SUGGEST_EMPTY_MESSAGE = + 'Nenhuma sugestão ainda — priorize municípios na planilha de projeção.' diff --git a/tests/e2e/campaignHomeActions.e2e.spec.ts b/tests/e2e/campaignHomeActions.e2e.spec.ts index d35966d0..ab2c7dc9 100644 --- a/tests/e2e/campaignHomeActions.e2e.spec.ts +++ b/tests/e2e/campaignHomeActions.e2e.spec.ts @@ -103,6 +103,29 @@ test.describe('Início — busca global (B47)', () => { await group.locator(`a[href$="/campanha/municipios/${municipality.slug}"]`).click() await page.waitForURL(new RegExp(`^https?://[^/]+/campanha/municipios/${municipality.slug}$`)) }) + + test('staff focused search without curated suggestions shows the honest empty state (OPS29)', async ({ + campaign, + page, + }) => { + const { fixtures } = campaign + // Advisor with no administered municipality: the suggest scope is the + // portfolio (empty), so the empty state is deterministic even if a + // parallel worker pins an `alta` municipality elsewhere (B126 sibling). + const advisor = await fixtures.createCampaignUser('advisor', { + name: fixtures.value('Assessora OPS29'), + }) + + await campaign.login(page, advisor.email!, advisor.password) + // Início renders the suggest payload server-side (initialSuggest), so this + // also covers the SSR path — no POST needed for the empty state (OPS29). + const search = page.getByLabel('Buscar na campanha') + await search.focus() + + const results = page.getByRole('region', { name: 'Resultados da busca' }) + await expect(results).toContainText('Nenhuma sugestão ainda', { timeout: 10000 }) + await expect(results.getByRole('region', { name: 'Sugestões' })).toHaveCount(0) + }) }) test.describe('Início — catálogo de ações (B45)', () => { diff --git a/tests/e2e/campaignMunicipalities.e2e.spec.ts b/tests/e2e/campaignMunicipalities.e2e.spec.ts index c543d99b..f09080ed 100644 --- a/tests/e2e/campaignMunicipalities.e2e.spec.ts +++ b/tests/e2e/campaignMunicipalities.e2e.spec.ts @@ -1068,6 +1068,17 @@ test.describe('Municípios — FAB overlay polish (B126)', () => { const coordinator = await fixtures.createCampaignUser('coordinator', { name: fixtures.value('Coordenador B126'), }) + // The suggest region only renders with curated hits for unrestricted + // roles (B68), so pin one `alta` municipality — deterministic in any + // environment, not only where the projection was imported (OPS29). + const municipality = await fixtures.claimMunicipality() + await fixtures.payload.update({ + collection: 'municipality', + id: municipality.id, + data: { priority: 'alta' }, + depth: 0, + }) + fixtures.touchMunicipality(municipality.id) await campaign.login(page, coordinator.email!, coordinator.password) await page.goto(`${campaign.baseURL}/campanha/municipios`) @@ -1076,6 +1087,12 @@ test.describe('Municípios — FAB overlay polish (B126)', () => { await page.getByRole('button', { name: 'Ações rápidas' }).click() const overlay = page.locator('#CampaignQuickActionsOverlay') await expect(overlay).toBeVisible() + // The Drawer/Dialog slides in (~450ms); `toBeVisible` fires mid-flight and + // focusing the input during the entrance animation can drop the focus + // (and with it the suggest POST). Same settle wait as the B136 sibling. + await overlay.evaluate(async (el) => { + await Promise.all(el.getAnimations().map((animation) => animation.finished)) + }) const registerSignal = overlay.getByRole('link', { name: 'Registrar atualização' }) await expect(registerSignal).toBeVisible() @@ -1094,4 +1111,42 @@ test.describe('Municípios — FAB overlay polish (B126)', () => { // Overlay scopes search suggest — list page has no E11 SuggestionsPanel. await expect(overlay.getByRole('region', { name: 'Sugestões' })).toBeVisible() }) + + test('overlay search without curated suggestions shows the honest empty state (OPS29)', async ({ + campaign, + page, + }) => { + const { fixtures } = campaign + // Advisor with no administered municipality: the suggest scope is the + // portfolio (empty), so the empty state is deterministic even if a + // parallel worker pins an `alta` municipality (B126 sibling) — the + // unrestricted coordinator case stays covered by the unit shell tests. + const advisor = await fixtures.createCampaignUser('advisor', { + name: fixtures.value('Assessor OPS29'), + }) + + await campaign.login(page, advisor.email!, advisor.password) + await page.goto(`${campaign.baseURL}/campanha/municipios`) + await expect(campaignPageChrome(page, 'Municípios')).toBeVisible() + + await page.getByRole('button', { name: 'Ações rápidas' }).click() + const overlay = page.locator('#CampaignQuickActionsOverlay') + await expect(overlay).toBeVisible() + + const search = overlay.getByLabel('Buscar na campanha') + const suggestResponse = page.waitForResponse( + (resp) => + resp.url().includes('/campanha/home-search') && + resp.request().method() === 'POST' && + resp.request().postDataJSON()?.mode === 'suggest', + ) + await search.focus() + await suggestResponse + + // The advisor has no portfolio yet (OPS29) — the suggest region must say + // so instead of rendering blank. + const results = overlay.getByRole('region', { name: 'Resultados da busca' }) + await expect(results).toContainText('Nenhuma sugestão ainda', { timeout: 10000 }) + await expect(overlay.getByRole('region', { name: 'Sugestões' })).toHaveCount(0) + }) }) diff --git a/tests/unit/campaignHomeSearch.unit.spec.tsx b/tests/unit/campaignHomeSearch.unit.spec.tsx index 48c3f186..7d0e6442 100644 --- a/tests/unit/campaignHomeSearch.unit.spec.tsx +++ b/tests/unit/campaignHomeSearch.unit.spec.tsx @@ -5,6 +5,8 @@ import { CampaignHomeSearch } from '@/components/campaign/dashboard/CampaignHome import { HomeSearchProvider } from '@/components/campaign/dashboard/HomeSearchContext' import { HomeSearchResultsProvider } from '@/components/campaign/dashboard/HomeSearchResultsContext' import { HomeSearchResultsShell } from '@/components/campaign/dashboard/HomeSearchResultsShell' +import type { HomeSearchSuccessResponse } from '@/lib/campaignHomeSearchHits' +import { HOME_SEARCH_SUGGEST_EMPTY_MESSAGE } from '@/lib/campaignHomeSearchMessages' const providerValue = { query: { raw: 'ca', debounced: 'ca', isActive: true }, @@ -16,6 +18,38 @@ const providerValue = { uiFocused: true, } +const suggestFocusedProviderValue = { + ...providerValue, + query: { raw: '', debounced: '', isActive: false }, +} + +const suggestSuccessEmpty: HomeSearchSuccessResponse = { + status: 'success', + resultKind: 'suggest', + municipalities: [], + territories: [], + advisors: [], + leaderships: [], + stateDeputies: [], + activities: [], + demands: [], + scopeMunicipalities: [], +} + +const suggestSuccessWithHits: HomeSearchSuccessResponse = { + ...suggestSuccessEmpty, + municipalities: [ + { + kind: 'municipality', + slug: 'salvador', + name: 'Salvador', + region: 'Região Metropolitana', + priority: 'alta', + votePosition2022: null, + }, + ], +} + afterEach(() => { cleanup() }) @@ -83,4 +117,88 @@ describe('HomeSearchResultsShell', () => { expect(screen.getByTestId('home-search-suggest-skeleton')).toBeTruthy() expect(screen.queryByText('Group stub')).toBeNull() }) + + it('shows the honest empty message when suggest succeeds without hits (OPS29)', () => { + render( +Group stub
+Group stub
+Group stub
+Group stub
+