Fix test breakage caused by r374424
authorLawrence D'Anna <lawrence_danna@apple.com>
Mon, 14 Oct 2019 18:53:27 +0000 (18:53 +0000)
committerLawrence D'Anna <lawrence_danna@apple.com>
Mon, 14 Oct 2019 18:53:27 +0000 (18:53 +0000)
Summary:
The build directory name is based on the test method name, so having
two test methods with the same name in the same test file is a
problem, even if they're in different test classes.

On linux and darwin this conflict can go unnoticed, but windows
has different filesystem semantics and it will fail when one
process tries to delete files still held open by another.

The problem is fixed just by changing the name of one of the test
methods.

Reviewers: JDevlieghere, jasonmolenda, labath, stella.stamenova

Reviewed By: stella.stamenova

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D68951

llvm-svn: 374803

lldb/packages/Python/lldbsuite/test/python_api/interpreter/TestRunCommandInterpreterAPI.py

index ede4013..a32805b 100644 (file)
@@ -30,7 +30,7 @@ class CommandRunInterpreterLegacyAPICase(TestBase):
         self.dbg.SetErrorFileHandle (self.devnull, False)
 
     @add_test_categories(['pyapi'])
-    def test_run_session_with_error_and_quit(self):
+    def test_run_session_with_error_and_quit_legacy(self):
         """Run non-existing and quit command returns appropriate values"""
 
         n_errors, quit_requested, has_crashed = self.dbg.RunCommandInterpreter(