From 10fa5596c81bedf1a343958b876ed8404e4eac91 Mon Sep 17 00:00:00 2001 From: idgserpro Date: Fri, 12 Feb 2021 16:42:53 -0300 Subject: [PATCH] Corrige codacy --- .../gov/agenda/browser/export_agenda.py | 29 ++++++++++--------- .../gov/agenda/tests/test_export_agendas.py | 24 ++++----------- 2 files changed, 21 insertions(+), 32 deletions(-) diff --git a/src/brasil/gov/agenda/browser/export_agenda.py b/src/brasil/gov/agenda/browser/export_agenda.py index c12491c..de3f4ae 100644 --- 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 index 3e0ed14..a8b57b2 100644 --- 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)