From 506aa077ae8471de4864ec2fd34c1a5077050637 Mon Sep 17 00:00:00 2001 From: Eduardo Calil Date: Mon, 4 Apr 2016 20:49:08 -0300 Subject: [PATCH] Coloca namespaces nas URLs --- materia/test_materia.py | 30 ++++++++++++------------- materia/test_materia_urls.py | 2 +- materia/views.py | 35 ++++++++++++++++------------- parlamentares/test_parlamentares.py | 20 +++++++++-------- parlamentares/views.py | 3 ++- 5 files changed, 48 insertions(+), 42 deletions(-) diff --git a/materia/test_materia.py b/materia/test_materia.py index d628856bb..04fdfddb4 100644 --- a/materia/test_materia.py +++ b/materia/test_materia.py @@ -98,7 +98,7 @@ def test_materia_anexada_submit(client): materia_anexada = MateriaLegislativa.objects.get(numero=32, ano=2004) # Testa POST - response = client.post(reverse('materia_anexada', + response = client.post(reverse('materia:materia_anexada', kwargs={'pk': materia_principal.pk}), {'tipo': materia_anexada.tipo.pk, 'numero': materia_anexada.numero, @@ -125,7 +125,7 @@ def test_autoria_submit(client): autor = mommy.make(Autor, tipo=tipo_autor, nome='Autor Teste') # Testa POST - response = client.post(reverse('autoria', + response = client.post(reverse('materia:autoria', kwargs={'pk': materia_principal.pk}), {'autor': autor.pk, 'primeiro_autor': True, @@ -155,7 +155,7 @@ def test_despacho_inicial_submit(client): data_criacao='2016-03-18') # Testa POST - response = client.post(reverse('despacho_inicial', + response = client.post(reverse('materia:despacho_inicial', kwargs={'pk': materia_principal.pk}), {'comissao': comissao.pk, 'salvar': 'salvar'}, @@ -177,7 +177,7 @@ def test_numeracao_submit(client): sigla='T2', descricao='Teste_2') # Testa POST - response = client.post(reverse('numeracao', + response = client.post(reverse('materia:numeracao', kwargs={'pk': materia_principal.pk}), {'tipo_materia': tipo.pk, 'numero_materia': '341', @@ -209,7 +209,7 @@ def test_documento_acessorio_submit(client): descricao='Teste') # Testa POST - response = client.post(reverse('documento_acessorio', + response = client.post(reverse('materia:documento_acessorio', kwargs={'pk': materia_principal.pk}), {'tipo': tipo.pk, 'nome': 'teste_nome', @@ -234,7 +234,7 @@ def test_legislacao_citada_submit(client): norma = make_norma() # Testa POST - response = client.post(reverse('legislacao_citada', + response = client.post(reverse('materia:legislacao_citada', kwargs={'pk': materia_principal.pk}), {'tipo': norma.tipo.pk, 'numero': norma.numero, @@ -260,7 +260,7 @@ def test_tramitacao_submit(client): descricao='Status_Teste') # Testa POST response = client.post( - reverse('tramitacao_materia', + reverse('materia:tramitacao_materia', kwargs={'pk': materia_principal.pk}), {'unidade_tramitacao_local': make_unidade_tramitacao( 'Unidade Local').pk, @@ -287,7 +287,7 @@ def test_tramitacao_submit(client): @pytest.mark.django_db(transaction=False) def test_form_errors_anexada(client): materia_principal = make_materia_principal() - response = client.post(reverse('materia_anexada', + response = client.post(reverse('materia:materia_anexada', kwargs={'pk': materia_principal.pk}), {'salvar': 'salvar'}, follow=True) @@ -306,7 +306,7 @@ def test_form_errors_anexada(client): def test_form_errors_autoria(client): materia_principal = make_materia_principal() - response = client.post(reverse('autoria', + response = client.post(reverse('materia:autoria', kwargs={'pk': materia_principal.pk}), {'materia_id': materia_principal.pk, 'partido': '', @@ -324,7 +324,7 @@ def test_form_errors_autoria(client): def test_form_errors_despacho_inicial(client): materia_principal = make_materia_principal() - response = client.post(reverse('despacho_inicial', + response = client.post(reverse('materia:despacho_inicial', kwargs={'pk': materia_principal.pk}), {'salvar': 'salvar'}, follow=True) @@ -337,7 +337,7 @@ def test_form_errors_despacho_inicial(client): def test_form_errors_documento_acessorio(client): materia_principal = make_materia_principal() - response = client.post(reverse('documento_acessorio', + response = client.post(reverse('materia:documento_acessorio', kwargs={'pk': materia_principal.pk}), {'salvar': 'salvar'}, follow=True) @@ -352,7 +352,7 @@ def test_form_errors_documento_acessorio(client): def test_form_errors_legislacao_citada(client): materia_principal = make_materia_principal() - response = client.post(reverse('legislacao_citada', + response = client.post(reverse('materia:legislacao_citada', kwargs={'pk': materia_principal.pk}), {'salvar': 'salvar'}, follow=True) @@ -369,7 +369,7 @@ def test_form_errors_legislacao_citada(client): def test_form_errors_numeracao(client): materia_principal = make_materia_principal() - response = client.post(reverse('numeracao', + response = client.post(reverse('materia:numeracao', kwargs={'pk': materia_principal.pk}), {'salvar': 'salvar'}, follow=True) @@ -388,7 +388,7 @@ def test_form_errors_numeracao(client): def test_form_errors_tramitacao(client): materia_principal = make_materia_principal() - response = client.post(reverse('tramitacao_materia', + response = client.post(reverse('materia:tramitacao_materia', kwargs={'pk': materia_principal.pk}), {'salvar': 'salvar'}, follow=True) @@ -409,7 +409,7 @@ def test_form_errors_tramitacao(client): def test_form_errors_relatoria(client): materia_principal = make_materia_principal() - response = client.post(reverse('relatoria', + response = client.post(reverse('materia:relatoria', kwargs={'pk': materia_principal.pk}), {'salvar': 'salvar'}, follow=True) diff --git a/materia/test_materia_urls.py b/materia/test_materia_urls.py index caad5f965..9814612a7 100644 --- a/materia/test_materia_urls.py +++ b/materia/test_materia_urls.py @@ -1,5 +1,5 @@ -from django.core.urlresolvers import reverse import pytest +from django.core.urlresolvers import reverse @pytest.mark.parametrize("test_input,kwargs,expected", [ diff --git a/materia/views.py b/materia/views.py index 35957c0c2..fd417ada7 100644 --- a/materia/views.py +++ b/materia/views.py @@ -223,7 +223,7 @@ class MateriaAnexadaEditView(FormView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('materia_anexada', kwargs={'pk': pk}) + return reverse('materia:materia_anexada', kwargs={'pk': pk}) class DespachoInicialView(CreateView): @@ -294,7 +294,7 @@ class DespachoInicialEditView(CreateView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('despacho_inicial', kwargs={'pk': pk}) + return reverse('materia:despacho_inicial', kwargs={'pk': pk}) class LegislacaoCitadaView(FormView): @@ -355,7 +355,7 @@ class LegislacaoCitadaView(FormView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('legislacao_citada', kwargs={'pk': pk}) + return reverse('materia:legislacao_citada', kwargs={'pk': pk}) class LegislacaoCitadaEditView(FormView): @@ -364,7 +364,7 @@ class LegislacaoCitadaEditView(FormView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('legislacao_citada', kwargs={'pk': pk}) + return reverse('materia:legislacao_citada', kwargs={'pk': pk}) def get(self, request, *args, **kwargs): materia = MateriaLegislativa.objects.get(id=kwargs['pk']) @@ -494,7 +494,7 @@ class NumeracaoEditView(CreateView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('numeracao', kwargs={'pk': pk}) + return reverse('materia:numeracao', kwargs={'pk': pk}) class DocumentoAcessorioView(CreateView): @@ -601,7 +601,7 @@ class DocumentoAcessorioEditView(CreateView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('documento_acessorio', kwargs={'pk': pk}) + return reverse('materia:documento_acessorio', kwargs={'pk': pk}) class RelatoriaEditView(FormView): @@ -610,7 +610,7 @@ class RelatoriaEditView(FormView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('relatoria', kwargs={'pk': pk}) + return reverse('materia:relatoria', kwargs={'pk': pk}) def get(self, request, *args, **kwargs): form = RelatoriaForm() @@ -667,7 +667,7 @@ class RelatoriaView(FormView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('relatoria', kwargs={'pk': pk}) + return reverse('materia:relatoria', kwargs={'pk': pk}) def post(self, request, *args, **kwargs): form = RelatoriaForm(request.POST) @@ -807,8 +807,9 @@ def criar_email_confirmacao(request, casa_legislativa, materia, hash_txt=''): casa_legislativa.uf) base_url = get_base_url(request) - materia_url = reverse('acompanhar_materia', kwargs={'pk': materia.id}) - confirmacao_url = reverse('acompanhar_confirmar', + materia_url = reverse('materia:acompanhar_materia', + kwargs={'pk': materia.id}) + confirmacao_url = reverse('materia:acompanhar_confirmar', kwargs={'pk': materia.id}) autores = [] @@ -843,8 +844,10 @@ def criar_email_tramitacao(request, casa_legislativa, materia, hash_txt=''): casa_legislativa.uf) base_url = get_base_url(request) - url_materia = reverse('acompanhar_materia', kwargs={'pk': materia.id}) - url_excluir = reverse('acompanhar_excluir', kwargs={'pk': materia.id}) + url_materia = reverse('materia:acompanhar_materia', + kwargs={'pk': materia.id}) + url_excluir = reverse('materia:acompanhar_excluir', + kwargs={'pk': materia.id}) autores = [] for autoria in materia.autoria_set.all(): @@ -1027,7 +1030,7 @@ class TramitacaoView(CreateView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('tramitacao_materia', kwargs={'pk': pk}) + return reverse('materia:tramitacao_materia', kwargs={'pk': pk}) class TramitacaoEditView(CreateView): @@ -1082,7 +1085,7 @@ class TramitacaoEditView(CreateView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('tramitacao_materia', kwargs={'pk': pk}) + return reverse('materia:tramitacao_materia', kwargs={'pk': pk}) class AutoriaView(CreateView): @@ -1124,7 +1127,7 @@ class AutoriaView(CreateView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('autoria', kwargs={'pk': pk}) + return reverse('materia:autoria', kwargs={'pk': pk}) class AutoriaEditView(CreateView): @@ -1163,7 +1166,7 @@ class AutoriaEditView(CreateView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('autoria', kwargs={'pk': pk}) + return reverse('materia:autoria', kwargs={'pk': pk}) class ProposicaoListView(ListView): diff --git a/parlamentares/test_parlamentares.py b/parlamentares/test_parlamentares.py index e8ea6c3bc..4668adda4 100644 --- a/parlamentares/test_parlamentares.py +++ b/parlamentares/test_parlamentares.py @@ -10,10 +10,12 @@ from .models import (Dependente, Filiacao, Legislatura, Mandato, Parlamentar, def test_cadastro_parlamentar(client): mommy.make(Legislatura, pk=5) - response = client.get(reverse('parlamentares_cadastro', kwargs={'pk': 5})) + response = client.get(reverse('parlamentares:parlamentares_cadastro', + kwargs={'pk': 5})) assert response.status_code == 200 - response = client.post(reverse('parlamentares_cadastro', kwargs={'pk': 5}), + response = client.post(reverse('parlamentares:parlamentares_cadastro', + kwargs={'pk': 5}), {'nome_completo': 'Teresa Barbosa', 'nome_parlamentar': 'Terezinha', 'sexo': 'F', @@ -30,7 +32,7 @@ def test_filiacao_submit(client): mommy.make(Parlamentar, pk=14) mommy.make(Partido, pk=32) - client.post(reverse('parlamentares_filiacao', + client.post(reverse('parlamentares:parlamentares_filiacao', kwargs={'pk': 14}), {'partido': 32, 'data': '2016-03-22', @@ -47,7 +49,7 @@ def test_dependente_submit(client): mommy.make(Partido, pk=32) mommy.make(TipoDependente, pk=3) - client.post(reverse('parlamentares_dependentes', + client.post(reverse('parlamentares:parlamentares_dependentes', kwargs={'pk': 14}), {'nome': 'Eduardo', 'tipo': 3, @@ -63,7 +65,7 @@ def test_dependente_submit(client): @pytest.mark.django_db(transaction=False) def test_form_errors_dependente(client): mommy.make(Parlamentar, pk=14) - response = client.post(reverse('parlamentares_dependentes', + response = client.post(reverse('parlamentares:parlamentares_dependentes', kwargs={'pk': 14}), {'salvar': 'salvar'}, follow=True) @@ -80,7 +82,7 @@ def test_form_errors_dependente(client): def test_form_errors_filiacao(client): mommy.make(Parlamentar, pk=14) - response = client.post(reverse('parlamentares_filiacao', + response = client.post(reverse('parlamentares:parlamentares_filiacao', kwargs={'pk': 14}), {'partido': '', 'salvar': 'salvar'}, @@ -97,7 +99,7 @@ def test_mandato_submit(client): mommy.make(Parlamentar, pk=14) mommy.make(Legislatura, pk=5) - client.post(reverse('parlamentares_mandato', + client.post(reverse('parlamentares:parlamentares_mandato', kwargs={'pk': 14}), {'legislatura': 5, 'data_fim_mandato': '2016-01-01', @@ -113,7 +115,7 @@ def test_mandato_submit(client): @pytest.mark.django_db(transaction=False) def test_form_errors_mandato(client): mommy.make(Parlamentar, pk=14) - response = client.post(reverse('parlamentares_mandato', + response = client.post(reverse('parlamentares:parlamentares_mandato', kwargs={'pk': 14}), {'legislatura': '', 'salvar': 'salvar'}, @@ -131,7 +133,7 @@ def test_form_errors_mandato(client): def test_incluir_parlamentar_errors(client): mommy.make(Legislatura, pk=5) - response = client.post(reverse('parlamentares_cadastro', + response = client.post(reverse('parlamentares:parlamentares_cadastro', kwargs={'pk': 5}), {'salvar': 'salvar'}, follow=True) diff --git a/parlamentares/views.py b/parlamentares/views.py index 4bf61e41d..b32bef99a 100644 --- a/parlamentares/views.py +++ b/parlamentares/views.py @@ -519,7 +519,8 @@ class MandatoEditView(UpdateView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('parlamentares_mandato', kwargs={'pk': pk}) + return reverse('parlamentares:parlamentares_mandato', + kwargs={'pk': pk}) def get_context_data(self, **kwargs): context = super(MandatoEditView, self).get_context_data(**kwargs)