Skip to content

Commit

Permalink
Update jobset_webhook_test.go to use ManagedBy field of JobSetSpec.
Browse files Browse the repository at this point in the history
  • Loading branch information
Justin Edwins committed Mar 28, 2024
1 parent 34c2b26 commit ef199e7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/integration/webhook/jobset_webhook_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -347,10 +347,10 @@ var _ = ginkgo.Describe("jobset webhook defaulting", func() {
Obj())
},
defaultsApplied: func(js *jobset.JobSet) bool {
return js.Labels[jobset.LabelManagedBy] == jobset.JobSetManager
return ptr.Deref(js.Spec.ManagedBy, "") == jobset.JobSetManager
},
updateJobSet: func(js *jobset.JobSet) {
js.Labels[jobset.LabelManagedBy] = "new-manager"
js.Spec.ManagedBy = ptr.To("new-manager")
},
updateShouldFail: true,
}),
Expand Down

0 comments on commit ef199e7

Please sign in to comment.