diff options
author | Jonas Devlieghere <jonas@devlieghere.com> | 2019-09-27 19:12:18 +0000 |
---|---|---|
committer | Jonas Devlieghere <jonas@devlieghere.com> | 2019-09-27 19:12:18 +0000 |
commit | 097ba7598455e1f1f8126979eeb96ed30db9ae04 (patch) | |
tree | 47ba03754f24b97849ebbe89353eafb3aef05bb1 | |
parent | 2605f8c4616e6c40d6d5d3fecdde1a9b4253719a (diff) | |
download | bcm5719-llvm-097ba7598455e1f1f8126979eeb96ed30db9ae04.tar.gz bcm5719-llvm-097ba7598455e1f1f8126979eeb96ed30db9ae04.zip |
[Reproducer] Update the unit tests to specify the path style.
The unit tests started failing on Windows after my recent patch that
ensured we always deal with absolute paths. This should fix that.
llvm-svn: 373114
-rw-r--r-- | lldb/unittests/Utility/ReproducerTest.cpp | 50 |
1 files changed, 33 insertions, 17 deletions
diff --git a/lldb/unittests/Utility/ReproducerTest.cpp b/lldb/unittests/Utility/ReproducerTest.cpp index a89812a03cb..91407cc0502 100644 --- a/lldb/unittests/Utility/ReproducerTest.cpp +++ b/lldb/unittests/Utility/ReproducerTest.cpp @@ -52,14 +52,19 @@ TEST(ReproducerTest, SetCapture) { EXPECT_EQ(nullptr, reproducer.GetLoader()); // Enable capture and check that means we have a generator. - EXPECT_THAT_ERROR(reproducer.SetCapture(FileSpec("/bogus/path")), - Succeeded()); + EXPECT_THAT_ERROR( + reproducer.SetCapture(FileSpec("/bogus/path", FileSpec::Style::posix)), + Succeeded()); EXPECT_NE(nullptr, reproducer.GetGenerator()); - EXPECT_EQ(FileSpec("/bogus/path"), reproducer.GetGenerator()->GetRoot()); - EXPECT_EQ(FileSpec("/bogus/path"), reproducer.GetReproducerPath()); + EXPECT_EQ(FileSpec("/bogus/path", FileSpec::Style::posix), + reproducer.GetGenerator()->GetRoot()); + EXPECT_EQ(FileSpec("/bogus/path", FileSpec::Style::posix), + reproducer.GetReproducerPath()); // Ensure that we cannot enable replay. - EXPECT_THAT_ERROR(reproducer.SetReplay(FileSpec("/bogus/path")), Failed()); + EXPECT_THAT_ERROR( + reproducer.SetReplay(FileSpec("/bogus/path", FileSpec::Style::posix)), + Failed()); EXPECT_EQ(nullptr, reproducer.GetLoader()); // Ensure we can disable the generator again. @@ -76,36 +81,45 @@ TEST(ReproducerTest, SetReplay) { EXPECT_EQ(nullptr, reproducer.GetLoader()); // Expected to fail because we can't load the index. - EXPECT_THAT_ERROR(reproducer.SetReplay(FileSpec("/bogus/path")), Failed()); + EXPECT_THAT_ERROR( + reproducer.SetReplay(FileSpec("/bogus/path", FileSpec::Style::posix)), + Failed()); // However the loader should still be set, which we check here. EXPECT_NE(nullptr, reproducer.GetLoader()); // Make sure the bogus path is correctly set. - EXPECT_EQ(FileSpec("/bogus/path"), reproducer.GetLoader()->GetRoot()); - EXPECT_EQ(FileSpec("/bogus/path"), reproducer.GetReproducerPath()); + EXPECT_EQ(FileSpec("/bogus/path", FileSpec::Style::posix), + reproducer.GetLoader()->GetRoot()); + EXPECT_EQ(FileSpec("/bogus/path", FileSpec::Style::posix), + reproducer.GetReproducerPath()); // Ensure that we cannot enable replay. - EXPECT_THAT_ERROR(reproducer.SetCapture(FileSpec("/bogus/path")), Failed()); + EXPECT_THAT_ERROR( + reproducer.SetCapture(FileSpec("/bogus/path", FileSpec::Style::posix)), + Failed()); EXPECT_EQ(nullptr, reproducer.GetGenerator()); } TEST(GeneratorTest, Create) { DummyReproducer reproducer; - EXPECT_THAT_ERROR(reproducer.SetCapture(FileSpec("/bogus/path")), - Succeeded()); + EXPECT_THAT_ERROR( + reproducer.SetCapture(FileSpec("/bogus/path", FileSpec::Style::posix)), + Succeeded()); auto &generator = *reproducer.GetGenerator(); auto *provider = generator.Create<DummyProvider>(); EXPECT_NE(nullptr, provider); - EXPECT_EQ(FileSpec("/bogus/path"), provider->GetRoot()); + EXPECT_EQ(FileSpec("/bogus/path", FileSpec::Style::posix), + provider->GetRoot()); } TEST(GeneratorTest, Get) { DummyReproducer reproducer; - EXPECT_THAT_ERROR(reproducer.SetCapture(FileSpec("/bogus/path")), - Succeeded()); + EXPECT_THAT_ERROR( + reproducer.SetCapture(FileSpec("/bogus/path", FileSpec::Style::posix)), + Succeeded()); auto &generator = *reproducer.GetGenerator(); auto *provider = generator.Create<DummyProvider>(); @@ -118,12 +132,14 @@ TEST(GeneratorTest, Get) { TEST(GeneratorTest, GetOrCreate) { DummyReproducer reproducer; - EXPECT_THAT_ERROR(reproducer.SetCapture(FileSpec("/bogus/path")), - Succeeded()); + EXPECT_THAT_ERROR( + reproducer.SetCapture(FileSpec("/bogus/path", FileSpec::Style::posix)), + Succeeded()); auto &generator = *reproducer.GetGenerator(); auto &provider = generator.GetOrCreate<DummyProvider>(); - EXPECT_EQ(FileSpec("/bogus/path"), provider.GetRoot()); + EXPECT_EQ(FileSpec("/bogus/path", FileSpec::Style::posix), + provider.GetRoot()); auto &provider_alt = generator.GetOrCreate<DummyProvider>(); EXPECT_EQ(&provider, &provider_alt); |