Handle empty PATH environment variable

In locked down environments, PATH might be unavailable
when running rootwrap.

Change-Id: Ia55514a7d69ab26c2bcf5d1839da1d36aaf46ebc

1 files changed, 8 insertions, 2 deletions

diff --git a/tests/unit/test_rootwrap.py b/tests/unit/test_rootwrap.py
index 02789ec..6e1e6e6 100644
--- a/tests/unit/test_rootwrap.py
+++ b/tests/unit/test_rootwrap.py
@@ -178,8 +178,9 @@ class RootwrapTestCase(utils.BaseTestCase):
             # Filter shouldn't be able to find binary in $PATH, so fail
             with fixtures.EnvironmentVariable("PATH", "/foo:/bar"):
                 self.assertFalse(f.match(usercmd))
-            pass
-
+            # ensure that unset $PATH is not causing an exception
+            with fixtures.EnvironmentVariable("PATH"):
+                self.assertFalse(f.match(usercmd))
         finally:
             # Terminate the "cat" process and wait for it to finish
             p.terminate()
@@ -314,6 +315,11 @@ class RootwrapTestCase(utils.BaseTestCase):
         config = wrapper.RootwrapConfig(raw)
         self.assertEqual(config.filters_path, ['/a', '/b'])
         self.assertEqual(config.exec_dirs, os.environ["PATH"].split(':'))
+
+        with fixtures.EnvironmentVariable("PATH"):
+            c = wrapper.RootwrapConfig(raw)
+            self.assertEqual(c.exec_dirs, [])
+
         self.assertFalse(config.use_syslog)
         self.assertEqual(config.syslog_log_facility,
                          logging.handlers.SysLogHandler.LOG_SYSLOG)