add unit test for jobset webhook updates

[kannon92]

Add unit test for updates.

[netlify]

✅ Deploy Preview for kubernetes-sigs-jobset canceled.

Name	Link
🔨 Latest commit	1e26dc4
🔍 Latest deploy log	https://app.netlify.com/sites/kubernetes-sigs-jobset/deploys/660b0a01cf5af30008f5df7c

[danielvegamyhre]

Thanks for improving the test coverage here. Let's merge this after the complex validation PR is checked in, I think merge conflicts will be easier to resolve that way than the reverse

[kannon92]

Thanks for improving the test coverage here. Let's merge this after the complex validation PR is checked in, I think merge conflicts will be easier to resolve that way than the reverse

Sounds good.

I was thinking about #463 and I was going to add unit tests for that code. Decided to move it to a separate PR so we can at least document better that elastic jobs are not supported yet.

[kannon92]

/hold

Waiting on kubectl-validate code.

[kannon92]

Thanks for improving the test coverage here. Let's merge this after the complex validation PR is checked in, I think merge conflicts will be easier to resolve that way than the reverse

I think we can merge this now.

[kannon92]

/hold cancel

[kannon92]

@danielvegamyhre I think we can get this in now. PTAL

[danielvegamyhre]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danielvegamyhre, kannon92

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [danielvegamyhre]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment