Skip to content

Commit

Permalink
Add info about sorting of datetime ISO strings, which now should work…
Browse files Browse the repository at this point in the history
… correctly
  • Loading branch information
seberm committed Nov 12, 2024
1 parent fcb9d72 commit 3ed197b
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions tmt/steps/report/reportportal.py
Original file line number Diff line number Diff line change
Expand Up @@ -400,8 +400,9 @@ def go(self, *, logger: Optional[tmt.log.Logger] = None) -> None:
# Support for idle tests
executed = bool(self.step.plan.execute.results())
if executed:
# launch time should be the earliest start time of all plans
# TODO: Does the 'min' work with datetime isoformat correctly?
# Launch time should be the earliest start time of all plans. The minimum over datetime
# strings will work, because the datetime in ISO format is designed to be
# lexicographically sortable.
launch_time = min([r.start_time or self.datetime
for r in self.step.plan.execute.results()])

Expand Down

0 comments on commit 3ed197b

Please sign in to comment.