Verify image build functionality using kubedock and podman when spec.devEnvironments.persistUserHome.enabled is set to true

[SkorikSergey]

What does this PR do?

Add a new E2E test to verify image build functionality using kubedock and podman when spec.devEnvironments.persistUserHome.enabled is set to true in the CheCluster.

Test Coverage

The new test (KubedockPodmanPersistUserHomeTest.spec.ts) covers P03 scenario:

• Enables persistUserHome in CheCluster before test execution
• Creates workspace from dockerfile-hello-world sample
• Builds and pushes container image using podman with kubedock
• Runs the built container and verifies output
• Restores original persistUserHome setting after test completion

Changes

• Added new test file tests/e2e/specs/miscellaneous/KubedockPodmanPersistUserHomeTest.spec.ts

Test Plan

Screenshot/screencast of this PR

What issues does this PR fix or reference?

https://redhat.atlassian.net/browse/CRW-10596

How to test this PR?

PR Checklist

As the author of this Pull Request I made sure that:

• The Eclipse Contributor Agreement is valid
• Code produced is complete
• Code builds without errors
• Tests are covering the bugfix or new feature
• The repository devfile is up to date and works
• Sections What issues does this PR fix or reference and How to test this PR completed
• Relevant user documentation updated
• Relevant contributing documentation updated
• CI/CD changes implemented, documented and communicated

Reviewers

Reviewers, please comment how you tested the PR when approving it.

[dmytro-ndp]

The test doesn't fail when oc patch command fails, which leaves patched config the same silently.

Possible fix:

const restoreResult: ShellString = shellExecutor.executeCommand(
    `oc patch checluster ${cheClusterName} --type=merge ` +
        `-p '{"spec":{"devEnvironments":{"persistUserHome":{"enabled": ${defaultPersistentHomeValue}}}}}'`
);                                                                                  

expect(restoreResult.code).to.equal(0, 'Failed to restore CheCluster persistUserHome setting');

[dmytro-ndp]

The test doesn't fail when oc patch command fails silently, making subsequent test failures harder to diagnose.

Possible fix:

const patchResult: ShellString = shellExecutor.executeCommand(                                                                                                                              
    `oc patch checluster ${cheClusterName} --type=merge ` +                                                                                                                                 
        '-p \'{"spec":{"devEnvironments":{"persistUserHome":{"enabled": true}}}}\''                                                                                                         
);                                                                                                                                                                                          
expect(patchResult.code).to.equal(0, 'Failed to patch CheCluster to enable persistUserHome'); 

[dmytro-ndp]

The test looks generally good, but there is still room for improvement.

[dmytro-ndp]

@SkorikSergey: thank you for taking review feedback into account.

Well done!