diff options
| author | zsdc <taras@vyos.io> | 2022-03-25 20:58:01 +0200 | 
|---|---|---|
| committer | zsdc <taras@vyos.io> | 2022-03-25 21:42:00 +0200 | 
| commit | 31448cccedd8f841fb3ac7d0f2e3cdefe08a53ba (patch) | |
| tree | 349631a02467dae0158f6f663cc8aa8537974a97 /tests/unittests/test_conftest.py | |
| parent | 5c4b3943343a85fbe517e5ec1fc670b3a8566b4b (diff) | |
| parent | 8537237d80a48c8f0cbf8e66aa4826bbc882b022 (diff) | |
| download | vyos-cloud-init-31448cccedd8f841fb3ac7d0f2e3cdefe08a53ba.tar.gz vyos-cloud-init-31448cccedd8f841fb3ac7d0f2e3cdefe08a53ba.zip | |
T2117: Cloud-init updated to 22.1
Merged with 22.1 tag from the upstream Cloud-init repository.
Our modules were slightly modified for compatibility with the new
version.
Diffstat (limited to 'tests/unittests/test_conftest.py')
| -rw-r--r-- | tests/unittests/test_conftest.py | 65 | 
1 files changed, 65 insertions, 0 deletions
| diff --git a/tests/unittests/test_conftest.py b/tests/unittests/test_conftest.py new file mode 100644 index 00000000..68903430 --- /dev/null +++ b/tests/unittests/test_conftest.py @@ -0,0 +1,65 @@ +import pytest + +from cloudinit import subp +from tests.unittests.helpers import CiTestCase + + +class TestDisableSubpUsage: +    """Test that the disable_subp_usage fixture behaves as expected.""" + +    def test_using_subp_raises_assertion_error(self): +        with pytest.raises(AssertionError): +            subp.subp(["some", "args"]) + +    def test_typeerrors_on_incorrect_usage(self): +        with pytest.raises(TypeError): +            # We are intentionally passing no value for a parameter, so: +            #  pylint: disable=no-value-for-parameter +            subp.subp() + +    @pytest.mark.allow_all_subp +    def test_subp_usage_can_be_reenabled(self): +        subp.subp(["whoami"]) + +    @pytest.mark.allow_subp_for("whoami") +    def test_subp_usage_can_be_conditionally_reenabled(self): +        # The two parameters test each potential invocation with a single +        # argument +        with pytest.raises(AssertionError) as excinfo: +            subp.subp(["some", "args"]) +        assert "allowed: whoami" in str(excinfo.value) +        subp.subp(["whoami"]) + +    @pytest.mark.allow_subp_for("whoami", "bash") +    def test_subp_usage_can_be_conditionally_reenabled_for_multiple_cmds(self): +        with pytest.raises(AssertionError) as excinfo: +            subp.subp(["some", "args"]) +        assert "allowed: whoami,bash" in str(excinfo.value) +        subp.subp(["bash", "-c", "true"]) +        subp.subp(["whoami"]) + +    @pytest.mark.allow_all_subp +    @pytest.mark.allow_subp_for("bash") +    def test_both_marks_raise_an_error(self): +        with pytest.raises(AssertionError, match="marked both"): +            subp.subp(["bash"]) + + +class TestDisableSubpUsageInTestSubclass(CiTestCase): +    """Test that disable_subp_usage doesn't impact CiTestCase's subp logic.""" + +    def test_using_subp_raises_exception(self): +        with pytest.raises(Exception): +            subp.subp(["some", "args"]) + +    def test_typeerrors_on_incorrect_usage(self): +        with pytest.raises(TypeError): +            subp.subp() + +    def test_subp_usage_can_be_reenabled(self): +        _old_allowed_subp = self.allow_subp +        self.allowed_subp = True +        try: +            subp.subp(["bash", "-c", "true"]) +        finally: +            self.allowed_subp = _old_allowed_subp | 
