diff --git a/sapl/base/tests/teststub_urls.py b/sapl/base/tests/teststub_urls.py index 33bf55d85..fbdcbb443 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/models.py b/sapl/compilacao/models.py index 1426c2908..2d3d80116 100644 --- a/sapl/compilacao/models.py +++ b/sapl/compilacao/models.py @@ -1,5 +1,6 @@ from datetime import datetime +import reversion from django.contrib import messages from django.contrib.contenttypes.fields import GenericForeignKey from django.contrib.contenttypes.models import ContentType @@ -12,7 +13,6 @@ from django.template import defaultfilters from django.utils.decorators import classonlymethod from django.utils.encoding import force_text from django.utils.translation import ugettext_lazy as _ -import reversion from sapl.compilacao.utils import (get_integrations_view_names, int_to_letter, int_to_roman) diff --git a/sapl/compilacao/tests/test_tipo_texto_articulado_form.py b/sapl/compilacao/tests/test_tipo_texto_articulado_form.py index bb35ef688..484833c66 100644 --- a/sapl/compilacao/tests/test_tipo_texto_articulado_form.py +++ b/sapl/compilacao/tests/test_tipo_texto_articulado_form.py @@ -1,6 +1,6 @@ +import pytest from django.utils.translation import ugettext_lazy as _ from model_mommy import mommy -import pytest from sapl.compilacao import forms from sapl.compilacao.models import PerfilEstruturalTextoArticulado diff --git a/sapl/compilacao/views.py b/sapl/compilacao/views.py index ba533072a..4dfad8e99 100644 --- a/sapl/compilacao/views.py +++ b/sapl/compilacao/views.py @@ -1,7 +1,7 @@ -from collections import OrderedDict -from datetime import timedelta import logging import sys +from collections import OrderedDict +from datetime import timedelta from braces.views import FormMessagesMixin from django import forms @@ -19,8 +19,8 @@ from django.http.response import (HttpResponse, HttpResponseRedirect, from django.shortcuts import get_object_or_404, redirect from django.utils.dateparse import parse_date from django.utils.encoding import force_text -from django.utils.translation import string_concat from django.utils.translation import ugettext_lazy as _ +from django.utils.translation import string_concat from django.views.generic.base import TemplateView from django.views.generic.detail import DetailView from django.views.generic.edit import (CreateView, DeleteView, FormView, @@ -50,7 +50,6 @@ from sapl.compilacao.utils import (DISPOSITIVO_SELECT_RELATED, from sapl.crud.base import Crud, CrudListView, make_pagination from sapl.settings import BASE_DIR - TipoNotaCrud = Crud.build(TipoNota, 'tipo_nota') TipoVideCrud = Crud.build(TipoVide, 'tipo_vide') TipoPublicacaoCrud = Crud.build(TipoPublicacao, 'tipo_publicacao') diff --git a/sapl/norma/forms.py b/sapl/norma/forms.py index ad410c2fb..5149ac926 100644 --- a/sapl/norma/forms.py +++ b/sapl/norma/forms.py @@ -133,9 +133,9 @@ class NormaJuridicaForm(ModelForm): ano=cleaned_data['ano_materia']) except ObjectDoesNotExist: raise forms.ValidationError( - _("Matéria %s/%s é inexistente." % ( - self.cleaned_data['numero_materia'], - self.cleaned_data['ano_materia']))) + _("Matéria %s/%s é inexistente." % ( + self.cleaned_data['numero_materia'], + self.cleaned_data['ano_materia']))) else: cleaned_data['materia'] = materia diff --git a/sapl/norma/tests/test_norma.py b/sapl/norma/tests/test_norma.py index 33baa9ae5..f816c035b 100644 --- a/sapl/norma/tests/test_norma.py +++ b/sapl/norma/tests/test_norma.py @@ -4,9 +4,9 @@ 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.norma.models import (NormaJuridica, NormaRelacionada, - TipoNormaJuridica, TipoVinculoNormaJuridica) +from sapl.norma.forms import NormaJuridicaForm, NormaRelacionadaForm +from sapl.norma.models import (NormaJuridica, TipoNormaJuridica, + TipoVinculoNormaJuridica) @pytest.mark.django_db(transaction=False) @@ -97,7 +97,6 @@ def test_norma_juridica_materia_inexistente(): @pytest.mark.django_db(transaction=False) def test_norma_juridica_materia_existente(): - tipo = mommy.make(TipoNormaJuridica) tipo_materia = mommy.make(TipoMateriaLegislativa) materia = mommy.make(MateriaLegislativa, @@ -117,6 +116,7 @@ def test_norma_juridica_materia_existente(): }) assert form.is_valid() + @pytest.mark.django_db(transaction=False) def test_norma_relacionada_form_campos_obrigatorios(): form = NormaRelacionadaForm(data={}) @@ -126,21 +126,22 @@ def test_norma_relacionada_form_campos_obrigatorios(): errors = form.errors assert errors['tipo'] == [_('Este campo é obrigatório.')] - assert errors['numero'] ==[_('Este campo é obrigatório.')] + assert errors['numero'] == [_('Este campo é obrigatório.')] assert errors['ano'] == [_('Este campo é obrigatório.')] assert errors['tipo_vinculo'] == [_('Este campo é obrigatório.')] assert len(errors) == 4 + @pytest.mark.django_db(transaction=False) def test_norma_relacionada_form_valido(): tipo = mommy.make(TipoNormaJuridica) tipo_vinculo = mommy.make(TipoVinculoNormaJuridica) - form = forms.NormaRelacionadaForm(data={'tipo': str(tipo.pk), - 'numero': '1', - 'ano': '2017', - 'tipo_vinculo': str(tipo_vinculo.pk) - }) + form = NormaRelacionadaForm(data={'tipo': str(tipo.pk), + 'numero': '1', + 'ano': '2017', + 'tipo_vinculo': str(tipo_vinculo.pk) + }) assert form.is_valid() diff --git a/sapl/parlamentares/forms.py b/sapl/parlamentares/forms.py index e08eb5451..9972b9574 100644 --- a/sapl/parlamentares/forms.py +++ b/sapl/parlamentares/forms.py @@ -72,14 +72,14 @@ class MandatoForm(ModelForm): data_inicio_mandato = data['data_inicio_mandato'] if data_inicio_mandato: if (data_inicio_mandato < legislatura.data_inicio or - data_inicio_mandato > legislatura.data_fim): + data_inicio_mandato > legislatura.data_fim): 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): + data_fim_mandato > legislatura.data_fim): raise ValidationError(_("Data fim mandato fora do intervalo de " "legislatura informada")) diff --git a/sapl/parlamentares/tests/test_parlamentares.py b/sapl/parlamentares/tests/test_parlamentares.py index 03e3f08e9..2b5223b0b 100644 --- a/sapl/parlamentares/tests/test_parlamentares.py +++ b/sapl/parlamentares/tests/test_parlamentares.py @@ -3,7 +3,7 @@ from django.core.urlresolvers import reverse from django.utils.translation import ugettext_lazy as _ from model_mommy import mommy -from sapl.parlamentares.forms import (LegislaturaForm, MandatoForm) +from sapl.parlamentares.forms import LegislaturaForm, MandatoForm from sapl.parlamentares.models import (Dependente, Filiacao, Legislatura, Mandato, Parlamentar, Partido, TipoDependente) @@ -181,7 +181,9 @@ def test_mandato_form_duplicado(): assert not form.is_valid() - assert form.errors['__all__'] == [_('Mandato nesta legislatura já existe.')] + assert form.errors['__all__'] == [ + _('Mandato nesta legislatura já existe.')] + @pytest.mark.django_db(transaction=False) def test_mandato_form_datas_invalidas(): @@ -230,13 +232,14 @@ def test_legislatura_form_invalido(): assert len(errors) == 4 + def test_legislatura_form_datas_invalidas(): legislatura_form = LegislaturaForm(data={'numero': '1', 'data_inicio': '2017-02-01', 'data_fim': '2021-12-31', 'data_eleicao': '2017-02-01' - }) + }) assert not legislatura_form.is_valid() @@ -248,7 +251,7 @@ def test_legislatura_form_datas_invalidas(): 'data_inicio': '2017-02-01', 'data_fim': '2017-01-01', 'data_eleicao': '2016-11-01' - }) + }) assert not legislatura_form.is_valid() diff --git a/sapl/protocoloadm/tests/test_protocoloadm.py b/sapl/protocoloadm/tests/test_protocoloadm.py index fd3e73df7..232c922f5 100644 --- a/sapl/protocoloadm/tests/test_protocoloadm.py +++ b/sapl/protocoloadm/tests/test_protocoloadm.py @@ -6,11 +6,11 @@ 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 MateriaLegislativa, UnidadeTramitacao from sapl.protocoloadm.forms import (AnularProcoloAdmForm, DocumentoAdministrativoForm, ProtocoloDocumentForm, - ProtocoloMateriaForm,) + ProtocoloMateriaForm) from sapl.protocoloadm.models import (DocumentoAdministrativo, Protocolo, StatusTramitacaoAdministrativo, TipoDocumentoAdministrativo, @@ -300,6 +300,7 @@ def test_anular_protocolo_form_anula_protocolo_anulado(): assert form.errors['__all__'] == [_( 'Protocolo 1/2017 já encontra-se anulado')] + @pytest.mark.django_db(transaction=False) def test_anular_protocolo_form_anula_protocolo_com_doc_vinculado(): tipo_materia = mommy.make(TipoMateriaLegislativa) @@ -323,14 +324,14 @@ def test_anular_protocolo_form_anula_protocolo_com_doc_vinculado(): assert form.errors['__all__'] == \ [_("Protocolo 1/2017 não pode ser removido pois existem " - "documentos vinculados a ele.")] + "documentos vinculados a ele.")] tipo_documento = mommy.make(TipoDocumentoAdministrativo) protocolo_documento = mommy.make(Protocolo, - numero=2, - ano=2017, - tipo_documento=tipo_documento, - anulado=False) + numero=2, + ano=2017, + tipo_documento=tipo_documento, + anulado=False) documento_administrativo = mommy.make(DocumentoAdministrativo, protocolo=protocolo_documento) @@ -344,7 +345,7 @@ def test_anular_protocolo_form_anula_protocolo_com_doc_vinculado(): assert form.errors['__all__'] == \ [_("Protocolo 2/2017 não pode ser removido pois existem " - "documentos vinculados a ele.")] + "documentos vinculados a ele.")] def test_documento_administrativo_invalido(): @@ -397,6 +398,7 @@ def test_protocolo_documento_form_invalido(): assert len(errors) == 5 + def test_protocolo_materia_invalido(): form = ProtocoloMateriaForm(data={}) diff --git a/sapl/sessao/tests/test_sessao.py b/sapl/sessao/tests/test_sessao.py index 8fd4e9ea4..9ae0b9138 100644 --- a/sapl/sessao/tests/test_sessao.py +++ b/sapl/sessao/tests/test_sessao.py @@ -2,10 +2,9 @@ import pytest from django.utils.translation import ugettext_lazy as _ from model_mommy import mommy -from sapl.parlamentares.models import (Legislatura, Partido, - SessaoLegislativa) +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 Bancada, SessaoPlenaria, TipoSessaoPlenaria def test_valida_campos_obrigatorios_sessao_plenaria_form(): @@ -16,7 +15,7 @@ def test_valida_campos_obrigatorios_sessao_plenaria_form(): errors = form.errors assert errors['legislatura'] == [_('Este campo é obrigatório.')] - assert errors['sessao_legislativa'] ==[_('Este campo é obrigatório.')] + assert errors['sessao_legislativa'] == [_('Este campo é obrigatório.')] assert errors['tipo'] == [_('Este campo é obrigatório.')] assert errors['numero'] == [_('Este campo é obrigatório.')] assert errors['data_inicio'] == [_('Este campo é obrigatório.')] @@ -68,6 +67,7 @@ def test_numero_duplicado_sessao_plenaria_form(): "e Tipo informados. Favor escolher um " "número distinto."] + @pytest.mark.django_db(transaction=False) def test_valida_campos_obrigatorios_bancada_form(): form = forms.BancadaForm(data={}) @@ -81,35 +81,35 @@ def test_valida_campos_obrigatorios_bancada_form(): assert len(errors) == 2 + @pytest.mark.django_db(transaction=False) def test_bancada_form_valido(): legislatura = mommy.make(Legislatura) partido = mommy.make(Partido) - form = forms.BancadaForm(data={'legislatura': str(legislatura.pk), - 'nome': 'Nome da Bancada', - 'partido': str(partido.pk), - 'data_criacao': '10/11/2017', - 'data_extincao': '10/12/2017', - 'descricao': 'teste' - }) + 'nome': 'Nome da Bancada', + 'partido': str(partido.pk), + 'data_criacao': '10/11/2017', + 'data_extincao': '10/12/2017', + 'descricao': 'teste' + }) assert form.is_valid() + @pytest.mark.django_db(transaction=False) def test_bancada_form_datas_invalidas(): legislatura = mommy.make(Legislatura) partido = mommy.make(Partido) - form = forms.BancadaForm(data={'legislatura': str(legislatura.pk), - 'nome': 'Nome da Bancada', - 'partido': str(partido.pk), - 'data_criacao': '2016-11-01', - 'data_extincao': '2016-10-01', - 'descricao': 'teste' - }) + 'nome': 'Nome da Bancada', + 'partido': str(partido.pk), + 'data_criacao': '2016-11-01', + 'data_extincao': '2016-10-01', + 'descricao': 'teste' + }) assert not form.is_valid() assert form.errors['__all__'] == [_('Data de extinção não pode ser menor ' 'que a de criação')] diff --git a/sapl/test_urls.py b/sapl/test_urls.py index 9daa58fab..8b5207276 100644 --- a/sapl/test_urls.py +++ b/sapl/test_urls.py @@ -174,24 +174,24 @@ apps_url_patterns_prefixs_and_users = { ]}, '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', + '/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': [