GustavoandClaude Sonnet 5
eed473c5f3
- app/routers/contatos.py: CLIENTE_CONTATO.TELEFONE passa a ser gravado sempre com DDI 55 (normalizar_telefone_com_ddi, mesma normalização já usada em CLIENTE.TELEFONE/CELULAR) — confirmado no node '10 - Validar Telefones' do workflow n8n de produção (ER6Piy9yu93uOVuY) que o telefone é usado exatamente como está gravado, sem nenhuma normalização própria. Um contato cadastrado pelo painel sem o funcionário digitar '55' na frente nunca recebia a chave automática por WhatsApp — falha silenciosa, sem nenhum erro visível. - app/db/clientes.py, app/db/contatos.py, app/db/funcionarios.py: excluir_cliente/atualizar_contato/excluir_contato/redefinir_senha passam a retornar bool (cur.rowcount > 0) em vez de None — um ID inexistente não gera erro nenhum do MySQL (WHERE só não bate com nada), então sem checar rowcount o router não tinha como saber que não alterou/excluiu nada. - app/routers/clientes.py, app/routers/contatos.py, app/routers/funcionarios.py: editar/excluir/redefinir-senha/ alternar-ativo passam a checar essa existência e mostrar 'não encontrado' em vez de registrar 'sucesso' no log e redirecionar pra uma tela que também vai dizer que não encontrou nada. - app/templates/clientes_lista.html: busca não estava com urlencode nos links de paginação/ordenação — um termo de busca com '&' (comum em razão social, ex. 'Fulano & Cia') cortava o resto da query string do link, perdendo página/ordenação. Suíte completa (192 testes) passando, com 13 testes novos cobrindo os casos de 'não encontrado' e o DDI do telefone de contato. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
68 lines
2.6 KiB
Python
68 lines
2.6 KiB
Python
"""
|
|
Testes de integração. Cria um CLIENTE de teste (CODIGO 999902) só para
|
|
pendurar os contatos, e limpa tudo ao final.
|
|
"""
|
|
import pytest
|
|
|
|
from app.db import contatos as db_contatos
|
|
from app.db import clientes as db_clientes
|
|
from tests.test_db_clientes import CLIENTE_TESTE, _limpar
|
|
|
|
|
|
@pytest.fixture
|
|
def cliente_id():
|
|
dados = dict(CLIENTE_TESTE, CODIGO="999902")
|
|
novo_id = db_clientes.criar_cliente(dados)
|
|
yield novo_id
|
|
# Clean up contacts first due to FK constraint, then client
|
|
from app.db.connection import get_connection
|
|
conn = get_connection()
|
|
try:
|
|
with conn.cursor() as cur:
|
|
cur.execute("DELETE FROM CLIENTE_CONTATO WHERE CLIENTE_ID = %s", (novo_id,))
|
|
cur.execute("DELETE FROM CLIENTE WHERE ID = %s", (novo_id,))
|
|
finally:
|
|
conn.close()
|
|
|
|
|
|
def test_inserir_e_listar_contato(cliente_id):
|
|
contato_id = db_contatos.inserir_contato(cliente_id, "Fulano", "75999090001")
|
|
contatos = db_contatos.listar_contatos_por_cliente(cliente_id)
|
|
assert len(contatos) == 1
|
|
assert contatos[0]["ID"] == contato_id
|
|
assert contatos[0]["NOME"] == "Fulano"
|
|
assert contatos[0]["TELEFONE"] == "75999090001"
|
|
|
|
|
|
def test_contato_duplicado(cliente_id):
|
|
db_contatos.inserir_contato(cliente_id, "Fulano", "75999090001")
|
|
assert db_contatos.contato_duplicado(cliente_id, "75999090001") is True
|
|
assert db_contatos.contato_duplicado(cliente_id, "75999090002") is False
|
|
|
|
|
|
def test_contato_duplicado_ignora_proprio_id(cliente_id):
|
|
contato_id = db_contatos.inserir_contato(cliente_id, "Fulano", "75999090001")
|
|
assert db_contatos.contato_duplicado(cliente_id, "75999090001", ignorar_id=contato_id) is False
|
|
|
|
|
|
def test_atualizar_contato(cliente_id):
|
|
contato_id = db_contatos.inserir_contato(cliente_id, "Fulano", "75999090001")
|
|
assert db_contatos.atualizar_contato(contato_id, cliente_id, "Fulano Editado", "75999090003") is True
|
|
contatos = db_contatos.listar_contatos_por_cliente(cliente_id)
|
|
assert contatos[0]["NOME"] == "Fulano Editado"
|
|
assert contatos[0]["TELEFONE"] == "75999090003"
|
|
|
|
|
|
def test_atualizar_contato_inexistente_retorna_false(cliente_id):
|
|
assert db_contatos.atualizar_contato(999999, cliente_id, "Fulano", "75999090001") is False
|
|
|
|
|
|
def test_excluir_contato(cliente_id):
|
|
contato_id = db_contatos.inserir_contato(cliente_id, "Fulano", "75999090001")
|
|
assert db_contatos.excluir_contato(contato_id, cliente_id) is True
|
|
assert db_contatos.listar_contatos_por_cliente(cliente_id) == []
|
|
|
|
|
|
def test_excluir_contato_inexistente_retorna_false(cliente_id):
|
|
assert db_contatos.excluir_contato(999999, cliente_id) is False
|