Added test case for: r334978 - Fixed file completion for paths that start with '~'

[labath]

Looks good. Thank you.

[stella.stamenova]

This test fails on Windows with the Visual Studio generator.

##[error]llvm\tools\lldb\unittests\interpreter\testcompletion.cpp(291,0): Error : Value of: ContainsExactString(Twine("~/foo") + path::get_separator() + "nested" + path::get_separator(), Results)
e:\_work\1\s\llvm\tools\lldb\unittests\interpreter\testcompletion.cpp(291): error : Value of: ContainsExactString(Twine("~/foo") + path::get_separator() + "nested" + path::get_separator(), Results) [E:\_work\1\b\LLVMBuild\tools\lldb\lit\check-lldb-lit.vcxproj]
          
            Actual: false
          
          Expected: true
          
 ##[error]llvm\tools\lldb\unittests\interpreter\testcompletion.cpp(297,0): Error : Value of: ContainsExactString(Twine("~/foo") + path::get_separator() + "nested" + path::get_separator(), Results)
e:\_work\1\s\llvm\tools\lldb\unittests\interpreter\testcompletion.cpp(297): error : Value of: ContainsExactString(Twine("~/foo") + path::get_separator() + "nested" + path::get_separator(), Results) [E:\_work\1\b\LLVMBuild\tools\lldb\lit\check-lldb-lit.vcxproj]
          
            Actual: false
          
          Expected: true
          
          [  FAILED  ] CompletionTest.DirCompletionUsername (1 ms)
          
          [----------] 1 test from CompletionTest (1 ms total)

lldb/trunk/unittests/Interpreter/TestCompletion.cpp

protected:
 
   static SmallString<128> DirFoo;
   static SmallString<128> DirFooA;
   static SmallString<128> DirFooB;
   static SmallString<128> DirFooC;
   static SmallString<128> DirBar;
   static SmallString<128> DirBaz;
   static SmallString<128> DirTestFolder;
+  static SmallString<128> DirNested;
 
   static SmallString<128> FileAA;
   static SmallString<128> FileAB;
   static SmallString<128> FileAC;
   static SmallString<128> FileFoo;
   static SmallString<128> FileBar;
   static SmallString<128> FileBaz;
 
   void SetUp() override { llvm::sys::fs::set_current_path(OriginalWorkingDir); }
 
   static void SetUpTestCase() {
     llvm::sys::fs::current_path(OriginalWorkingDir);
 
     ASSERT_NO_ERROR(fs::createUniqueDirectory("FsCompletion", BaseDir));
-    const char *DirNames[] = {"foo", "fooa", "foob",       "fooc",
-                              "bar", "baz",  "test_folder"};
+    const char *DirNames[] = {"foo", "fooa", "foob",        "fooc",
+                              "bar", "baz",  "test_folder", "foo/nested"};
     const char *FileNames[] = {"aa1234.tmp",  "ab1234.tmp",  "ac1234.tmp",
                                "foo1234.tmp", "bar1234.tmp", "baz1234.tmp"};
-    SmallString<128> *Dirs[] = {&DirFoo, &DirFooA, &DirFooB,      &DirFooC,
-                                &DirBar, &DirBaz,  &DirTestFolder};
+    SmallString<128> *Dirs[] = {&DirFoo, &DirFooA, &DirFooB,       &DirFooC,
+                                &DirBar, &DirBaz,  &DirTestFolder, &DirNested};
     for (auto Dir : llvm::zip(DirNames, Dirs)) {
       auto &Path = *std::get<1>(Dir);
       Path = BaseDir;
       path::append(Path, std::get<0>(Dir));
-      ASSERT_NO_ERROR(fs::create_directory(Path));
+      ASSERT_NO_ERROR(fs::create_directories(Path));
     }
 
     SmallString<128> *Files[] = {&FileAA,  &FileAB,  &FileAC,
                                  &FileFoo, &FileBar, &FileBaz};
     for (auto File : llvm::zip(FileNames, Files)) {
       auto &Path = *std::get<1>(File);
       Path = BaseDir;
       path::append(Path, std::get<0>(File));

 
 SmallString<128> CompletionTest::DirFoo;
 SmallString<128> CompletionTest::DirFooA;
 SmallString<128> CompletionTest::DirFooB;
 SmallString<128> CompletionTest::DirFooC;
 SmallString<128> CompletionTest::DirBar;
 SmallString<128> CompletionTest::DirBaz;
 SmallString<128> CompletionTest::DirTestFolder;
+SmallString<128> CompletionTest::DirNested;
 
 SmallString<128> CompletionTest::FileAA;
 SmallString<128> CompletionTest::FileAB;
 SmallString<128> CompletionTest::FileAC;
 SmallString<128> CompletionTest::FileFoo;
 SmallString<128> CompletionTest::FileBar;
 SmallString<128> CompletionTest::FileBaz;
 }
EXPECT_TRUE(
       ContainsExactString(Twine("~/fooc") + path::get_separator(), Results));
   EXPECT_TRUE(
       ContainsExactString(Twine("~/bar") + path::get_separator(), Results));
   EXPECT_TRUE(
       ContainsExactString(Twine("~/baz") + path::get_separator(), Results));
   EXPECT_TRUE(ContainsExactString(
       Twine("~/test_folder") + path::get_separator(), Results));
 
+  // Check that we can complete directories in nested paths
+  Count = CommandCompletions::DiskDirectories("~/foo/", Results, Resolver);
+  ASSERT_EQ(1u, Count);
+  ASSERT_EQ(Count, Results.GetSize());
+  EXPECT_TRUE(ContainsExactString(Twine("~/foo") + path::get_separator() +
+                                      "nested" + path::get_separator(),
+                                  Results));
+  Count = CommandCompletions::DiskDirectories("~/foo/nes", Results, Resolver);
+  ASSERT_EQ(1u, Count);
+  ASSERT_EQ(Count, Results.GetSize());
+  EXPECT_TRUE(ContainsExactString(Twine("~/foo") + path::get_separator() +
+                                      "nested" + path::get_separator(),
+                                  Results));
+
   // With ~username syntax it should return one match if there is an exact
   // match.
   // It shouldn't translate to the actual directory, it should keep the form the
   // user typed.
   Count = CommandCompletions::DiskDirectories("~Lars", Results, Resolver);
   ASSERT_EQ(1u, Count);
   ASSERT_EQ(Count, Results.GetSize());
   EXPECT_TRUE(