#115 added tests and logging for deletion of execution logs

Author: bugy

src/execution/logging.py

@@ -213,6 +213,7 @@ def _renew_files_cache(self):
                 obsolete_ids.append(id)
 
         for obsolete_id in obsolete_ids:
+            LOGGER.info('Logs for execution #' + obsolete_id + ' were deleted')
             del cache[obsolete_id]
 
         for file in os.listdir(self._output_folder):

src/tests/execution_logging_test.py

@@ -194,6 +194,24 @@ def test_history_entries_after_restart(self):
         entry = new_service.get_history_entries()[0]
         self.validate_history_entry(entry, id='id1')
 
+    def test_get_history_entries_after_delete(self):
+        self.simulate_logging(execution_id='id1')
+
+        for file in os.listdir(test_utils.temp_folder):
+            os.remove(os.path.join(test_utils.temp_folder, file))
+
+        entries = self.logging_service.get_history_entries()
+        self.assertCountEqual([], entries)
+
+    def test_find_history_entry_after_delete(self):
+        self.simulate_logging(execution_id='id1')
+
+        for file in os.listdir(test_utils.temp_folder):
+            os.remove(os.path.join(test_utils.temp_folder, file))
+
+        entry = self.logging_service.find_history_entry('id1')
+        self.assertIsNone(entry)
+
     def test_find_history_entry(self):
         self.simulate_logging(execution_id='id1')