Fix other instances of job_id missed in #363.

flow/operations.py

@@ -200,7 +200,7 @@ def hello(job):
         help="The operation to execute.",
     )
     parser.add_argument(
-        "jobid",
+        "job_id",
         type=str,
         nargs="*",
         help="The job ids, as registered in the signac project. "
@@ -238,9 +238,9 @@ def _open_job_by_id(_id):
         except LookupError:
             raise LookupError(f"Multiple matches for id '{_id}'.")
 
-    if len(args.jobid):
+    if len(args.job_id):
         try:
-            jobs = [_open_job_by_id(jid) for jid in args.jobid]
+            jobs = [_open_job_by_id(jid) for jid in args.job_id]
         except (KeyError, LookupError) as error:
             print(error, file=sys.stderr)
             sys.exit(1)

flow/project.py

@@ -4599,7 +4599,7 @@ def operation_function(job):
                 operation_function(*aggregate)
 
     def _select_jobs_from_args(self, args):
-        """Select jobs with the given command line arguments ('-j/-f/--doc-filter/--jobid')."""
+        """Select jobs with the given command line arguments ('-j/-f/--doc-filter/--job-id')."""
         if (
             not args.func == self._main_exec
             and args.job_id

flow/scheduling/base.py

@@ -25,8 +25,8 @@ class JobStatus(enum.IntEnum):
 class ClusterJob:
     """This class represents a cluster job."""
 
-    def __init__(self, jobid, status=None):
-        self._job_id = jobid
+    def __init__(self, job_id, status=None):
+        self._job_id = job_id
         self._status = status
 
     def _id(self):

tests/test_project.py

@@ -1183,9 +1183,9 @@ def test_main_run(self):
 
     def test_main_next(self):
         assert len(self.project)
-        jobids = set(self.call_subcmd("next op1").decode().split())
+        job_ids = set(self.call_subcmd("next op1").decode().split())
         even_jobs = [job.get_id() for job in self.project if job.sp.b % 2 == 0]
-        assert jobids == set(even_jobs)
+        assert job_ids == set(even_jobs)
 
     def test_main_status(self):
         assert len(self.project)