From 579b8c90c6c953bb51e1b9d3f2e33df19700c5c3 Mon Sep 17 00:00:00 2001 From: Pneha1234 Date: Fri, 17 Oct 2025 08:37:30 +0545 Subject: [PATCH] test(config): Convert test_cc_spacewalk.py from unittest to pytest Refactored tests/unittests/config/test_cc_spacewalk.py to use pytest instead of unittest.TestCase as part of the pytest migration effort. - Removed TestCase inheritance - Removed unused helpers import - Converted self.assertFalse() to strict boolean check (is False) - Converted self.assertTrue() to strict boolean check (is True) - Maintained all original test functionality Related: #6427 --- tests/unittests/config/test_cc_spacewalk.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/tests/unittests/config/test_cc_spacewalk.py b/tests/unittests/config/test_cc_spacewalk.py index f202af40b79..42f07e0c76e 100644 --- a/tests/unittests/config/test_cc_spacewalk.py +++ b/tests/unittests/config/test_cc_spacewalk.py @@ -5,12 +5,11 @@ from cloudinit import subp from cloudinit.config import cc_spacewalk -from tests.unittests import helpers LOG = logging.getLogger(__name__) -class TestSpacewalk(helpers.TestCase): +class TestSpacewalk: space_cfg = { "spacewalk": { "server": "localhost", @@ -21,12 +20,12 @@ class TestSpacewalk(helpers.TestCase): @mock.patch("cloudinit.config.cc_spacewalk.subp.subp") def test_not_is_registered(self, mock_subp): mock_subp.side_effect = subp.ProcessExecutionError(exit_code=1) - self.assertFalse(cc_spacewalk.is_registered()) + assert cc_spacewalk.is_registered() is False @mock.patch("cloudinit.config.cc_spacewalk.subp.subp") def test_is_registered(self, mock_subp): mock_subp.side_effect = None - self.assertTrue(cc_spacewalk.is_registered()) + assert cc_spacewalk.is_registered() is True @mock.patch("cloudinit.config.cc_spacewalk.subp.subp") def test_do_register(self, mock_subp):