diff options
author | Andrei Zeliankou <zelenkov@nginx.com> | 2023-06-14 18:20:09 +0100 |
---|---|---|
committer | Andrei Zeliankou <zelenkov@nginx.com> | 2023-06-14 18:20:09 +0100 |
commit | c183bd8749a19477390f8cb77efe5f6d223f0905 (patch) | |
tree | 4e821e9cb07be9a86bf2d442acb3ea6740ba5a99 /test/test_python_isolation_chroot.py | |
parent | c6d05191a069ac150cc8eb2bece75cf79c0a465a (diff) | |
download | unit-c183bd8749a19477390f8cb77efe5f6d223f0905.tar.gz unit-c183bd8749a19477390f8cb77efe5f6d223f0905.tar.bz2 |
Tests: get rid of classes in test files.
Class usage came from the unittest framework and it was always redundant
after migration to the pytest. This commit removes classes from files
containing tests to make them more readable and understandable.
Diffstat (limited to 'test/test_python_isolation_chroot.py')
-rw-r--r-- | test/test_python_isolation_chroot.py | 37 |
1 files changed, 18 insertions, 19 deletions
diff --git a/test/test_python_isolation_chroot.py b/test/test_python_isolation_chroot.py index fd16c1fb..60fac5ef 100644 --- a/test/test_python_isolation_chroot.py +++ b/test/test_python_isolation_chroot.py @@ -1,30 +1,29 @@ -from unit.applications.lang.python import TestApplicationPython +from unit.applications.lang.python import ApplicationPython prerequisites = {'modules': {'python': 'any'}, 'privileged_user': True} +client = ApplicationPython() -class TestPythonIsolation(TestApplicationPython): - def test_python_isolation_chroot(self, temp_dir): - isolation = {'rootfs': temp_dir} - self.load('ns_inspect', isolation=isolation) +def test_python_isolation_chroot(temp_dir): + client.load('ns_inspect', isolation={'rootfs': temp_dir}) - assert not ( - self.getjson(url=f'/?path={temp_dir}')['body']['FileExists'] - ), 'temp_dir does not exists in rootfs' + assert not ( + client.getjson(url=f'/?path={temp_dir}')['body']['FileExists'] + ), 'temp_dir does not exists in rootfs' - assert self.getjson(url='/?path=/proc/self')['body'][ - 'FileExists' - ], 'no /proc/self' + assert client.getjson(url='/?path=/proc/self')['body'][ + 'FileExists' + ], 'no /proc/self' - assert not ( - self.getjson(url='/?path=/dev/pts')['body']['FileExists'] - ), 'no /dev/pts' + assert not ( + client.getjson(url='/?path=/dev/pts')['body']['FileExists'] + ), 'no /dev/pts' - assert not ( - self.getjson(url='/?path=/sys/kernel')['body']['FileExists'] - ), 'no /sys/kernel' + assert not ( + client.getjson(url='/?path=/sys/kernel')['body']['FileExists'] + ), 'no /sys/kernel' - ret = self.getjson(url='/?path=/app/python/ns_inspect') + ret = client.getjson(url='/?path=/app/python/ns_inspect') - assert ret['body']['FileExists'], 'application exists in rootfs' + assert ret['body']['FileExists'], 'application exists in rootfs' |