diff --git a/woudc_data_registry/processing.py b/woudc_data_registry/processing.py index 7f15c36..fafc87a 100644 --- a/woudc_data_registry/processing.py +++ b/woudc_data_registry/processing.py @@ -939,7 +939,8 @@ def check_location(self): height_numeric = float(height) if height else None if not height or -50 <= height_numeric <= 5100: LOGGER.debug('Validated instrument height') - elif not self._add_to_report(326, valueline, lower=-50, upper=5100): + elif not self._add_to_report( + 326, valueline, lower=-50, upper=5100): success = False except ValueError: if not self._add_to_report(324, valueline): @@ -1147,7 +1148,10 @@ def check_time_series(self): success = False else: if other_date > dg_date: - err_code = 336 if table.startswith('TIMESTAMP') else 337 + err_code = ( + 336 if table.startswith('TIMESTAMP') + else 337 + ) if not self._add_to_report(err_code, line, table=table): success = False diff --git a/woudc_data_registry/tests/test_report_generation.py b/woudc_data_registry/tests/test_report_generation.py index 5690628..2b5c70c 100644 --- a/woudc_data_registry/tests/test_report_generation.py +++ b/woudc_data_registry/tests/test_report_generation.py @@ -136,7 +136,6 @@ def test_uses_error_definition(self): _, success = op_report.add_message(101) self.assertFalse(success) - def test_passing_operator_report(self): """Test that a passing file is written in the operator report""" @@ -938,7 +937,6 @@ def test_email_summary_single_fix(self): with open(output_path) as output: lines = output.read().splitlines() self.assertEqual(len(lines), 8) - self.assertEqual(lines[0], 'MSC (placeholder@mail.com)') self.assertEqual(lines[1], 'Total files received: 1') self.assertEqual(lines[2], 'Number of passed files: 0') @@ -1147,7 +1145,6 @@ def test_email_summary_multiple_causes(self): self.assertIn(lines[9], fail_group) self.assertNotIn('.csv', lines[10]) self.assertIn(lines[11], fail_group) - self.assertEqual(lines[12], 'Summary of Fixes:') self.assertNotIn('.csv', lines[13]) self.assertIn(lines[14], fix_group)