tests: simplify test client

[AndrewChubatiuk]

removed custom logic to intercept k8s client requests with built in functionality

---

Summary by cubic

Simplified the test client by switching from custom request tracking to controller-runtime interceptors, cutting boilerplate and making tests deterministic and clearer.

• Refactors

  • Removed TestClientWithStatsTrack, call counters, wait loops, and ad-hoc goroutines/tickers in cluster tests.
  • Added GetTestClientWithObjects and GetTestClientWithObjectsAndInterceptors (returns client.Client); unified inputs to []runtime.Object.
  • Updated tests across configmap, deployment, service, statefulset (incl. PVC expand), vlagent, vlcluster, vmagent (incl. scrapeconfig), vmalertmanager, vmanomaly, vmcluster, vmdistributed, vtcluster to assert Create/Get/Update via interceptor events.
  • Switched validate funcs to direct assertions; preserved scheme setup and status subresources.

• Migration

  • Replace TestClientWithStatsTrack and GetTestClientWithClientObjects with GetTestClientWithObjects.
  • Use GetTestClientWithObjectsAndInterceptors to capture and assert client actions; drop manual status updaters and wait loops.

^{Written for commit 883e3e5. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 6 files