Remove intermediate function time_ms

Co-authored-by: Dustin Ingram <[email protected]>

Author: ewjoachim

tests/unit/test_tasks.py

@@ -167,6 +167,8 @@ def test_creates_request(self, monkeypatch):
         assert obj.request.pyramid_env == pyramid_env
         assert request is pyramid_env["request"]
         assert isinstance(request.tm, transaction.TransactionManager)
+        assert 1.5e12 < request.timings["new_request_start"] < 1e13
+        assert request.remote_addr == "127.0.0.1"
 
     def test_reuses_request(self):
         pyramid_env = {"request": pretend.stub()}

warehouse/tasks.py

@@ -49,10 +49,6 @@ def _params_from_url(self, url, defaults):
         return params
 
 
-def time_ms():
-    return time.time() * 1000
-
-
 class WarehouseTask(celery.Task):
     def __new__(cls, *args, **kwargs):
         obj = super().__new__(cls, *args, **kwargs)
@@ -95,7 +91,7 @@ def get_request(self):
             registry = self.app.pyramid_config.registry
             env = pyramid.scripting.prepare(registry=registry)
             env["request"].tm = transaction.TransactionManager(explicit=True)
-            env["request"].timings = {"new_request_start": time_ms()}
+            env["request"].timings = {"new_request_start": time.time() * 1000}
             env["request"].remote_addr = "127.0.0.1"
             self.request.update(pyramid_env=env)