Update webhook test for VM webhook update

[rajivnathan]

Paired with codeready-toolchain/member-operator#607

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alexeykazakov, rajivnathan

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 [alexeykazakov,rajivnathan]

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

[mfrancisc]

Should we add test cases for creating and updating vms with runStrategy set and expect that the webhook rejects that ?

Basically updating this test

toolchain-e2e/test/e2e/parallel/vm_webhook_test.go

Line 24 in 9f9588d

func TestCreateVirtualMachine(t *testing.T) {

.

[rajivnathan]

Should we add test cases for creating and updating vms with runStrategy set and expect that the webhook rejects that ?

Basically updating this test

toolchain-e2e/test/e2e/parallel/vm_webhook_test.go

Line 24 in 9f9588d

func TestCreateVirtualMachine(t *testing.T) {

.

We mainly rely on unit testing for all the validating webhooks if I'm not mistaken. But if we think it's needed, I can give it a try.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud