refactor: rename model_dump -> dump

The pydantic model_dump method contains a lot more information, so
rename to allow using these in parallel until all the Deploy classes
are migrated to pydantic.

Signed-off-by: James McCorrie <[email protected]>

Author: machshev

src/dvsim/flow/base.py

@@ -422,7 +422,7 @@ def deploy_objects(self) -> Mapping[str, CompletedJobStatus]:
                 json.dumps(
                     # Sort on full name to ensure consistent ordering
                     sorted(
-                        [d.model_dump() for d in deploy],
+                        [d.dump() for d in deploy],
                         key=lambda d: d["full_name"],
                     ),
                     indent=2,

src/dvsim/job/deploy.py

@@ -328,12 +328,9 @@ def extract_info_from_log(self, job_runtime_secs: int, log_text: list) -> None:
             log.warning(f"{self.full_name}: {e} Using dvsim-maintained job_runtime instead.")
             self.job_runtime.set(job_runtime_secs, "s")
 
-    def model_dump(self) -> Mapping:
+    def dump(self) -> Mapping:
         """Dump the deployment object to mapping object.
 
-        This method matches the interface provided by pydantic models to dump a
-        subset of the class attributes
-
         Returns:
             Representation of a deployment object as a dict.