[lldb] [testsuite] TestReproducerAttach.py: Fix dependency on external symbol files

[jankratochvil]

Summary

D55859 and D63339 prevented needless dependencies on system symbol files.
This testcase was checked-in afterwards and it brings back one such unwanted dependency.
Under some circumstances it may cause false FAILs and/or excessive resource usage to run the testcase.

clang-format does not support .py so I have formatted it as I found most compatible.

Also this is not a full testcase-style initialization, for example --no-lldbinit ignores env("NO_LLDBINIT") setting which lldbtest.py does implement:

# If we spawn an lldb process for test (via pexpect), do not load the
# init file unless told otherwise.
if os.environ.get("NO_LLDBINIT") != "NO":
    self.lldbOption += " --no-lldbinit"

But this is what lldbpexpect.py does - it also ignores env("NO_LLDBINIT"). Sure one could also fix lldbpexpect.py to unify the initialization more with lldbtest.py but I find that outside of the scope of this patch.

[JDevlieghere]

Thanks Jan, LGTM.

For now this appears to be the only test doing this, but if we end up having more of those we should probably abstract that away behind a helper that gives you an array with the executable and the setup commands.

lldb/test/API/functionalities/reproducers/attach/TestReproducerAttach.py

def test_reproducer_attach(self):
         inferior = self.spawnSubprocess(self.getBuildArtifact(exe), [token])
         pid = inferior.pid
 
         lldbutil.wait_for_file_on_target(self, token)
 
         # Use Popen because pexpect is overkill and spawnSubprocess is
         # asynchronous.
         capture = subprocess.Popen([
-            lldbtest_config.lldbExec, '-b', '--capture', '--capture-path',
-            reproducer, '-o', 'proc att -n {}'.format(exe), '-o',
-            'reproducer generate'
+            lldbtest_config.lldbExec, '-b', '--no-lldbinit', '--no-use-colors']
+            + sum(map(lambda x: ['-O', x], self.setUpCommands()), [])
+            + ['--capture', '--capture-path', reproducer,
+            '-o', 'proc att -n {}'.format(exe), '-o', 'reproducer generate'
         ],
                                    stdin=subprocess.PIPE,
                                    stdout=subprocess.PIPE,
                                    stderr=subprocess.PIPE)
         outs, _ = capture.communicate()
         outs = outs.decode('utf-8')
         self.assertIn('Process {} stopped'.format(pid), outs)
         self.assertIn('Reproducer written', outs)