Randomize application names in tests

[ishan-karmakar]

Closes #1359

[devdattakulkarni]

I have added some comments.

[devdattakulkarni]

Here are some comments from GitHub Copilot:

A.
In cleanup() (application-upgrade section) there is:
cmd1 = f"kubectl delete -f {tenant_file} --kubeconfig=./application-upgrade/provider.conf"
TestKubePlus.run_command(cmd)
The code calls run_command(cmd) instead of run_command(cmd1). This looks like a copy/paste bug that will cause the delete to be skipped and the previous command to be run again. Search the file for similar patterns: assignment to cmd1 but calling cmd.
Fix: call TestKubePlus.run_command(cmd1) (or consolidate to reuse cmd variable consistently).

B. Use of classmethod parameter named self
Classmethods are defined as @classmethod def setUpClass(self): and def _process_template(self,…):
By convention and clarity, rename the first arg to cls (def setUpClass(cls): ... def _process_template(cls, file):). This avoids confusion between class and instance scope and makes code more idiomatic.
Functionally not fatal but improves readability.

C. Path.unlink(..., missing_ok=True) compatibility
Path.unlink(missing_ok=True) requires Python 3.8+. Ensure the CI/test runner uses Python >= 3.8. If CI uses older Python, this will fail; use try/except instead for maximum compatibility:
try: pathlib.Path(path).unlink() except FileNotFoundError: # Use some other Python approach to delete/unlink file.
D. Temporary files and cleanup robustness
tearDownClass deletes temp files recorded in self.tmp_files. Good, but ensure tearDownClass runs if tests abort early (unittest will call tearDownClass only if setUpClass succeeded; for other failures you may still leave temp files). Consider:
Using tempfile.TemporaryDirectory with context or registering cleanup via atexit, or wrapping test run with try/finally.
At minimum, keep current approach but handle FileNotFoundError (see C).

[devdattakulkarni]

Looks good! 💯