test: replace unittest.skip decorators with pytest equivalents

[blackboxsw]

Proposed Commit Message

See commit descriptions

• test: fix bug in tmp_path used as a callable for path join operation

• test: replace unittest.skip decoartors with pytest equivalents

  Issue: When running tests with pytest, unittest-style skip decorators raise unittest.case.SkipTest exceptions which pytest interprets as FAILED rather than SKIPPED on non-unittest.TestCase classes. This caused tests to incorrectly show as failures.

Replace unittest.skipIf/skip with pytest.mark.skipif/skip to prevent SkipTest exceptions from being treated as test failures.

Fixes GH-6569

• I have signed the CLA: https://ubuntu.com/legal/contributors
• I have included a comprehensive commit message using the guide below
• I have adapted unit tests to cover the new behavior under tests/unittests/
• I have kept the change small, avoiding unnecessary whitespace or non-functional changes.
• I have added a reference to issues that this PR relates to in the PR message (Refs integration: do not LXD bind mount /etc/cloud/cloud.cfg.d #1234, Fixes integration: do not LXD bind mount /etc/cloud/cloud.cfg.d #1234)

Additional Context

CI build failure reproducing this issue

Test Steps

In a questing environment:  `tox -e py3 `

Merge type

• Squash merge using "Proposed Commit Message"
• Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

[Copilot]

Pull Request Overview

This PR addresses a test framework compatibility issue where unittest-style skip decorators (unittest.skipIf, unittest.skip) were incorrectly marking tests as FAILED instead of SKIPPED when running non-unittest.TestCase classes with pytest. The changes replace all unittest skip decorators with their pytest equivalents (pytest.mark.skipif, pytest.mark.skip).

Key changes:

• Replaced unittest.skipIf and unittest.skip decorators with pytest.mark.skipif and pytest.mark.skip throughout the test suite
• Updated helper functions that returned unittest skip decorators to return pytest skip markers instead
• Added reason parameter to all skip decorators for better test output clarity

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tests/unittests/helpers.py	Removed skipIf alias to unittest.skipIf and updated all helper functions to return pytest.mark.skipif instead
tests/unittests/test_util.py	Replaced unittest.skipIf decorators with pytest.mark.skipif on test classes and methods, removed import of skipIf helper
tests/unittests/sources/test_smartos.py	Replaced skipIf decorator with pytest.mark.skipif, removed import of skipIf helper
tests/unittests/sources/test_azure_helper.py	Replaced unittest.skip decorators with pytest.mark.skip
tests/unittests/config/test_cc_chef.py	Replaced skipIf decorators with pytest.mark.skipif, removed import of skipIf helper
tests/unittests/test_templating.py	Fixed unrelated bug where tmp_path was being called as a function instead of using path concatenation

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[cjp256]

LGTM! Cade was just working on the azure tests to ensure they run - can follow after this PR merges.

[blackboxsw]

Thanks @cjp256 since these test changes affect a few PRs in flight, let's merge this so affected developers can rebase and pull in the 'fixes'.