Fix NormaRelacionadaForm e alguns erros relacionados ao check_qa.sh

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(
+                                                   TemplateView.as_view(
-                                               template_name='index.html'),
+                                                       template_name='index.html'),
-                                           name='zzzz'))
+                                                   name='zzzz'))

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)

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

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

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." % (
+                    _("Matéria %s/%s é inexistente." % (
-                    self.cleaned_data['numero_materia'],
+                        self.cleaned_data['numero_materia'],
-                    self.cleaned_data['ano_materia'])))
+                        self.cleaned_data['ano_materia'])))
             else:
                 cleaned_data['materia'] = materia
 

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.forms import NormaJuridicaForm, NormaRelacionadaForm
-from sapl.norma.models import (NormaJuridica, NormaRelacionada,
+from sapl.norma.models import (NormaJuridica, TipoNormaJuridica,
-                                TipoNormaJuridica, TipoVinculoNormaJuridica)
+                               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),
+    form = NormaRelacionadaForm(data={'tipo': str(tipo.pk),
-                                          'numero': '1',
+                                      'numero': '1',
-                                          'ano': '2017',
+                                      'ano': '2017',
-                                          'tipo_vinculo': str(tipo_vinculo.pk)
+                                      'tipo_vinculo': str(tipo_vinculo.pk)
-                                          })
+                                      })
 
     assert form.is_valid()

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

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

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,
+                                     numero=2,
-                                   ano=2017,
+                                     ano=2017,
-                                   tipo_documento=tipo_documento,
+                                     tipo_documento=tipo_documento,
-                                   anulado=False)
+                                     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={})

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,
+from sapl.parlamentares.models import Legislatura, Partido, SessaoLegislativa
-                                        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',
+                                   'nome': 'Nome da Bancada',
-                                          'partido': str(partido.pk),
+                                   'partido': str(partido.pk),
-                                          'data_criacao': '10/11/2017',
+                                   'data_criacao': '10/11/2017',
-                                          'data_extincao': '10/12/2017',
+                                   'data_extincao': '10/12/2017',
-                                          'descricao': 'teste'
+                                   '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',
+                                   'nome': 'Nome da Bancada',
-                                          'partido': str(partido.pk),
+                                   'partido': str(partido.pk),
-                                          'data_criacao': '2016-11-01',
+                                   'data_criacao': '2016-11-01',
-                                          'data_extincao': '2016-10-01',
+                                   'data_extincao': '2016-10-01',
-                                          'descricao': 'teste'
+                                   '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')]

sapl/test_urls.py

@@ -174,24 +174,24 @@ apps_url_patterns_prefixs_and_users = {
         ]},
     'redireciona_urls': {
         'prefixs': [
-           '/default_index_html',
+            '/default_index_html',
-           '/consultas/parlamentar/parlamentar_',
+            '/consultas/parlamentar/parlamentar_',
-           '/consultas/comissao/comissao_',
+            '/consultas/comissao/comissao_',
-           '/consultas/pauta_sessao/pauta_sessao_',
+            '/consultas/pauta_sessao/pauta_sessao_',
-           '/consultas/sessao_plenaria/',
+            '/consultas/sessao_plenaria/',
-           '/relatorios_administrativos/relatorios_administrativos_index_html',
+            '/relatorios_administrativos/relatorios_administrativos_index_html',
-           '/tramitacaoMaterias/tramitacaoMaterias',
+            '/tramitacaoMaterias/tramitacaoMaterias',
-           '/tramitacaoMaterias/materia_mostrar_proc',
+            '/tramitacaoMaterias/materia_mostrar_proc',
-           '/generico/materia_pesquisar_',
+            '/generico/materia_pesquisar_',
-           '/consultas/mesa_diretora/mesa_diretora_index_html',
+            '/consultas/mesa_diretora/mesa_diretora_index_html',
-           '/consultas/mesa_diretora/parlamentar/parlamentar_',
+            '/consultas/mesa_diretora/parlamentar/parlamentar_',
-           '/generico/norma_juridica_pesquisar_',
+            '/generico/norma_juridica_pesquisar_',
-           '/consultas/norma_juridica/norma_juridica_mostrar_proc',
+            '/consultas/norma_juridica/norma_juridica_mostrar_proc',
-           '/historicoTramitacoes/historicoTramitacoes',
+            '/historicoTramitacoes/historicoTramitacoes',
-           '/atasSessao',
+            '/atasSessao',
-           '/presencaSessao',
+            '/presencaSessao',
-           '/resumoPropositurasAutor',
+            '/resumoPropositurasAutor',
-           '/propositurasAnoAutorTipo',
+            '/propositurasAnoAutorTipo',
         ]},
     'lexml': {
         'prefixs': [