Conversation
## Linear Issue: PAV-128 ## Branch flow - [x] Este PR é uma feature/fix/chore destinada a `develop`. - [x] Este PR não contém alteração direta ou fluxo indevido para `master`. - [x] A branch foi criada a partir da base prevista pelo fluxo do projeto. ## Objetivo Corrigir e ampliar o coverage do sistema após as alterações no catálogo de vagas, filtros e cache, adicionando testes para fluxos críticos de backend, painel admin e frontend, e estabilizando a execução da suíte do front_admin. ## Escopo O que faz parte desta entrega: - Ampliação/adição de testes unitários e de integração no backend (SavedJobs/Application Notes, busca de vagas, match de perfil, scripts de backfill de PII e seed). - Adição de testes no `front_admin` para layouts, dashboard, observabilidade, permissões, scrapers e usuários. - Adição de testes no `frontend` para schemas do novo dashboard, utils, `HelpTab` e ajuste de mocks de login. - Ajuste de configuração do Vitest no `front_admin` (`maxWorkers: 2`). O que está fora do escopo: - Alterações de comportamento funcional em produção. - Alterações de schema, migrations ou contratos de API. - Refatorações estruturais não relacionadas a coverage. ## Resumo das alterações - **Backend / SavedJobs**: testes de integração para notas de candidatura (`list`, `create`, `update`, `delete`) e testes unitários do `ApplicationNotesService` com validação de ownership e escopo por usuário/vaga/nota. - **Backend / Jobs**: novo teste de integração de `GET /jobs/search` cobrindo filtros, cache, fallback legacy, paginação e match score; ampliação do teste do `searchJobs.service`; novo teste do `JobProfileMatchService`. - **Backend / Scripts**: testes para `backfillUserPii` (dry-run e `--write`) e para `seed`. - **Front Admin**: cobertura de `NotificationButton`, `SidebarFooter`, `TimeFilter`, `DashboardPage`, `PlatformChart`, hooks do dashboard, `ObservabilityPage`/`ObservabilityPanelCard`, `PermissionsPage`, `ScrapersPage`/`useScrapers`, `UsersPage` e `UserList`. - **Frontend**: cobertura de schemas runtime do new_dashboard, utils (`jobModelFilters`, `locationFilters`, `searchKeywords`), `HelpTab` e ajuste de testes de loading do login. - **Config**: `front_admin/vite.config.ts` com `maxWorkers: 2`. ## Arquivos e módulos afetados - `backend/tests/integration/routes/savedJobs.routes.test.ts` - `backend/tests/integration/routes/searchJobs.routes.test.ts` - `backend/tests/unit/modules/jobs/jobProfileMatch.service.test.ts` - `backend/tests/unit/modules/jobs/searchJobs.service.test.ts` - `backend/tests/unit/modules/savedJobs/applicationNotes.service.test.ts` - `backend/tests/unit/scripts/backfillUserPii.test.ts` - `backend/tests/unit/scripts/seed.test.ts` - `front_admin/tests/app/layouts/**` - `front_admin/tests/modules/**` - `front_admin/vite.config.ts` - `frontend/tests/unit/components/login/RigthSide.test.tsx` - `frontend/tests/unit/new_dashboard/**` ## Como testar ```bash # backend cd backend npm test npm run test:coverage # front_admin cd front_admin npm test npm run test:coverage # frontend cd frontend npm test npm run test:coverage ``` ## Validation - [x] Lint executado - [x] Testes relevantes executados - [x] Coverage permanece dentro do mínimo do projeto - [x] Build executado quando aplicável - [x] Validação manual realizada quando aplicável - [ ] Documentação atualizada quando necessária ## Impacto de banco / migration - [x] Não altera schema nem migrations - [ ] Altera schema ou migrations Detalhes: N/A ## Impacto em contratos/API - [x] Não altera contratos - [ ] Altera endpoint, payload, schema ou comportamento de API Detalhes: N/A — os testes apenas validam contratos existentes. ## Impacto de configuração / infraestrutura - [ ] Não altera configuração - [x] Altera env, Docker, filas, cache, serviços ou infraestrutura Detalhes: ajuste em `front_admin/vite.config.ts` adicionando `maxWorkers: 2` para estabilizar a execução paralela dos testes. Sem impacto em runtime de produção. ## Impacto de segurança - [x] Sem impacto relevante - [ ] Impacto de segurança avaliado Detalhes: os testes de `backfillUserPii` apenas exercitam a criptografia/hash de PII já existente; nenhuma mudança em lógica de segurança. ## Impacto de dados pessoais / privacidade - [x] Sem tratamento novo ou alteração de dados pessoais - [ ] Impacto avaliado Detalhes: os testes utilizam dados fictícios (fixtures) para validar o pipeline de PII já existente. ## Compatibilidade / dependências - [x] Não depende de outra task/PR - [ ] Existe dependência ou ordem de deploy/merge Detalhes: N/A ## Evidências - Saída de `npm run test:coverage` (backend, front_admin, frontend) anexada na thread do card/PR. - Commits relacionados: `Develop - Alterações no catálogo de vagas, filtros e cache`, `Develop (#258)`, `feat(PAV-128): correção do coverage do sistema`. ## Riscos - **Baixo**: alterações restritas a testes e configuração de execução do Vitest. - Possível aumento do tempo total da suíte por conta do `maxWorkers: 2` no `front_admin` (trade-off para estabilidade). ## Rollback Reverter o merge deste PR restaura o estado anterior dos testes e da configuração do Vitest. Como não há alterações de produção, o rollback é seguro e não impacta dados, contratos ou infraestrutura. ## Checklist final - [x] Escopo limitado ao card - [x] Não inclui segredos, `.env`, certificados ou tokens - [x] Não inclui arquivos temporários, builds, caches ou artefatos desnecessários - [x] Não executa deploy como efeito desta PR - [x] Critérios de aceite do card foram conferidos individualmente - [x] Alterações de contrato possuem testes e documentação correspondentes - [x] Alterações em banco/migrations foram validadas quando aplicável > Consulte o guia de contribuição do projeto para as regras completas.# PAV-128 — Correção do coverage do sistema ## Linear Issue: PAV-128 ## Branch flow - [x] Este PR é uma feature/fix/chore destinada a `develop`. - [x] Este PR não contém alteração direta ou fluxo indevido para `master`. - [x] A branch foi criada a partir da base prevista pelo fluxo do projeto. ## Objetivo Corrigir e ampliar o coverage do sistema após as alterações no catálogo de vagas, filtros e cache, adicionando testes para fluxos críticos de backend, painel admin e frontend, e estabilizando a execução da suíte do front_admin. ## Escopo O que faz parte desta entrega: - Ampliação/adição de testes unitários e de integração no backend (SavedJobs/Application Notes, busca de vagas, match de perfil, scripts de backfill de PII e seed). - Adição de testes no `front_admin` para layouts, dashboard, observabilidade, permissões, scrapers e usuários. - Adição de testes no `frontend` para schemas do novo dashboard, utils, `HelpTab` e ajuste de mocks de login. - Ajuste de configuração do Vitest no `front_admin` (`maxWorkers: 2`). O que está fora do escopo: - Alterações de comportamento funcional em produção. - Alterações de schema, migrations ou contratos de API. - Refatorações estruturais não relacionadas a coverage. ## Resumo das alterações - **Backend / SavedJobs**: testes de integração para notas de candidatura (`list`, `create`, `update`, `delete`) e testes unitários do `ApplicationNotesService` com validação de ownership e escopo por usuário/vaga/nota. - **Backend / Jobs**: novo teste de integração de `GET /jobs/search` cobrindo filtros, cache, fallback legacy, paginação e match score; ampliação do teste do `searchJobs.service`; novo teste do `JobProfileMatchService`. - **Backend / Scripts**: testes para `backfillUserPii` (dry-run e `--write`) e para `seed`. - **Front Admin**: cobertura de `NotificationButton`, `SidebarFooter`, `TimeFilter`, `DashboardPage`, `PlatformChart`, hooks do dashboard, `ObservabilityPage`/`ObservabilityPanelCard`, `PermissionsPage`, `ScrapersPage`/`useScrapers`, `UsersPage` e `UserList`. - **Frontend**: cobertura de schemas runtime do new_dashboard, utils (`jobModelFilters`, `locationFilters`, `searchKeywords`), `HelpTab` e ajuste de testes de loading do login. - **Config**: `front_admin/vite.config.ts` com `maxWorkers: 2`. ## Arquivos e módulos afetados - `backend/tests/integration/routes/savedJobs.routes.test.ts` - `backend/tests/integration/routes/searchJobs.routes.test.ts` - `backend/tests/unit/modules/jobs/jobProfileMatch.service.test.ts` - `backend/tests/unit/modules/jobs/searchJobs.service.test.ts` - `backend/tests/unit/modules/savedJobs/applicationNotes.service.test.ts` - `backend/tests/unit/scripts/backfillUserPii.test.ts` - `backend/tests/unit/scripts/seed.test.ts` - `front_admin/tests/app/layouts/**` - `front_admin/tests/modules/**` - `front_admin/vite.config.ts` - `frontend/tests/unit/components/login/RigthSide.test.tsx` - `frontend/tests/unit/new_dashboard/**` ## Como testar ```bash # backend cd backend npm test npm run test:coverage # front_admin cd front_admin npm test npm run test:coverage # frontend cd frontend npm test npm run test:coverage ``` ## Validation - [x] Lint executado - [x] Testes relevantes executados - [x] Coverage permanece dentro do mínimo do projeto - [x] Build executado quando aplicável - [x] Validação manual realizada quando aplicável - [ ] Documentação atualizada quando necessária ## Impacto de banco / migration - [x] Não altera schema nem migrations - [ ] Altera schema ou migrations Detalhes: N/A ## Impacto em contratos/API - [x] Não altera contratos - [ ] Altera endpoint, payload, schema ou comportamento de API Detalhes: N/A — os testes apenas validam contratos existentes. ## Impacto de configuração / infraestrutura - [ ] Não altera configuração - [x] Altera env, Docker, filas, cache, serviços ou infraestrutura Detalhes: ajuste em `front_admin/vite.config.ts` adicionando `maxWorkers: 2` para estabilizar a execução paralela dos testes. Sem impacto em runtime de produção. ## Impacto de segurança - [x] Sem impacto relevante - [ ] Impacto de segurança avaliado Detalhes: os testes de `backfillUserPii` apenas exercitam a criptografia/hash de PII já existente; nenhuma mudança em lógica de segurança. ## Impacto de dados pessoais / privacidade - [x] Sem tratamento novo ou alteração de dados pessoais - [ ] Impacto avaliado Detalhes: os testes utilizam dados fictícios (fixtures) para validar o pipeline de PII já existente. ## Compatibilidade / dependências - [x] Não depende de outra task/PR - [ ] Existe dependência ou ordem de deploy/merge Detalhes: N/A ## Evidências - Saída de `npm run test:coverage` (backend, front_admin, frontend) anexada na thread do card/PR. - Commits relacionados: `Develop - Alterações no catálogo de vagas, filtros e cache`, `Develop (#258)`, `feat(PAV-128): correção do coverage do sistema`. ## Riscos - **Baixo**: alterações restritas a testes e configuração de execução do Vitest. - Possível aumento do tempo total da suíte por conta do `maxWorkers: 2` no `front_admin` (trade-off para estabilidade). ## Rollback Reverter o merge deste PR restaura o estado anterior dos testes e da configuração do Vitest. Como não há alterações de produção, o rollback é seguro e não impacta dados, contratos ou infraestrutura. ## Checklist final - [x] Escopo limitado ao card - [x] Não inclui segredos, `.env`, certificados ou tokens - [x] Não inclui arquivos temporários, builds, caches ou artefatos desnecessários - [x] Não executa deploy como efeito desta PR - [x] Critérios de aceite do card foram conferidos individualmente - [x] Alterações de contrato possuem testes e documentação correspondentes - [x] Alterações em banco/migrations foram validadas quando aplicável
hltav
requested review from
Benevanio,
nayarakarinesilva and
pedrosilvaadev
as code owners
October 1, 2026 09:46
Benevanio
approved these changes
Oct 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Melhora do coverage do Frontend.