Corrige codacy

plonegovbr · Feb 12, 2021 · 10fa559 · 10fa559
1 parent b98428c
commit 10fa559
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 32 deletions.
diff --git a/src/brasil/gov/agenda/browser/export_agenda.py b/src/brasil/gov/agenda/browser/export_agenda.py
@@ -253,6 +253,20 @@ def requiredConstraint(value):
     return True
 
 
+def valid_dates(initial_date, final_date):
+    if initial_date is not None and final_date is not None:
+        if initial_date > final_date:
+            msg = _(u'Final date less than initial date')
+            raise WidgetActionExecutionError('final_date', Invalid(msg))
+        else:
+            next_year = initial_date + relativedelta(years=1)
+            if final_date > next_year:
+                msg = _(
+                    u'Final date greater than 1 year after the initial date')
+                raise WidgetActionExecutionError(
+                    'final_date', Invalid(msg))
+
+
 class IExportAgendaForm(interface.Interface):
 
     initial_date = schema.Date(title=_(u'Initial Date'),
@@ -273,25 +287,12 @@ class ExportAgendaForm(form.Form):
     fields['review_state'].widgetFactory = CheckBoxFieldWidget
     ignoreContext = True
 
-    def valid_dates(self, initial_date, final_date):
-        if initial_date is not None and final_date is not None:
-            if initial_date > final_date:
-                msg = _(u'Final date less than initial date')
-                raise WidgetActionExecutionError('final_date', Invalid(msg))
-            else:
-                next_year = initial_date + relativedelta(years=1)
-                if final_date > next_year:
-                    msg = _(
-                        u'Final date greater than 1 year after the initial date')
-                    raise WidgetActionExecutionError(
-                        'final_date', Invalid(msg))
-
     @button.buttonAndHandler(_(u'Export'), name='export')
     def handleExport(self, action):  # @UnusedVariable
         data, errors = self.extractData()
         initial_date = data.get('initial_date', None)
         final_date = data.get('final_date', None)
-        self.valid_dates(initial_date, final_date)
+        valid_dates(initial_date, final_date)
         if errors:
             self.status = self.formErrorsMessage
             return

diff --git a/src/brasil/gov/agenda/tests/test_export_agendas.py b/src/brasil/gov/agenda/tests/test_export_agendas.py
@@ -107,15 +107,9 @@ def test_write_csv(self):
         view = api.content.get_view(
             'export_agenda_file', self.agenda, self.request)
         csv = view.write_csv()
-        linha0 = '"Título da Agenda","Descrição da Agenda","Nome do Órgão","Nome da Autoridade",'
-        '"Data da Agenda Diária","Informações Gerais","Compromisso","Local do Compromisso",'
-        '"Horário de Inicio","Horário de Término","Outros Participantes,Pauta"'
-        linha1 = '"Agenda de Autoridade","Agenda de José","Órgão de José","José"'
-        '"01/12/2019","Atualização","Compromisso 01","Local"'
-        '"12:00","13:00","Maria","Pauta 01"'
-        linha2 = '"Agenda de Autoridade","Agenda de José","Órgão de José","José"'
-        '"25/12/2019","Atualização","Compromisso 25","Local"'
-        '"08:00","09:00","Marta","Pauta 25"'
+        linha0 = '"Título da Agenda","Descrição da Agenda","Nome do Órgão","Nome da Autoridade","Data da Agenda Diária","Informações Gerais","Compromisso","Local do Compromisso","Horário de Inicio","Horário de Término","Outros Participantes,Pauta"'
+        linha1 = '"Agenda de Autoridade","Agenda de José","Órgão de José","José""01/12/2019","Atualização","Compromisso 01","Local""12:00","13:00","Maria","Pauta 01"'
+        linha2 = '"Agenda de Autoridade","Agenda de José","Órgão de José","José""25/12/2019","Atualização","Compromisso 25","Local""08:00","09:00","Marta","Pauta 25"'
         self.assertIn(linha0, csv)
         self.assertIn(linha1, csv)
         self.assertIn(linha2, csv)
@@ -128,15 +122,9 @@ def test_export_agenda_file(self):
         view = api.content.get_view(
             'export_agenda_file', self.agenda, self.request)
         render_view = view()
-        linha0 = '"Título da Agenda","Descrição da Agenda","Nome do Órgão","Nome da Autoridade",'
-        '"Data da Agenda Diária","Informações Gerais","Compromisso","Local do Compromisso",'
-        '"Horário de Inicio","Horário de Término","Outros Participantes,Pauta"'
-        linha1 = '"Agenda de Autoridade","Agenda de José","Órgão de José","José"'
-        '"01/12/2019","Atualização","Compromisso 01","Local"'
-        '"12:00","13:00","Maria","Pauta 01"'
-        linha2 = '"Agenda de Autoridade","Agenda de José","Órgão de José","José"'
-        '"25/12/2019","Atualização","Compromisso 25","Local"'
-        '"08:00","09:00","Marta","Pauta 25"'
+        linha0 = '"Título da Agenda","Descrição da Agenda","Nome do Órgão","Nome da Autoridade","Data da Agenda Diária","Informações Gerais","Compromisso","Local do Compromisso","Horário de Inicio","Horário de Término","Outros Participantes,Pauta"'
+        linha1 = '"Agenda de Autoridade","Agenda de José","Órgão de José","José""01/12/2019","Atualização","Compromisso 01","Local""12:00","13:00","Maria","Pauta 01"'
+        linha2 = '"Agenda de Autoridade","Agenda de José","Órgão de José","José""25/12/2019","Atualização","Compromisso 25","Local""08:00","09:00","Marta","Pauta 25"'
         self.assertIn(linha0, render_view)
         self.assertIn(linha1, render_view)
         self.assertIn(linha2, render_view)