Corrige teste em NotaForm

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():

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()

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',

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'],

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={})

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'

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',