Add updated_at to JobItem class (tableau#1182)

nosnilmot · web-flow · commit d9f64e144ced · 2023-01-23T19:21:59.000-08:00
diff --git a/tableauserverclient/models/job_item.py b/tableauserverclient/models/job_item.py
@@ -34,6 +34,7 @@ def __init__(
         workbook_id: Optional[str] = None,
         datasource_id: Optional[str] = None,
         flow_run: Optional[FlowRunItem] = None,
+        updated_at: Optional["datetime.datetime"] = None,
     ):
         self._id = id_
         self._type = job_type
@@ -47,6 +48,7 @@ def __init__(
         self._workbook_id = workbook_id
         self._datasource_id = datasource_id
         self._flow_run = flow_run
+        self._updated_at = updated_at
 
     @property
     def id(self) -> str:
@@ -113,9 +115,13 @@ def flow_run(self):
     def flow_run(self, value):
         self._flow_run = value
 
+    @property
+    def updated_at(self) -> Optional["datetime.datetime"]:
+        return self._updated_at
+
     def __repr__(self):
         return (
-            "<Job#{_id} {_type} created_at({_created_at}) started_at({_started_at}) completed_at({_completed_at})"
+            "<Job#{_id} {_type} created_at({_created_at}) started_at({_started_at}) updated_at({_updated_at}) completed_at({_completed_at})"
             " progress ({_progress}) finish_code({_finish_code})>".format(**self.__dict__)
         )
 
@@ -144,6 +150,7 @@ def _parse_element(cls, element, ns):
         datasource = element.find(".//t:datasource[@id]", namespaces=ns)
         datasource_id = datasource.get("id") if datasource is not None else None
         flow_run = None
+        updated_at = parse_datetime(element.get("updatedAt", None))
         for flow_job in element.findall(".//t:runFlowJobType", namespaces=ns):
             flow_run = FlowRunItem()
             flow_run._id = flow_job.get("flowRunId", None)
@@ -163,6 +170,7 @@ def _parse_element(cls, element, ns):
             workbook_id,
             datasource_id,
             flow_run,
+            updated_at,
         )
 
 
diff --git a/test/test_job.py b/test/test_job.py
@@ -53,8 +53,10 @@ def test_get_by_id(self) -> None:
         with requests_mock.mock() as m:
             m.get("{0}/{1}".format(self.baseurl, job_id), text=response_xml)
             job = self.server.jobs.get_by_id(job_id)
+            updated_at = datetime(2020, 5, 13, 20, 25, 18, tzinfo=utc)
 
             self.assertEqual(job_id, job.id)
+            self.assertEqual(updated_at, job.updated_at)
             self.assertListEqual(job.notes, ["Job detail notes"])
 
     def test_get_before_signin(self) -> None:
