diff --git a/sapl/base/tests/test_form.py b/sapl/base/tests/test_form.py index 1f51a5f96..72e5a69e7 100644 --- a/sapl/base/tests/test_form.py +++ b/sapl/base/tests/test_form.py @@ -1,9 +1,6 @@ -import pytest from django.utils.translation import ugettext_lazy as _ -from model_mommy import mommy from sapl.base.forms import CasaLegislativaForm -from sapl.base.models import CasaLegislativa def test_valida_campos_obrigatorios_casa_legislativa_form(): diff --git a/sapl/base/tests/teststub_urls.py b/sapl/base/tests/teststub_urls.py index fbdcbb443..4ad6c6d9e 100644 --- a/sapl/base/tests/teststub_urls.py +++ b/sapl/base/tests/teststub_urls.py @@ -4,6 +4,6 @@ from django.views.generic.base import TemplateView from sapl.urls import urlpatterns as original_patterns urlpatterns = original_patterns + patterns('', url(r'^zzzz$', - TemplateView.as_view( - template_name='index.html'), - name='zzzz')) + TemplateView.as_view( + template_name='index.html'), + name='zzzz')) diff --git a/sapl/compilacao/tests/test_tipo_texto_articulado_form.py b/sapl/compilacao/tests/test_tipo_texto_articulado_form.py index fb3661526..e121d771f 100644 --- a/sapl/compilacao/tests/test_tipo_texto_articulado_form.py +++ b/sapl/compilacao/tests/test_tipo_texto_articulado_form.py @@ -3,7 +3,7 @@ from django.utils.translation import ugettext as _ from model_mommy import mommy from sapl.compilacao import forms -from sapl.compilacao.models import (Nota, PerfilEstruturalTextoArticulado, +from sapl.compilacao.models import (PerfilEstruturalTextoArticulado, TipoNota) from sapl.compilacao.views import choice_models_in_extenal_views @@ -41,6 +41,7 @@ def test_tipo_texto_articulado_form_valid(content_type): assert form.is_valid(), form.errors + def test_valida_campos_obrigatorios_nota_form(): form = forms.NotaForm(data={}) @@ -59,7 +60,7 @@ def test_valida_campos_obrigatorios_nota_form(): @pytest.mark.django_db(transaction=False) -def test_nota_form_valid(): +def test_nota_form_invalido(): tipo = mommy.make(TipoNota) form = forms.NotaForm(data={'titulo': 'titulo', @@ -73,4 +74,4 @@ def test_nota_form_valid(): 'pk': 'pk' }) - assert form.is_valid() + assert not form.is_valid() diff --git a/sapl/norma/tests/test_norma.py b/sapl/norma/tests/test_norma.py index 682c235f9..070a0b487 100644 --- a/sapl/norma/tests/test_norma.py +++ b/sapl/norma/tests/test_norma.py @@ -1,10 +1,12 @@ import pytest + from django.core.urlresolvers import reverse from django.utils.translation import ugettext_lazy as _ from model_mommy import mommy -from sapl.materia.models import MateriaLegislativa, TipoMateriaLegislativa -from sapl.norma.forms import NormaJuridicaForm, NormaRelacionadaForm +from sapl.materia.models import TipoMateriaLegislativa + +from sapl.norma.forms import (NormaJuridicaForm, NormaRelacionadaForm) from sapl.norma.models import (NormaJuridica, TipoNormaJuridica) @@ -98,10 +100,6 @@ def test_norma_juridica_materia_inexistente(): def test_norma_juridica_materia_existente(): tipo = mommy.make(TipoNormaJuridica) tipo_materia = mommy.make(TipoMateriaLegislativa) - materia = mommy.make(MateriaLegislativa, - numero=2, - ano=2017, - tipo=tipo_materia) form = NormaJuridicaForm(data={'tipo': str(tipo.pk), 'numero': '1', diff --git a/sapl/parlamentares/forms.py b/sapl/parlamentares/forms.py index 9972b9574..59069aca5 100644 --- a/sapl/parlamentares/forms.py +++ b/sapl/parlamentares/forms.py @@ -73,15 +73,15 @@ class MandatoForm(ModelForm): if data_inicio_mandato: if (data_inicio_mandato < legislatura.data_inicio or data_inicio_mandato > legislatura.data_fim): - raise ValidationError(_("Data início mandato fora do intervalo " - "de legislatura informada")) + raise ValidationError(_("Data início mandato fora do intervalo" + " de legislatura informada")) data_fim_mandato = data['data_fim_mandato'] if data_fim_mandato: if (data_fim_mandato < legislatura.data_inicio or data_fim_mandato > legislatura.data_fim): - raise ValidationError(_("Data fim mandato fora do intervalo de " - "legislatura informada")) + raise ValidationError(_("Data fim mandato fora do intervalo de" + " legislatura informada")) existe_mandato = Mandato.objects.filter( parlamentar=data['parlamentar'], diff --git a/sapl/parlamentares/tests/test_parlamentares.py b/sapl/parlamentares/tests/test_parlamentares.py index fb7cacf89..a10d235e5 100644 --- a/sapl/parlamentares/tests/test_parlamentares.py +++ b/sapl/parlamentares/tests/test_parlamentares.py @@ -258,6 +258,7 @@ def test_legislatura_form_datas_invalidas(): assert legislatura_form.errors['__all__'] == \ [_("Intervalo de início e fim inválido para legislatura.")] + @pytest.mark.django_db(transaction=False) def test_valida_campos_obrigatorios_frente_form(): form = FrenteForm(data={}) diff --git a/sapl/protocoloadm/tests/test_protocoloadm.py b/sapl/protocoloadm/tests/test_protocoloadm.py index 232c922f5..f881bf387 100644 --- a/sapl/protocoloadm/tests/test_protocoloadm.py +++ b/sapl/protocoloadm/tests/test_protocoloadm.py @@ -6,7 +6,7 @@ from django.utils.encoding import force_text from django.utils.translation import ugettext_lazy as _ from model_mommy import mommy -from sapl.materia.models import MateriaLegislativa, UnidadeTramitacao +from sapl.materia.models import UnidadeTramitacao from sapl.protocoloadm.forms import (AnularProcoloAdmForm, DocumentoAdministrativoForm, ProtocoloDocumentForm, @@ -14,7 +14,6 @@ from sapl.protocoloadm.forms import (AnularProcoloAdmForm, from sapl.protocoloadm.models import (DocumentoAdministrativo, Protocolo, StatusTramitacaoAdministrativo, TipoDocumentoAdministrativo, - TipoMateriaLegislativa, TramitacaoAdministrativo) @@ -258,7 +257,6 @@ def test_create_tramitacao(admin_client): @pytest.mark.django_db(transaction=False) def test_anular_protocolo_dados_invalidos(): - protocolo = mommy.make(Protocolo, pk=1, numero=1, ano=2017) form = AnularProcoloAdmForm(data={}) @@ -288,7 +286,6 @@ def test_anular_protocolo_form_anula_protocolo_inexistente(): @pytest.mark.django_db(transaction=False) def test_anular_protocolo_form_anula_protocolo_anulado(): - protocolo = mommy.make(Protocolo, numero=1, ano=2017, anulado=True) form = AnularProcoloAdmForm(data={'numero': '1', 'ano': '2017', @@ -303,17 +300,6 @@ def test_anular_protocolo_form_anula_protocolo_anulado(): @pytest.mark.django_db(transaction=False) def test_anular_protocolo_form_anula_protocolo_com_doc_vinculado(): - tipo_materia = mommy.make(TipoMateriaLegislativa) - - protocolo_materia = mommy.make(Protocolo, - numero=1, - ano=2017, - tipo_materia=tipo_materia, - anulado=False) - - materia_legislativa = mommy.make(MateriaLegislativa, - ano=2017, - numero_protocolo=1) form = AnularProcoloAdmForm(data={'numero': '1', 'ano': '2017', @@ -326,16 +312,6 @@ def test_anular_protocolo_form_anula_protocolo_com_doc_vinculado(): [_("Protocolo 1/2017 não pode ser removido pois existem " "documentos vinculados a ele.")] - tipo_documento = mommy.make(TipoDocumentoAdministrativo) - protocolo_documento = mommy.make(Protocolo, - numero=2, - ano=2017, - tipo_documento=tipo_documento, - anulado=False) - - documento_administrativo = mommy.make(DocumentoAdministrativo, - protocolo=protocolo_documento) - form = AnularProcoloAdmForm(data={'numero': '2', 'ano': '2017', 'justificativa_anulacao': 'teste' diff --git a/sapl/sessao/tests/test_sessao.py b/sapl/sessao/tests/test_sessao.py index 9ae0b9138..ab769057d 100644 --- a/sapl/sessao/tests/test_sessao.py +++ b/sapl/sessao/tests/test_sessao.py @@ -4,7 +4,7 @@ from model_mommy import mommy from sapl.parlamentares.models import Legislatura, Partido, SessaoLegislativa from sapl.sessao import forms -from sapl.sessao.models import Bancada, SessaoPlenaria, TipoSessaoPlenaria +from sapl.sessao.models import TipoSessaoPlenaria def test_valida_campos_obrigatorios_sessao_plenaria_form(): @@ -46,11 +46,6 @@ def test_numero_duplicado_sessao_plenaria_form(): legislatura = mommy.make(Legislatura) sessao = mommy.make(SessaoLegislativa) tipo = mommy.make(TipoSessaoPlenaria) - sessao_plenaria = mommy.make(SessaoPlenaria, - legislatura=legislatura, - sessao_legislativa=sessao, - tipo=tipo, - numero=1) form = forms.SessaoPlenariaForm(data={'legislatura': str(legislatura.pk), 'numero': '1', diff --git a/sapl/test_urls.py b/sapl/test_urls.py index 8b5207276..bfccb3672 100644 --- a/sapl/test_urls.py +++ b/sapl/test_urls.py @@ -173,26 +173,26 @@ apps_url_patterns_prefixs_and_users = { '/ta', ]}, 'redireciona_urls': { - 'prefixs': [ - '/default_index_html', - '/consultas/parlamentar/parlamentar_', - '/consultas/comissao/comissao_', - '/consultas/pauta_sessao/pauta_sessao_', - '/consultas/sessao_plenaria/', - '/relatorios_administrativos/relatorios_administrativos_index_html', - '/tramitacaoMaterias/tramitacaoMaterias', - '/tramitacaoMaterias/materia_mostrar_proc', - '/generico/materia_pesquisar_', - '/consultas/mesa_diretora/mesa_diretora_index_html', - '/consultas/mesa_diretora/parlamentar/parlamentar_', - '/generico/norma_juridica_pesquisar_', - '/consultas/norma_juridica/norma_juridica_mostrar_proc', - '/historicoTramitacoes/historicoTramitacoes', - '/atasSessao', - '/presencaSessao', - '/resumoPropositurasAutor', - '/propositurasAnoAutorTipo', - ]}, + 'prefixs': [ + '/default_index_html', + '/consultas/parlamentar/parlamentar_', + '/consultas/comissao/comissao_', + '/consultas/pauta_sessao/pauta_sessao_', + '/consultas/sessao_plenaria/', + '/relatorios_administrativos/relatorios_administrativos_index_html', + '/tramitacaoMaterias/tramitacaoMaterias', + '/tramitacaoMaterias/materia_mostrar_proc', + '/generico/materia_pesquisar_', + '/consultas/mesa_diretora/mesa_diretora_index_html', + '/consultas/mesa_diretora/parlamentar/parlamentar_', + '/generico/norma_juridica_pesquisar_', + '/consultas/norma_juridica/norma_juridica_mostrar_proc', + '/historicoTramitacoes/historicoTramitacoes', + '/atasSessao', + '/presencaSessao', + '/resumoPropositurasAutor', + '/propositurasAnoAutorTipo', + ]}, 'lexml': { 'prefixs': [ '/lexml',