run: Normalize paths

Fixes #364.

Signed-off-by: Stephen Finucane <[email protected]>

stestr/commands/run.py

@@ -527,7 +527,7 @@ def run_command(
             ids, klass = ids.split("::", 1)
             klass = ".".join(klass.split("::"))
         if ids.find(os.path.sep) != -1:
-            root, _ = os.path.splitext(ids)
+            root, _ = os.path.splitext(os.path.normpath(ids))
             ids = ".".join(root.split(os.path.sep))
         if klass:
             ids = f"{ids}.{klass}"
@@ -594,7 +594,7 @@ def run_tests():
             ids, klass = ids.split("::", 1)
             klass = ".".join(klass.split("::"))
         if ids.find(os.path.sep) != -1:
-            root, _ = os.path.splitext(ids)
+            root, _ = os.path.splitext(os.path.normpath(ids))
             ids = ".".join(root.split(os.path.sep))
         if klass:
             ids = f"{ids}.{klass}"

stestr/tests/test_return_codes.py

@@ -500,6 +500,24 @@ def test_run_no_discover_file_path_failing(self):
         self.assertIn(" - Failed: 2", lines)
         self.assertIn(" - Unexpected Success: 1", lines)
 
+    def test_run_no_discover_unnormalized_file_path(self):
+        # we don't use os.path.join since we want an unnormalized path
+        passing_string = f"tests{os.path.sep}{os.path.sep}test_passing.py"
+        out, err = self.assertRunExit("stestr run -n %s" % passing_string, 0)
+        lines = out.decode("utf8").splitlines()
+        self.assertIn(" - Passed: 2", lines)
+        self.assertIn(" - Failed: 0", lines)
+        self.assertIn(" - Expected Fail: 1", lines)
+
+    def test_run_no_discover_unnormalized_file_path_failing(self):
+        # we don't use os.path.join since we want an unnormalized path
+        failing_string = f"tests{os.path.sep}{os.path.sep}test_failing.py"
+        out, err = self.assertRunExit("stestr run -n %s" % failing_string, 1)
+        lines = out.decode("utf8").splitlines()
+        self.assertIn(" - Passed: 0", lines)
+        self.assertIn(" - Failed: 2", lines)
+        self.assertIn(" - Unexpected Success: 1", lines)
+
 
 class TestReturnCodesToxIni(TestReturnCodes):
     def setUp(self):