feat: log de acoes/envios, ordenacao/filtro, codigo sequencial, campos opcionais

- Tabela LOG_PAINEL (nova, ja criada no banco de dev) + app/db/log.py:
  toda criacao/edicao/exclusao de cliente e contato fica registrada
  (usuario, acao, sucesso/erro, detalhe). Tela /log mostra esse
  historico.
- app/n8n_client.py: leitura opcional (N8N_API_URL/N8N_API_KEY) da
  Data Table pagamentos_processados do n8n, mostrando os envios
  automaticos de chave na mesma tela /log. Degrada de forma segura
  (lista vazia) se nao configurado ou se a chamada falhar.
- Lista de clientes ganha ordenacao configuravel (codigo/razao/
  fantasia/CNPJ) via whitelist, alem da busca que ja existia.
- CODIGO duplicado agora BLOQUEIA de verdade na criacao/edicao (sem
  bypass) — antes so avisava com opcao de confirmar mesmo assim.
- Formulario de novo cliente sugere o proximo CODIGO sequencial
  (editavel), ignorando a faixa reservada para dados de teste
  (>= 900000) para nao ser contaminado por sobra de teste no banco.
- IMPLANTACAO/MENSALIDADE/JUROS_MORA/JUROS_MULTA/DATA_CONTRATO/
  VENCIMENTO_CONTRATO deixam de ser obrigatorios — em branco, gravam
  default (0,00 / hoje) em vez de bloquear o cadastro. Só CODIGO/CNPJ/
  RAZAO/FANTASIA continuam obrigatorios.
- Limpeza: removidas 10 linhas de teste vazadas (CODIGO 999902) que
  ficaram no banco de dev por sessoes de teste interrompidas.

69/69 testes passando (16 novos).
This commit is contained in:
Gustavo
2026-08-13 14:04:24 -03:00
parent bb8fd596b1
commit 9ed0bf57c2
20 changed files with 689 additions and 170 deletions
+22
View File
@@ -87,6 +87,28 @@ def test_excluir_cliente_sem_dependencias(cliente_id):
assert db_clientes.obter_cliente(cliente_id) is None
def test_listar_clientes_ordena_por_codigo(cliente_id):
resultado = db_clientes.listar_clientes(busca="TESTE PAINEL RAZAO", ordenar_por="codigo")
assert resultado[0]["CODIGO"] == "999901"
def test_listar_clientes_ordenacao_invalida_usa_padrao(cliente_id):
# "ordenar_por" fora da whitelist não deve quebrar nem virar SQL arbitrário
# — cai no padrão (RAZAO).
resultado = db_clientes.listar_clientes(busca="TESTE PAINEL RAZAO", ordenar_por="ID; DROP TABLE CLIENTE")
assert any(c["CODIGO"] == "999901" for c in resultado)
def test_proximo_codigo_sugerido_ignora_faixa_de_teste(cliente_id):
# CLIENTE_TESTE usa CODIGO "999901" (faixa reservada para teste, >= 900000)
# — a sugestão não pode considerar isso, senão qualquer sobra de dado de
# teste no banco faria a sugestão pular para a casa dos 999xxx.
sugerido = db_clientes.proximo_codigo_sugerido()
assert int(sugerido) < 900000
assert len(sugerido) == 6
assert db_clientes.codigo_em_uso(sugerido) is False
def test_excluir_cliente_com_contato_bloqueia(cliente_id):
conn = get_connection()
try:
+42
View File
@@ -0,0 +1,42 @@
import pytest
from app.db import log as db_log
from app.db.connection import get_connection
def _limpar_teste():
conn = get_connection()
try:
with conn.cursor() as cur:
cur.execute("DELETE FROM LOG_PAINEL WHERE USUARIO = 'teste-log'")
finally:
conn.close()
@pytest.fixture
def limpar():
_limpar_teste()
yield
_limpar_teste()
def test_registrar_e_listar(limpar):
db_log.registrar("teste-log", "criar", "cliente", "1", True)
db_log.registrar("teste-log", "excluir", "cliente", "1", False, "tem contato")
linhas = db_log.listar(limite=10)
linhas_teste = [l for l in linhas if l["USUARIO"] == "teste-log"]
assert len(linhas_teste) == 2
# ORDER BY CRIADO_EM DESC, ID DESC -> a mais recente (excluir) vem primeiro.
assert linhas_teste[0]["ACAO"] == "excluir"
assert linhas_teste[0]["SUCESSO"] == 0
assert linhas_teste[0]["DETALHE"] == "tem contato"
assert linhas_teste[1]["ACAO"] == "criar"
assert linhas_teste[1]["SUCESSO"] == 1
def test_contar(limpar):
antes = db_log.contar()
db_log.registrar("teste-log", "criar", "cliente", "1", True)
depois = db_log.contar()
assert depois == antes + 1
+56
View File
@@ -0,0 +1,56 @@
import httpx
import pytest
from app import n8n_client
def test_nao_configurado_retorna_vazio(monkeypatch):
monkeypatch.delenv("N8N_API_URL", raising=False)
monkeypatch.delenv("N8N_API_KEY", raising=False)
assert n8n_client.configurado() is False
assert n8n_client.listar_envios() == []
def test_configurado_com_ambas_variaveis(monkeypatch):
monkeypatch.setenv("N8N_API_URL", "https://devn8n.exemplo.com")
monkeypatch.setenv("N8N_API_KEY", "chave-teste")
assert n8n_client.configurado() is True
def test_listar_envios_falha_de_rede_retorna_vazio(monkeypatch):
monkeypatch.setenv("N8N_API_URL", "https://devn8n.exemplo.com")
monkeypatch.setenv("N8N_API_KEY", "chave-teste")
def _get_com_erro(*args, **kwargs):
raise httpx.ConnectError("falhou")
monkeypatch.setattr(n8n_client.httpx, "get", _get_com_erro)
assert n8n_client.listar_envios() == []
def test_listar_envios_sucesso_retorna_linhas(monkeypatch):
monkeypatch.setenv("N8N_API_URL", "https://devn8n.exemplo.com/")
monkeypatch.setenv("N8N_API_KEY", "chave-teste")
linhas = [{"codigo_cliente": "000001", "status": "enviado"}]
class _RespostaFalsa:
def raise_for_status(self):
pass
def json(self):
return {"data": linhas}
capturado = {}
def _get_falso(url, headers=None, params=None, timeout=None):
capturado["url"] = url
capturado["headers"] = headers
return _RespostaFalsa()
monkeypatch.setattr(n8n_client.httpx, "get", _get_falso)
resultado = n8n_client.listar_envios(limite=10)
assert resultado == linhas
assert capturado["url"] == "https://devn8n.exemplo.com/api/v1/data-tables/bJHhFUDwFJ30YJVt/rows"
assert capturado["headers"]["X-N8N-API-KEY"] == "chave-teste"
+142 -99
View File
@@ -3,6 +3,7 @@ Testa só a camada HTTP: monta um FastAPI() com apenas este router, sobrescreve
exigir_login (sem exigir Basic Auth real) e faz monkeypatch das funções de
app.db.clientes usadas pelo router — não toca no banco de verdade.
"""
import pytest
from fastapi import FastAPI
from fastapi.testclient import TestClient
@@ -21,11 +22,42 @@ CSRF_TESTE = "teste-csrf-token"
client.cookies.set("csrf_token", CSRF_TESTE)
@pytest.fixture(autouse=True)
def _sem_log_real(monkeypatch):
"""
O router registra cada ação em LOG_PAINEL (app/db/log.py) — por padrão
isso bateria no banco real. Substitui por um no-op nos testes que não
são especificamente sobre o log (test_router_clientes não testa
app/db/log.py, isso já é coberto em tests/test_db_log.py).
"""
monkeypatch.setattr(router_clientes.db_log, "registrar", lambda *args, **kwargs: None)
def _form_valido(codigo: str = "000001") -> dict:
"""Monta um dict de formulário válido (todos os campos preenchidos)."""
dados = {campo: "x" for campo in router_clientes.db_clientes.CAMPOS_CLIENTE}
dados.update(
{
"CODIGO": codigo,
"CNPJ": "11222333000181",
"IMPLANTACAO": "0",
"MENSALIDADE": "100",
"JUROS_MORA": "0",
"JUROS_MULTA": "0",
"DATA_CONTRATO": "2026-01-01",
"VENCIMENTO_CONTRATO": "2027-01-01",
"DIA_VENCIMENTO": "10",
"csrf_token": CSRF_TESTE,
}
)
return dados
def test_listar_clientes(monkeypatch):
monkeypatch.setattr(
router_clientes.db_clientes,
"listar_clientes",
lambda busca=None, limite=100, offset=0: [
lambda busca=None, limite=100, offset=0, ordenar_por="razao": [
{"ID": 1, "CODIGO": "000001", "RAZAO": "Cliente Um", "FANTASIA": "Um", "CNPJ": "11222333000181"}
],
)
@@ -39,7 +71,7 @@ def test_listar_clientes_pagina_com_controles_de_paginacao(monkeypatch):
monkeypatch.setattr(
router_clientes.db_clientes,
"listar_clientes",
lambda busca=None, limite=100, offset=0: [
lambda busca=None, limite=100, offset=0, ordenar_por="razao": [
{"ID": i, "CODIGO": f"{i:06d}", "RAZAO": f"Cliente {i}", "FANTASIA": f"C{i}", "CNPJ": "11222333000181"}
for i in range(1, 6)
],
@@ -54,6 +86,30 @@ def test_listar_clientes_pagina_com_controles_de_paginacao(monkeypatch):
assert "Próxima" in resposta.text
def test_listar_clientes_ordenar_por_invalido_cai_no_padrao(monkeypatch):
capturado = {}
def _listar_clientes(busca=None, limite=100, offset=0, ordenar_por="razao"):
capturado["ordenar_por"] = ordenar_por
return []
monkeypatch.setattr(router_clientes.db_clientes, "listar_clientes", _listar_clientes)
monkeypatch.setattr(router_clientes.db_clientes, "contar_clientes", lambda busca=None: 0)
client.get("/clientes?ordenar_por=coisa_invalida_qualquer")
assert capturado["ordenar_por"] == "razao"
client.get("/clientes?ordenar_por=codigo")
assert capturado["ordenar_por"] == "codigo"
def test_form_novo_preenche_codigo_sugerido(monkeypatch):
monkeypatch.setattr(router_clientes.db_clientes, "proximo_codigo_sugerido", lambda: "000243")
resposta = client.get("/clientes/novo")
assert resposta.status_code == 200
assert 'value="000243"' in resposta.text
def test_detalhe_cliente_mostra_formulario_de_edicao_de_contato(monkeypatch):
monkeypatch.setattr(
router_clientes.db_clientes,
@@ -74,39 +130,71 @@ def test_detalhe_cliente_mostra_formulario_de_edicao_de_contato(monkeypatch):
assert 'value="75999090001"' in resposta.text
def test_criar_cliente_codigo_duplicado_exige_confirmacao(monkeypatch):
def test_criar_cliente_codigo_duplicado_bloqueia_sem_bypass(monkeypatch):
monkeypatch.setattr(router_clientes.db_clientes, "codigo_em_uso", lambda codigo, ignorar_id=None: True)
criado = {}
monkeypatch.setattr(
router_clientes.db_clientes, "criar_cliente", lambda dados: criado.setdefault("ok", True) and 42
)
dados_form = {campo: "000001" if campo == "CODIGO" else "x" for campo in router_clientes.db_clientes.CAMPOS_CLIENTE}
dados_form.update(
{
"IMPLANTACAO": "0",
"MENSALIDADE": "100",
"JUROS_MORA": "0",
"JUROS_MULTA": "0",
"DATA_CONTRATO": "2026-01-01",
"VENCIMENTO_CONTRATO": "2027-01-01",
"DIA_VENCIMENTO": "10",
"CNPJ": "11222333000181",
"csrf_token": CSRF_TESTE,
}
)
dados_form = _form_valido()
# Sem "confirmar_codigo_duplicado": não deve criar, deve pedir confirmação.
resposta = client.post("/clientes/novo", data=dados_form)
assert resposta.status_code == 200
assert "já está em uso" in resposta.text
assert "ok" not in criado
# Com confirmação: cria.
dados_form["confirmar_codigo_duplicado"] = "1"
# Não existe mais bypass: mesmo insistindo, sem trocar o código, continua bloqueado.
resposta = client.post("/clientes/novo", data=dados_form)
assert resposta.status_code == 200
assert "já está em uso" in resposta.text
assert "ok" not in criado
def test_criar_cliente_campo_obrigatorio_faltando_bloqueia(monkeypatch):
monkeypatch.setattr(router_clientes.db_clientes, "codigo_em_uso", lambda codigo, ignorar_id=None: False)
criado = {}
monkeypatch.setattr(
router_clientes.db_clientes, "criar_cliente", lambda dados: criado.setdefault("ok", True) and 42
)
dados_form = _form_valido()
dados_form["RAZAO"] = ""
resposta = client.post("/clientes/novo", data=dados_form)
assert resposta.status_code == 200
assert "RAZAO" in resposta.text
assert "ok" not in criado
def test_criar_cliente_campos_administrativos_em_branco_usa_defaults(monkeypatch):
"""
IMPLANTACAO/MENSALIDADE/JUROS_MORA/JUROS_MULTA/DATA_CONTRATO/
VENCIMENTO_CONTRATO não são mais obrigatórios — em branco, o cliente é
criado mesmo assim, com defaults (0,00 / hoje).
"""
import datetime
monkeypatch.setattr(router_clientes.db_clientes, "codigo_em_uso", lambda codigo, ignorar_id=None: False)
capturado = {}
monkeypatch.setattr(
router_clientes.db_clientes, "criar_cliente", lambda dados: capturado.setdefault("dados", dados) and 42
)
dados_form = _form_valido()
for campo in ("IMPLANTACAO", "MENSALIDADE", "JUROS_MORA", "JUROS_MULTA", "DATA_CONTRATO", "VENCIMENTO_CONTRATO"):
dados_form[campo] = ""
resposta = client.post("/clientes/novo", data=dados_form)
assert resposta.status_code == 303
assert criado.get("ok") is True
dados = capturado["dados"]
assert dados["IMPLANTACAO"] == "0.00"
assert dados["MENSALIDADE"] == "0.00"
assert dados["JUROS_MORA"] == "0.00"
assert dados["JUROS_MULTA"] == "0.00"
hoje = datetime.date.today().isoformat()
assert dados["DATA_CONTRATO"] == hoje
assert dados["VENCIMENTO_CONTRATO"] == hoje
def test_criar_cliente_normaliza_cnpj_e_telefones(monkeypatch):
@@ -116,22 +204,12 @@ def test_criar_cliente_normaliza_cnpj_e_telefones(monkeypatch):
router_clientes.db_clientes, "criar_cliente", lambda dados: capturado.setdefault("dados", dados) and 42
)
dados_form = {campo: "x" for campo in router_clientes.db_clientes.CAMPOS_CLIENTE}
dados_form = _form_valido()
dados_form.update(
{
"CODIGO": "000001",
"CNPJ": "11.222.333/0001-81",
"TELEFONE": "(75) 3333-4444",
"CELULAR": "(75) 99909-1780",
"IMPLANTACAO": "0",
"MENSALIDADE": "100",
"JUROS_MORA": "0",
"JUROS_MULTA": "0",
"DATA_CONTRATO": "2026-01-01",
"VENCIMENTO_CONTRATO": "2027-01-01",
"DIA_VENCIMENTO": "10",
"confirmar_dv_invalido": "1",
"csrf_token": CSRF_TESTE,
}
)
@@ -151,21 +229,12 @@ def test_editar_cliente_normaliza_cnpj_e_telefones(monkeypatch):
lambda cliente_id, dados: capturado.setdefault("dados", dados),
)
dados_form = {campo: "x" for campo in router_clientes.db_clientes.CAMPOS_CLIENTE}
dados_form = _form_valido()
dados_form.update(
{
"CODIGO": "000001",
"CNPJ": "111.444.777-35",
"TELEFONE": "(75) 3333-4444",
"CELULAR": "(75) 99909-1780",
"IMPLANTACAO": "0",
"MENSALIDADE": "100",
"JUROS_MORA": "0",
"JUROS_MULTA": "0",
"DATA_CONTRATO": "2026-01-01",
"VENCIMENTO_CONTRATO": "2027-01-01",
"DIA_VENCIMENTO": "10",
"csrf_token": CSRF_TESTE,
}
)
@@ -176,6 +245,22 @@ def test_editar_cliente_normaliza_cnpj_e_telefones(monkeypatch):
assert capturado["dados"]["CELULAR"] == "75999091780"
def test_editar_cliente_codigo_duplicado_bloqueia_sem_bypass(monkeypatch):
monkeypatch.setattr(router_clientes.db_clientes, "codigo_em_uso", lambda codigo, ignorar_id=None: True)
atualizado = {}
monkeypatch.setattr(
router_clientes.db_clientes,
"atualizar_cliente",
lambda cliente_id, dados: atualizado.setdefault("ok", True),
)
dados_form = _form_valido()
resposta = client.post("/clientes/1/editar", data=dados_form)
assert resposta.status_code == 200
assert "já está em uso" in resposta.text
assert "ok" not in atualizado
def test_excluir_cliente_com_dependencia_mostra_erro(monkeypatch):
monkeypatch.setattr(
router_clientes.db_clientes,
@@ -204,83 +289,41 @@ def _limpar_codigo(codigo: str) -> None:
conn.close()
def test_criar_cliente_com_data_e_decimais_vazios_nao_gera_500():
def test_criar_cliente_com_data_e_decimais_vazios_cria_com_defaults():
"""
Integração com o banco real de dev: DATA_CONTRATO/decimais são NOT NULL
e o dev roda em STRICT_TRANS_TABLES — submeter strings vazias deve ser
capturado pelo router (pymysql.err.DataError) e reexibir o formulário
com erro_gravacao, nunca estourar como 500.
Integração com o banco real de dev: DATA_CONTRATO/decimais deixaram de
ser obrigatórios — em branco, o router aplica defaults (0,00 / hoje)
antes de gravar, então o cliente é criado normalmente (303), não gera
erro nem 500.
"""
codigo = "999906"
dados_form = {campo: "x" for campo in router_clientes.db_clientes.CAMPOS_CLIENTE}
dados_form.update(
{
"CODIGO": codigo,
"CNPJ": "11222333000181",
"IMPLANTACAO": "",
"MENSALIDADE": "",
"JUROS_MORA": "",
"JUROS_MULTA": "",
"DATA_CONTRATO": "",
"VENCIMENTO_CONTRATO": "",
"DIA_VENCIMENTO": "10",
"csrf_token": CSRF_TESTE,
}
)
dados_form = _form_valido(codigo)
for campo in ("IMPLANTACAO", "MENSALIDADE", "JUROS_MORA", "JUROS_MULTA", "DATA_CONTRATO", "VENCIMENTO_CONTRATO"):
dados_form[campo] = ""
try:
resposta = client.post("/clientes/novo", data=dados_form)
assert resposta.status_code == 200
assert "Não foi possível salvar" in resposta.text
assert resposta.status_code == 303
finally:
_limpar_codigo(codigo)
def test_criar_cliente_com_ambos_avisos_confirma_ambos_de_uma_vez(monkeypatch):
"""
Reproduz Fix 3: um cliente com CODIGO duplicado E DV de documento
inválido não pode ficar preso em loop entre os dois avisos — confirmar
os dois de uma vez (hidden fields carregados de um aviso para o outro)
deve efetivamente criar o cliente.
"""
monkeypatch.setattr(router_clientes.db_clientes, "codigo_em_uso", lambda codigo, ignorar_id=None: True)
def test_criar_cliente_com_dv_invalido_exige_confirmacao(monkeypatch):
monkeypatch.setattr(router_clientes.db_clientes, "codigo_em_uso", lambda codigo, ignorar_id=None: False)
criado = {}
monkeypatch.setattr(
router_clientes.db_clientes, "criar_cliente", lambda dados: criado.setdefault("ok", True) and 42
)
dados_form = {campo: "000002" if campo == "CODIGO" else "x" for campo in router_clientes.db_clientes.CAMPOS_CLIENTE}
dados_form.update(
{
"IMPLANTACAO": "0",
"MENSALIDADE": "100",
"JUROS_MORA": "0",
"JUROS_MULTA": "0",
"DATA_CONTRATO": "2026-01-01",
"VENCIMENTO_CONTRATO": "2027-01-01",
"DIA_VENCIMENTO": "10",
"CNPJ": "11144477734", # CPF com DV inválido (1 dígito diferente do válido "...735")
"csrf_token": CSRF_TESTE,
}
)
dados_form = _form_valido()
dados_form["CNPJ"] = "11144477734" # CPF com DV inválido (1 dígito diferente do válido "...735")
# Round 1: nem codigo nem dv confirmados -> deve pedir confirmação de código duplicado.
resposta = client.post("/clientes/novo", data=dados_form)
assert resposta.status_code == 200
assert "já está em uso" in resposta.text
assert "ok" not in criado
# Round 2: confirma o código duplicado -> agora deve pedir confirmação do DV inválido.
dados_form["confirmar_codigo_duplicado"] = "1"
# Sem confirmar: pede confirmação, não cria.
resposta = client.post("/clientes/novo", data=dados_form)
assert resposta.status_code == 200
assert "não confere" in resposta.text
assert "ok" not in criado
# A raiz do Fix 3: o form do aviso de DV precisa carregar
# confirmar_codigo_duplicado=1 adiante (senão confirmar só o DV a seguir
# reabriria o aviso de código duplicado, em loop infinito).
assert 'name="confirmar_codigo_duplicado" value="1"' in resposta.text
# Round 3: confirma ambos de uma vez -> deve criar (303), sem cair em outro loop.
# Confirmando: cria.
dados_form["confirmar_dv_invalido"] = "1"
resposta = client.post("/clientes/novo", data=dados_form)
assert resposta.status_code == 303
+8
View File
@@ -1,3 +1,4 @@
import pytest
from fastapi import FastAPI
from fastapi.testclient import TestClient
@@ -17,6 +18,13 @@ CSRF_TESTE = "teste-csrf-token"
client.cookies.set("csrf_token", CSRF_TESTE)
@pytest.fixture(autouse=True)
def _sem_log_real(monkeypatch):
"""Os routers registram cada ação em LOG_PAINEL — sem no-op, bateria no banco real."""
monkeypatch.setattr(router_contatos.db_log, "registrar", lambda *args, **kwargs: None)
monkeypatch.setattr(router_clientes.db_log, "registrar", lambda *args, **kwargs: None)
def test_adicionar_contato_com_sucesso(monkeypatch):
monkeypatch.setattr(router_contatos.db_clientes, "obter_cliente", lambda cid: {"ID": cid, "CODIGO": "000001"})
monkeypatch.setattr(router_contatos.db_contatos, "contato_duplicado", lambda cid, tel, ignorar_id=None: False)
+52
View File
@@ -0,0 +1,52 @@
from fastapi import FastAPI
from fastapi.testclient import TestClient
from app.auth import exigir_login
from app.routers import log as router_log
app_teste = FastAPI()
app_teste.include_router(router_log.router)
app_teste.dependency_overrides[exigir_login] = lambda: "teste"
client = TestClient(app_teste, follow_redirects=False)
def test_pagina_log_mostra_acoes_do_painel(monkeypatch):
monkeypatch.setattr(
router_log.db_log,
"listar",
lambda limite=100: [
{
"ID": 1, "CRIADO_EM": "2026-08-13 10:00:00", "USUARIO": "admin", "ACAO": "criar",
"ENTIDADE": "cliente", "ENTIDADE_ID": "000001", "SUCESSO": 1, "DETALHE": None,
}
],
)
monkeypatch.setattr(router_log.n8n_client, "listar_envios", lambda limite=50: [])
monkeypatch.setattr(router_log.n8n_client, "configurado", lambda: False)
resposta = client.get("/log")
assert resposta.status_code == 200
assert "admin" in resposta.text
assert "criar" in resposta.text
assert "N8N_API_URL" in resposta.text # aviso de integração não configurada
def test_pagina_log_mostra_envios_quando_configurado(monkeypatch):
monkeypatch.setattr(router_log.db_log, "listar", lambda limite=100: [])
monkeypatch.setattr(
router_log.n8n_client,
"listar_envios",
lambda limite=50: [
{
"data_processamento": "2026-08-13T10:00:00", "nome_cliente": "Cliente Teste",
"codigo_cliente": "000001", "competencia": "2026-08", "status": "enviado",
"telefones_enviados": "5575999090000", "telefones_com_erro": "",
}
],
)
monkeypatch.setattr(router_log.n8n_client, "configurado", lambda: True)
resposta = client.get("/log")
assert resposta.status_code == 200
assert "Cliente Teste" in resposta.text
assert "enviado" in resposta.text