kbase / execution_engine2

KBase Execution Engine
http://www.kbase.us/services/ee2
MIT License
0 stars 7 forks source link

Fix tests on non-UTC systems #428

Closed MrCreosote closed 2 years ago

MrCreosote commented 2 years ago

Description of PR purpose/changes

The tests pass with or without the changes in this commit in the docker container, which is UTC, but fail without the changes on my laptop, which is not UTC.

The problem is that datetime.utcnow() is not timezone aware, so it only works on UTC systems.

Jira Ticket / Github Issue

Testing Instructions

Dev Checklist:

Updating Version and Release Notes (if applicable)

bio-boris commented 2 years ago

@MrCreosote It should probably be "time.time()" in order to match the other timestamps

MrCreosote commented 2 years ago

@bio-boris updated