[lldb-server unittest] Add missing teardown logic

[aadsm]

Summary

This test base class is missing the teardown making the second set of tests extending it to fail in an assertion in the FileSystem::Initialize() (as it's being initialized twice).
Not sure why this isn't failing the build bots.. (unless they're running without asserts?).

With this fix ninja LLDBServerTests && ./tools/lldb/unittests/tools/lldb-server/tests/LLDBServerTests successfully runs and passes all tests.

[xiaobai]

Thanks for fixing this.

[labath]

Not sure why this isn't failing the build bots.. (unless they're running without asserts?).

The build bots (and lit, in general) runs each of the unit tests in isolation (so it first gets a list of all tests, then runs the executable with --gtest_filter=TestName for each of the tests in turn). This means issues like this don't show up there.

However, it's good to fix them nonetheless.

[labath]

I should add (since I'm guessing you found this problem while trying to add unit tests for your lldb-server changes) a bit of history about these tests. There are two kinds of tests for lldb-server. The python tests (test/testcases/tools/lldb-server) are older ones. The c++ ones were added in an attempt to fix the issues which made the python tests hard to understand. However, the person who was doing that left the project, so the were never quite finished. In fact, I've been lately considering just removing them and instead to try to fix the python issues incrementally in-place.

I don't think this has any immediate impact on what you're doing now -- feel free to use whichever method you find the easiest to write the tests you need. But if, after doing that, you have any feedback about one test method or the other, I'd very much like to hear it.

lldb/unittests/tools/lldb-server/tests/TestBase.h

 
 class TestBase: public ::testing::Test {
 public:
   static void SetUpTestCase() {
     lldb_private::FileSystem::Initialize();
     lldb_private::HostInfo::Initialize();
   }
 
+  static void TearDownTestCase() {
+    lldb_private::HostInfo::Terminate();
+    lldb_private::FileSystem::Terminate();
+  }
+
   static std::string getInferiorPath(llvm::StringRef Name) {
     llvm::SmallString<64> Path(LLDB_TEST_INFERIOR_PATH);
     llvm::sys::path::append(Path, Name + LLDB_TEST_INFERIOR_SUFFIX);
     return Path.str();
   }
 
   static std::string getLogFileName();
 };