Fix TestFileHandle.py. Remove redundant skipIfReproducer annotation

[PatriosTheGreat]

Summary

My recent change (https://reviews.llvm.org/D104413) broke the lldb tests (https://lab.llvm.org/buildbot#builders/17/builds/13800).
The problem is that skipIfReproducer annotation was removed in this change: https://github.com/llvm/llvm-project/commit/b505ed9d313653782b81bbc97979c98edb205558

In this change I remove unnecessary annotation. I checked tests on my local Linux machine they are passing on it.
Not sure if it's better to revert original commit (https://reviews.llvm.org/rGf23b829a2635a6d833bdc81a12d6217b52ae9e45) and then re-commit the change with this fix.

[labath]

I've fixed this in rGc0e3bb4d4ba306.

Not sure if it's better to revert original commit (https://reviews.llvm.org/rGf23b829a2635a6d833bdc81a12d6217b52ae9e45) and then re-commit the change with this fix.

For a simple fix like this, it doesn't really matter what you do. Just don't leave the build broken for extended periods of time. :)

That said, it's pretty hard to do anything when you don't have commit access to the repository, so part of the responsibility also lies with the person who commits the patch for you. I think that, for a while, we had the problem that the blame emails were not going to the person who committed the patch (only the author), but I was under the impression that was now fixed. David, did you get any breakage emails about that commit ?

[PatriosTheGreat]

Hi Pavel.

Got it,
Thanks for fixing and clarification.

In D114554#3152953, @labath wrote:

I've fixed this in rGc0e3bb4d4ba306.

Not sure if it's better to revert original commit (https://reviews.llvm.org/rGf23b829a2635a6d833bdc81a12d6217b52ae9e45) and then re-commit the change with this fix.

For a simple fix like this, it doesn't really matter what you do. Just don't leave the build broken for extended periods of time. :)

That said, it's pretty hard to do anything when you don't have commit access to the repository, so part of the responsibility also lies with the person who commits the patch for you. I think that, for a while, we had the problem that the blame emails were not going to the person who committed the patch (only the author), but I was under the impression that was now fixed. David, did you get any breakage emails about that commit ?

[DavidSpickett]

David, did you get any breakage emails about that commit ?

I did not.

[JDevlieghere]

LGTM

[PatriosTheGreat]

Closing this revision since Pavel already fixed this in rGc0e3bb4d4ba306.

lldb/test/API/python_api/file_handle/TestFileHandle.py

def test_sbstream(self):
         f = open(self.out_filename,  'w')
         stream.RedirectToFile(lldb.SBFile.Create(f, borrow=False))
         stream.Print("Frobozz")
         stream = None
         self.assertTrue(f.closed)
         with open(self.out_filename, 'r') as f:
             self.assertEqual(f.read().strip(), "Frobozz")
 
-    @skipIfReproducer # lldb::FileSP used in typemap cannot be instrumented.
     def test_set_sbstream(self):
         with open(self.out_filename, 'w') as outf:
             outsbf = lldb.SBFile(outf.fileno(), "w", False)
             status = self.dbg.SetOutputFile(outsbf)
             self.assertTrue(status.Success())
             self.dbg.SetInputString("version\nhelp\n")
 
             opts = lldb.SBCommandInterpreterRunOptions()
             self.dbg.RunCommandInterpreter(True, False, opts, 0, False, False)
             self.dbg.GetOutputFile().Flush()
 
         with open(self.out_filename, 'r') as f:
             output = f.read()
             self.assertTrue(re.search(r'Show a list of all debugger commands', output))
             self.assertTrue(re.search(r'llvm revision', output))
 No newline at end of file