diff options
| author | Hervé Beraud <hberaud@redhat.com> | 2022-08-02 12:27:25 +0200 |
|---|---|---|
| committer | Hervé Beraud <hberaud@redhat.com> | 2022-08-03 16:47:25 +0200 |
| commit | 7b838f5c31c3219d13661ec612d750e58683cbe6 (patch) | |
| tree | 526b1db72a935cfd7559d182fdd403d96838b99f | |
| parent | 78098e6b18026ff9ef03a948b57348f02d42e13b (diff) | |
| download | oslo-config-7b838f5c31c3219d13661ec612d750e58683cbe6.tar.gz | |
Disable a config test if ran under the root user
The `test_conf_file_permission_denied` is designed to test
if a config file can't be read due to missing file permissions.
However if we run unit tests with the root user we will beneficiat
from the root privileges and so this test will fails. This can
be reproduced by running `tox` with `sudo`.
These changes propose to skip this test if root privileges are
detected.
Closes-bug: #1983358
Change-Id: Idd124082265032c30f83242b612d002d7093aa61
| -rw-r--r-- | oslo_config/tests/test_cfg.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/oslo_config/tests/test_cfg.py b/oslo_config/tests/test_cfg.py index 16ec21d..ec2791e 100644 --- a/oslo_config/tests/test_cfg.py +++ b/oslo_config/tests/test_cfg.py @@ -21,6 +21,7 @@ import os import shutil import sys import tempfile +import unittest from unittest import mock import fixtures @@ -3839,6 +3840,8 @@ class SadPathTestCase(BaseTestCase): self.assertRaises(cfg.ConfigFilesNotFoundError, self.conf, ['--config-file', path]) + @unittest.skipIf(os.getuid() == 0, + "Not supported with the root privileges") def test_conf_file_permission_denied(self): (fd, path) = tempfile.mkstemp() |