The latest updates on your projects. Learn more about Vercel for Git ↗︎

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
7.5% Duplication on New Code

See analysis details on SonarCloud

Olá @Nathan-P2, boa proposta! A estrutura geral está no caminho certo (usa app(), aliases @/, NotFoundError, dados do IBGE oficial), mas há alguns ajustes antes do merge:

Bloqueadores:

1. Testes usam sintaxe CommonJS em um projeto ESM

// ❌ atual
const axios = require('axios');

// ✅ correto
import axios from 'axios';
import { describe, test, expect } from 'vitest';

O projeto usa "type": "module" — require() não funciona aqui. Os testes falharão ao rodar. Veja tests/ibge-uf-v1.test.js como referência para a sintaxe correta.

2. Testes de erro sem expect.assertions()

Os blocos try/catch dos testes de erro não garantem que as assertions rodem:

// ❌ se a requisição não lançar, o catch nunca executa e o teste passa silenciosamente
try {
  await axios.get(requestUrl);
} catch (error) {
  expect(response.status).toBe(404);
}

// ✅
test('...', async () => {
  expect.assertions(1);
  try {
    await axios.get(requestUrl);
  } catch (error) {
    expect(error.response.status).toBe(404);
  }
});

Sugestões (não bloqueiam):

3. response.status(status) é frágil

Passar o status do IBGE diretamente é arriscado — se o IBGE retornar 4xx ou 5xx, o handler repassa o código sem tratar. Prefira:

return response.status(200).json(data);

4. Path parameters não documentados no OpenAPI

As rotas com {regionCode} (ex: /ibge/regioes/v1/{regionCode}) não têm o parâmetro descrito no array parameters. Adicione conforme o padrão dos outros endpoints em ibge.json.

5. Faltam testes de CORS

Adicione ao final do arquivo de testes:

import { testCorsForRoute } from './helpers/cors';
testCorsForRoute('/api/ibge/regioes/v1');
testCorsForRoute('/api/ibge/regioes/v1/1');
testCorsForRoute('/api/ibge/uf/v1/regiao/S');

Com as correções 1 e 2, o PR está quase pronto para merge.