🌱 Unit test and builder cleanups

[killianmuldoon]

This is a set of small cleanups for unit tests and the builder package:

1. create a spec for unstructured objects at build time and remove unnecessary setting inside tests.
2. remove resourceVersion where it was still set unnecessarily.
3. Add WithSpecFields comments throughout to make it consistent
4. Removed a TODO which is unnecessary (WILLNOTDO!)

Fixes #5207

[sbueringer]

@killianmuldoon Could it be that there is something missing? I couldn't find 2. and 4. (and probably also not 3.)

[killianmuldoon]

@killianmuldoon Could it be that there is something missing? I couldn't find 2. and 4. (and probably also not 3.)

Definitely was something missing! Should be updated now.

[sbueringer]

Nice cleanup. Looks a lot better now! :)

/lgtm

[fabriziopandini]

Thanks for getting rid of some of ours todo!
lgtm pending squash

[sbueringer]

/lgtm

[fabriziopandini]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fabriziopandini

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 [fabriziopandini]

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