Fix path completion test case added in rL335905 on Windows

[teemperor]

Summary

The test fails because we don't rewrite the slash behind foo to the OS specific
separator (as the completion API doesn't support this kind of rewriting). However,
we assume that this part of the string is rewritten in the test case, which broke
on Windows.

[teemperor]

@stella.stamenova I don't have access to a Windows machine. Would you mind testing if this fixes the issue?

[teemperor]

• Fixed an syntax error. Should work now.

[stella.stamenova]

The test passes with the fix. Thanks!

unittests/Interpreter/TestCompletion.cpp

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(),
+  EXPECT_TRUE(ContainsExactString(Twine("~/foo/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(),
+  EXPECT_TRUE(ContainsExactString(Twine("~/foo/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);