diff --git a/sapl/compilacao/models.py b/sapl/compilacao/models.py index 2b0e63473..fa0581b70 100644 --- a/sapl/compilacao/models.py +++ b/sapl/compilacao/models.py @@ -11,6 +11,7 @@ from django.db.models.deletion import PROTECT from django.http.response import Http404 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 _ from sapl.compilacao.utils import (get_integrations_view_names, int_to_letter, @@ -843,7 +844,7 @@ class Publicacao(TimestampedMixin): @reversion.register() class Dispositivo(BaseModel, TimestampedMixin): - TEXTO_PADRAO_DISPOSITIVO_REVOGADO = _('(Revogado)') + TEXTO_PADRAO_DISPOSITIVO_REVOGADO = force_text(_('(Revogado)')) INTERVALO_ORDEM = 1000 ordem = models.PositiveIntegerField( default=0, diff --git a/sapl/parlamentares/forms.py b/sapl/parlamentares/forms.py index 8c5ce16b5..6f29e0bf8 100644 --- a/sapl/parlamentares/forms.py +++ b/sapl/parlamentares/forms.py @@ -68,8 +68,8 @@ class MandatoForm(ModelForm): data = self.cleaned_data existe_mandato = Mandato.objects.filter( - parlamentar=data['parlamentar'], - legislatura=data['legislatura']).exists() + parlamentar=data['parlamentar'], + legislatura=data['legislatura']).exists() if existe_mandato: raise ValidationError(_('Mandato nesta legislatura já existe.')) diff --git a/sapl/parlamentares/tests/test_parlamentares.py b/sapl/parlamentares/tests/test_parlamentares.py index f0de9cd8a..b2a738851 100644 --- a/sapl/parlamentares/tests/test_parlamentares.py +++ b/sapl/parlamentares/tests/test_parlamentares.py @@ -2,12 +2,11 @@ import pytest from django.core.urlresolvers import reverse from model_mommy import mommy +from sapl.parlamentares.forms import MandatoForm from sapl.parlamentares.models import (Dependente, Filiacao, Legislatura, Mandato, Parlamentar, Partido, TipoDependente) -from sapl.parlamentares.forms import MandatoForm - @pytest.mark.django_db(transaction=False) def test_cadastro_parlamentar(admin_client): @@ -152,9 +151,10 @@ def test_form_errors_mandato(admin_client): assert (response.context_data['form'].errors['data_expedicao_diploma'] == ['Este campo é obrigatório.']) + def test_mandato_form_invalido(): - form = MandatoForm(data = {}) + form = MandatoForm(data={}) assert not form.is_valid() @@ -163,6 +163,7 @@ def test_mandato_form_invalido(): assert errors['parlamentar'] == ['Este campo é obrigatório.'] assert errors['data_expedicao_diploma'] == ['Este campo é obrigatório.'] + @pytest.mark.django_db(transaction=False) def test_mandato_form_duplicado(): parlamentar = mommy.make(Parlamentar, pk=1) @@ -173,10 +174,10 @@ def test_mandato_form_duplicado(): data_expedicao_diploma='2017-07-25') form = MandatoForm(data={ - 'parlamentar': str(parlamentar.pk), - 'legislatura': str(legislatura.pk), - 'data_expedicao_diploma': '01/07/2015' - }) + 'parlamentar': str(parlamentar.pk), + 'legislatura': str(legislatura.pk), + 'data_expedicao_diploma': '01/07/2015' + }) assert not form.is_valid() diff --git a/sapl/sessao/forms.py b/sapl/sessao/forms.py index 056fee35e..9d4a7c0d3 100644 --- a/sapl/sessao/forms.py +++ b/sapl/sessao/forms.py @@ -81,9 +81,9 @@ class SessaoPlenariaForm(ModelForm): sessao_legislativa=sl, legislatura=leg, tipo=tipo).\ - values_list('id', flat=True) + values_list('id', flat=True) - qtd_sessoes = len(sessoes) + qtd_sessoes = len(sessoes) if qtd_sessoes > 0: if instance.pk: # update diff --git a/sapl/sessao/tests/test_sessao.py b/sapl/sessao/tests/test_sessao.py index b42ab54f1..d1a1ab8e6 100644 --- a/sapl/sessao/tests/test_sessao.py +++ b/sapl/sessao/tests/test_sessao.py @@ -1,12 +1,11 @@ import pytest +from django.utils.translation import ugettext_lazy as _ +from model_mommy import mommy from sapl.parlamentares.models import Legislatura, SessaoLegislativa from sapl.sessao import forms from sapl.sessao.models import SessaoPlenaria, TipoSessaoPlenaria -from django.utils.translation import ugettext_lazy as _ -from model_mommy import mommy - def test_valida_campos_obrigatorios_sessao_plenaria_form(): form = forms.SessaoPlenariaForm(data={}) @@ -24,6 +23,7 @@ def test_valida_campos_obrigatorios_sessao_plenaria_form(): assert len(errors) == 6 + @pytest.mark.django_db(transaction=False) def test_sessao_plenaria_form_valido(): legislatura = mommy.make(Legislatura) @@ -40,6 +40,7 @@ def test_sessao_plenaria_form_valido(): assert form.is_valid() + @pytest.mark.django_db(transaction=False) def test_numero_duplicado_sessao_plenaria_form(): legislatura = mommy.make(Legislatura) @@ -49,7 +50,7 @@ def test_numero_duplicado_sessao_plenaria_form(): legislatura=legislatura, sessao_legislativa=sessao, tipo=tipo, - numero = 1) + numero=1) form = forms.SessaoPlenariaForm(data={'legislatura': str(legislatura.pk), 'numero': '1',