fix(CodeChunk.duration): Recording duration in seconds rather than mi…

…lliseconds

stencila/pyla/interpreter.py

@@ -98,12 +98,12 @@ def __exit__(self, exc_type, exc_val, exc_tb):
         self.duration = datetime.datetime.now() - self._start_time
 
     @property
-    def duration_ms(self) -> float:
+    def duration_seconds(self) -> float:
         """Calculate the duration (time between `__enter__` and `__exit__` in microseconds."""
         if not self.duration:
             raise RuntimeError('CodeTimer has not yet been run')
 
-        return self.duration.total_seconds() * 1000
+        return self.duration.total_seconds()
 
 
 class StdoutBuffer(TextIOWrapper):
@@ -374,7 +374,7 @@ def execute_statement(self, statement: ast.stmt, chunk: CodeChunk, _locals: typi
             try:
                 with CodeTimer() as code_timer:
                     result = run_function(code_to_run, self.globals, _locals)
-                duration += code_timer.duration_ms
+                duration += code_timer.duration_seconds
             # pylint: disable=W0703  # we really don't know what Exception some exec'd code might raise.
             except Exception as exc:
                 error_occurred = True

tests/test_interpreter.py

@@ -106,7 +106,7 @@ def test_execute_compilation(mock_interpreter_class, mock_pp_class):
 
     mock_pp_class.assert_called_with(compilation_result.parameters)
     parameter_parser.parse_cli_args.assert_called_with(parameters)
-    interpreter.execute.assert_not_called() #Because nothing in compilation_result
+    interpreter.execute.assert_not_called()  # Because nothing in compilation_result
 
 
 def test_sempahore_skipping():