Improve runner speed by not copying skeletons for virtual users

blechschmidt · Jun 22, 2024 · bd97738 · bd97738
1 parent d2cf320
commit bd97738
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 11 deletions.
diff --git a/tests/test_cli.py b/tests/test_cli.py
@@ -92,15 +92,21 @@ class PalliumTestCase(unittest.TestCase):
     def test_virtuser(self):
         profile = {
             'sandbox': {
-                'virtuser': 'johndoe'
+                'virtuser': {
+                    'name': 'johndoe',
+                    'skeleton': False
+                }
             }
         }
         assert pallium_exec_output(profile, ['whoami']) == 'johndoe'
 
     def test_mv(self):
         profile = {
             'sandbox': {
-                'virtuser': 'johndoe'
+                'virtuser': {
+                    'name': 'johndoe',
+                    'skeleton': False
+                }
             },
             'run': {
                 'command': ['sleep', 'infinity']

diff --git a/tests/test_sandbox.py b/tests/test_sandbox.py
@@ -43,7 +43,10 @@ def test_gvisor(self):
     def test_virtuser(self):
         profile = {
             "sandbox": {
-                "virtuser": "johndoe"
+                "virtuser": {
+                    "name": "johndoe",
+                    "skeleton": False
+                }
             }
         }
         whoami_output = get_output(profile, 'whoami')
@@ -109,14 +112,6 @@ def test_mount(self):
             assert get_output(profile, ["cat", dst]) == "Hello world", "Test bind mounting test file at " % dst
 
 
-    def test_mv(self):
-        profile = {
-            "sandbox": {
-                "virtuser": "$tmp"
-            }
-        }
-
-
 if __name__ == '__main__':
     assert os.getuid() != 0, "These tests should not be run as root user"
     unittest.main()