| {{ c.CODIGO }} |
@@ -30,11 +41,11 @@
{% if total_paginas and total_paginas > 1 %}
{% if pagina > 1 %}
- ← Anterior
+ ← Anterior
{% endif %}
Página {{ pagina }} de {{ total_paginas }} ({{ total }} clientes)
{% if pagina < total_paginas %}
- Próxima →
+ Próxima →
{% endif %}
{% endif %}
diff --git a/tests/test_db_clientes.py b/tests/test_db_clientes.py
index 3012d48..4d9efc8 100644
--- a/tests/test_db_clientes.py
+++ b/tests/test_db_clientes.py
@@ -106,6 +106,29 @@ def test_listar_clientes_ordenacao_invalida_usa_padrao(cliente_id):
assert any(c["CODIGO"] == "999901" for c in resultado)
+def test_listar_clientes_direcao_asc_e_desc(cliente_id):
+ # Dois clientes de teste com códigos diferentes, mesma busca por RAZAO,
+ # pra comparar a ordem entre asc/desc de forma determinística.
+ outro_id = db_clientes.criar_cliente(dict(CLIENTE_TESTE, CODIGO="999905"))
+ try:
+ asc = db_clientes.listar_clientes(busca="TESTE PAINEL RAZAO", ordenar_por="codigo", direcao="asc")
+ desc = db_clientes.listar_clientes(busca="TESTE PAINEL RAZAO", ordenar_por="codigo", direcao="desc")
+ codigos_asc = [c["CODIGO"] for c in asc]
+ codigos_desc = [c["CODIGO"] for c in desc]
+ assert codigos_asc == sorted(codigos_asc)
+ assert codigos_desc == sorted(codigos_desc, reverse=True)
+ finally:
+ _limpar("999905")
+
+
+def test_listar_clientes_direcao_invalida_usa_padrao_asc(cliente_id):
+ # Não deve quebrar nem virar SQL arbitrário — cai em ASC.
+ resultado = db_clientes.listar_clientes(
+ busca="999901", ordenar_por="codigo", direcao="ASC; DROP TABLE CLIENTE"
+ )
+ assert resultado[0]["CODIGO"] == "999901"
+
+
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
diff --git a/tests/test_main.py b/tests/test_main.py
index bb95d8d..09fa8fc 100644
--- a/tests/test_main.py
+++ b/tests/test_main.py
@@ -22,7 +22,7 @@ def test_docs_desabilitado_retorna_404():
def test_raiz_redireciona_para_clientes_com_login(monkeypatch):
from app.routers import clientes as router_clientes
- monkeypatch.setattr(router_clientes.db_clientes, "listar_clientes", lambda busca=None, limite=100, offset=0: [])
+ monkeypatch.setattr(router_clientes.db_clientes, "listar_clientes", lambda busca=None, limite=100, offset=0, ordenar_por="razao", direcao="asc": [])
monkeypatch.setattr(router_clientes.db_clientes, "contar_clientes", lambda busca=None, ordenar_por="razao": 0)
resposta = client.get("/", headers=_AUTH_HEADERS, follow_redirects=False)
assert resposta.status_code == 307
diff --git a/tests/test_router_clientes.py b/tests/test_router_clientes.py
index 92b7831..3681d3f 100644
--- a/tests/test_router_clientes.py
+++ b/tests/test_router_clientes.py
@@ -57,7 +57,7 @@ def test_listar_clientes(monkeypatch):
monkeypatch.setattr(
router_clientes.db_clientes,
"listar_clientes",
- lambda busca=None, limite=100, offset=0, ordenar_por="razao": [
+ lambda busca=None, limite=100, offset=0, ordenar_por="razao", direcao="asc": [
{"ID": 1, "CODIGO": "000001", "RAZAO": "Cliente Um", "FANTASIA": "Um", "CNPJ": "11222333000181"}
],
)
@@ -71,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, ordenar_por="razao": [
+ lambda busca=None, limite=100, offset=0, ordenar_por="razao", direcao="asc": [
{"ID": i, "CODIGO": f"{i:06d}", "RAZAO": f"Cliente {i}", "FANTASIA": f"C{i}", "CNPJ": "11222333000181"}
for i in range(1, 6)
],
@@ -89,7 +89,7 @@ def test_listar_clientes_pagina_com_controles_de_paginacao(monkeypatch):
def test_listar_clientes_ordenar_por_invalido_cai_no_padrao(monkeypatch):
capturado = {}
- def _listar_clientes(busca=None, limite=100, offset=0, ordenar_por="razao"):
+ def _listar_clientes(busca=None, limite=100, offset=0, ordenar_por="razao", direcao="asc"):
capturado["ordenar_por"] = ordenar_por
return []
@@ -103,6 +103,43 @@ def test_listar_clientes_ordenar_por_invalido_cai_no_padrao(monkeypatch):
assert capturado["ordenar_por"] == "codigo"
+def test_cabecalhos_da_lista_alternam_direcao(monkeypatch):
+ monkeypatch.setattr(
+ router_clientes.db_clientes, "listar_clientes", lambda **kwargs: []
+ )
+ monkeypatch.setattr(router_clientes.db_clientes, "contar_clientes", lambda busca=None, ordenar_por="razao": 0)
+
+ # Sem ordenar_por/direcao na URL: padrão é razao/asc. Clicar de novo em
+ # "Razão" deve levar pra desc; clicar em "Código" (coluna diferente da
+ # atual) deve levar pra asc (não continua o desc de outra coluna).
+ resposta = client.get("/clientes")
+ assert 'ordenar_por=razao&direcao=desc' in resposta.text
+ assert 'ordenar_por=codigo&direcao=asc' in resposta.text
+
+ # Já ordenado por codigo/desc: clicar em "Código" de novo deve voltar pra asc.
+ resposta = client.get("/clientes?ordenar_por=codigo&direcao=desc")
+ assert 'ordenar_por=codigo&direcao=asc' in resposta.text
+ # E as outras colunas oferecem asc (primeira vez que seriam clicadas).
+ assert 'ordenar_por=razao&direcao=asc' in resposta.text
+
+
+def test_listar_clientes_direcao_invalida_cai_no_padrao(monkeypatch):
+ capturado = {}
+
+ def _listar_clientes(busca=None, limite=100, offset=0, ordenar_por="razao", direcao="asc"):
+ capturado["direcao"] = direcao
+ return []
+
+ monkeypatch.setattr(router_clientes.db_clientes, "listar_clientes", _listar_clientes)
+ monkeypatch.setattr(router_clientes.db_clientes, "contar_clientes", lambda busca=None, ordenar_por="razao": 0)
+
+ client.get("/clientes?direcao=coisa_invalida")
+ assert capturado["direcao"] == "asc"
+
+ client.get("/clientes?direcao=desc")
+ assert capturado["direcao"] == "desc"
+
+
def test_form_novo_preenche_codigo_sugerido(monkeypatch):
monkeypatch.setattr(router_clientes.db_clientes, "proximo_codigo_sugerido", lambda: "000243")
resposta = client.get("/clientes/novo")