From 40eecaa07cdcb1bb83cca605e5bd076120fc8705 Mon Sep 17 00:00:00 2001
From: Mariana Mendes <mare.s.mendes@gmail.com>
Date: Fri, 6 Jul 2018 10:11:00 -0300
Subject: [PATCH] =?UTF-8?q?HOT-FIX=20-=20Adiciona=20o=20teste=20de=20comis?=
 =?UTF-8?q?s=C3=A3o=20(#2061)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Adiciona teste em test_comissões e corrige mensagem no forms

* Corrige erro de admin_client

* Adiciona testes referente a reunião de comissão

* Adiciona mais testes
---
 sapl/comissoes/forms.py                |  3 +-
 sapl/comissoes/tests/test_comissoes.py | 48 +++++++++++++++++++++++++-
 2 files changed, 49 insertions(+), 2 deletions(-)

diff --git a/sapl/comissoes/forms.py b/sapl/comissoes/forms.py
index 05a0a97b9..75b45c976 100644
--- a/sapl/comissoes/forms.py
+++ b/sapl/comissoes/forms.py
@@ -61,7 +61,8 @@ class PeriodoForm(forms.ModelForm):
         data_fim = cleaned_data['data_fim']
 
         if data_fim and data_fim < data_inicio:
-            raise ValidationError('Data início não pode ser superior a data de fim')
+            raise ValidationError('A Data Final não pode ser menor que '
+                                        'a Data Inicial')
         return cleaned_data
 
 
diff --git a/sapl/comissoes/tests/test_comissoes.py b/sapl/comissoes/tests/test_comissoes.py
index fee303192..4d12ba810 100644
--- a/sapl/comissoes/tests/test_comissoes.py
+++ b/sapl/comissoes/tests/test_comissoes.py
@@ -1,9 +1,11 @@
 import pytest
 from django.core.urlresolvers import reverse
+from django.utils.translation import ugettext as _
 from model_mommy import mommy
 
-from sapl.comissoes.models import Comissao, Composicao, Periodo, TipoComissao
+from sapl.comissoes.models import Comissao, Composicao, Periodo, TipoComissao, Reuniao
 from sapl.parlamentares.models import Filiacao, Parlamentar, Partido
+from sapl.comissoes import forms
 
 
 def make_composicao(comissao):
@@ -96,3 +98,47 @@ def test_incluir_comissao_errors(admin_client):
             ['Este campo é obrigatório.'])
     assert (response.context_data['form'].errors['data_criacao'] ==
             ['Este campo é obrigatório.'])
+
+
+@pytest.mark.django_db(transaction=False)
+def test_periodo_invalidas():
+
+    form = forms.PeriodoForm(data={'data_inicio': '10/11/2017',
+                                   'data_fim': '09/11/2017'
+                                  })
+    assert not form.is_valid()
+    assert form.errors['__all__'] == [_('A Data Final não pode ser menor que '
+                                        'a Data Inicial')]
+
+
+@pytest.mark.django_db(transaction=False)
+def test_valida_campos_obrigatorios_periodo_form():
+    form = forms.PeriodoForm(data={})
+
+    assert not form.is_valid()
+
+    errors = form.errors
+
+    assert errors['data_inicio'] == [_('Este campo é obrigatório.')]
+
+    assert len(errors) == 1
+    
+
+@pytest.mark.django_db(transaction=False)
+def test_valida_campos_obrigatorios_reuniao_form():
+    form = forms.ReuniaoForm(data={})
+
+    assert not form.is_valid()
+
+    errors = form.errors
+
+    assert errors['comissao'] == [_('Este campo é obrigatório.')]
+    assert errors['periodo'] == [_('Este campo é obrigatório.')]
+    assert errors['numero'] == [_('Este campo é obrigatório.')]
+    assert errors['nome'] == [_('Este campo é obrigatório.')]
+    assert errors['data'] == [_('Este campo é obrigatório.')]
+    assert errors['hora_inicio'] == [_('Este campo é obrigatório.')]
+    assert errors['hora_fim'] == [_('Este campo é obrigatório.')]
+
+    assert len(errors) == 7
+