[llvm] r182715 - Fixed bug when tests in executable partially used absolute paths.

Galina Kistanova gkistanova at gmail.com
Sat May 25 20:58:41 PDT 2013


Author: gkistanova
Date: Sat May 25 22:58:41 2013
New Revision: 182715

URL: http://llvm.org/viewvc/llvm-project?rev=182715&view=rev
Log:
Fixed bug when tests in executable partially used absolute paths.

Modified:
    llvm/trunk/utils/lit/lit/TestFormats.py

Modified: llvm/trunk/utils/lit/lit/TestFormats.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/TestFormats.py?rev=182715&r1=182714&r2=182715&view=diff
==============================================================================
--- llvm/trunk/utils/lit/lit/TestFormats.py (original)
+++ llvm/trunk/utils/lit/lit/TestFormats.py Sat May 25 22:58:41 2013
@@ -61,9 +61,9 @@ class GoogleTest(object):
         (dirname, basename) = os.path.split(execpath)
         # Discover the tests in this executable.
         for testname in self.getGTestTests(execpath, litConfig, localConfig):
-            testPath = path_in_suite + (dirname, basename, testname)
-            yield Test.Test(testSuite, testPath, localConfig)
-    
+            testPath_in_suite = path_in_suite + (basename, testname)
+            yield Test.Test(testSuite, testPath_in_suite, localConfig)
+
     def getTestsInDirectory(self, testSuite, path_in_suite,
                             litConfig, localConfig):
         source_path = testSuite.getSourcePath(path_in_suite)
@@ -73,15 +73,17 @@ class GoogleTest(object):
                 # Iterate over executables in a directory.
                 if not os.path.normcase(filename) in self.test_sub_dir:
                     continue
+                filepath_in_suite = path_in_suite + (filename, )
                 for subfilename in os.listdir(filepath):
                     execpath = os.path.join(filepath, subfilename)
                     for test in self.getTestsInExecutable(
-                            testSuite, path_in_suite, execpath,
+                            testSuite, filepath_in_suite, execpath,
                             litConfig, localConfig):
                       yield test
             elif ('.' in self.test_sub_dir):
+                filepath_in_suite = path_in_suite + (filename, )
                 for test in self.getTestsInExecutable(
-                        testSuite, path_in_suite, filepath,
+                        testSuite, filepath_in_suite, filepath,
                         litConfig, localConfig):
                     yield test
 





More information about the llvm-commits mailing list