refactor: make timeout seconds a JobSpec property

In future scheduler/launcher work this property may need to be used
multiple times, so refactor slightly to add it is a property of the
`JobSpec` itself.

Signed-off-by: Alex Jones <alex.jones@lowrisc.org>

Author: AlexJones0

src/dvsim/job/data.py

@@ -117,6 +117,11 @@ class JobSpec(BaseModel):
     fail_patterns: Sequence[str]
     """regex patterns to match on to determine if the job has failed."""
 
+    @property
+    def timeout_secs(self) -> int | None:
+        """Returns the timeout applied to the launched job, in seconds."""
+        return None if self.timeout_mins is None else self.timeout_mins * 60
+
 
 class CompletedJobStatus(BaseModel):
     """Job status."""

src/dvsim/launcher/local.py

@@ -59,9 +59,7 @@ def _do_launch(self) -> None:
             self._log_file.flush()
 
             if not self.job_spec.interactive:
-                timeout_mins = self.job_spec.timeout_mins
-
-                self.timeout_secs = timeout_mins * 60 if timeout_mins else None
+                self.timeout_secs = self.job_spec.timeout_secs
 
                 try:
                     self._process = subprocess.Popen(