Fix bug with deleting reports (#372)

* Allow ARP from 0.0.0.0 * Fix logic * Fix deleting report
google · Apr 5, 2024 · f51aaf5 · f51aaf5
1 parent 61e69f1
commit f51aaf5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/framework/python/src/common/device.py b/framework/python/src/common/device.py
@@ -43,9 +43,9 @@ def get_reports(self):
 
   def remove_report(self, timestamp: datetime):
     for report in self.reports:
-      if report.get_started() == timestamp:
+      if report.get_started().strftime('%Y-%m-%dT%H:%M:%S') == timestamp:
         self.reports.remove(report)
-        break
+        return
 
   def to_dict(self):
     """Returns the device as a python dictionary. This is used for the

diff --git a/framework/python/src/core/testrun.py b/framework/python/src/core/testrun.py
@@ -232,6 +232,7 @@ def delete_report(self, device: Device, timestamp):
       if report_folder == timestamp:
         shutil.rmtree(os.path.join(reports_folder, report_folder))
         device.remove_report(timestamp)
+        LOGGER.debug('Successfully deleted the report')
         return True
 
     return False