Backport of api: prevent panic on job plan into release/1.4.x (#17710)

This pull request was automerged via backport-assistant

.changelog/17689.txt

@@ -0,0 +1,3 @@
+```release-note:bug
+api: Fixed a bug that caused a panic when calling the `Jobs().Plan()` function with a job missing an ID
+```

api/jobs.go

@@ -427,6 +427,9 @@ func (j *Jobs) PlanOpts(job *Job, opts *PlanOptions, q *WriteOptions) (*JobPlanR
 	if job == nil {
 		return nil, nil, errors.New("must pass non-nil job")
 	}
+	if job.ID == nil {
+		return nil, nil, errors.New("job is missing ID")
+	}
 
 	// Setup the request
 	req := &JobPlanRequest{

api/jobs_test.go

@@ -1753,6 +1753,12 @@ func TestJobs_Plan(t *testing.T) {
 	_, _, err = jobs.Plan(nil, true, nil)
 	must.Error(t, err)
 
+	// Check that passing a nil job ID fails
+	invalidJob := testJob()
+	invalidJob.ID = nil
+	_, _, err = jobs.Plan(invalidJob, true, nil)
+	must.Error(t, err)
+
 	// Make a plan request
 	planResp, wm, err := jobs.Plan(job, true, nil)
 	must.NoError(t, err)