Add debug output on failing assertions

Signed-off-by: Michael Carroll <[email protected]>
gazebosim · Feb 9, 2022 · 0038e2f · 0038e2f
1 parent 354a04a
commit 0038e2f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/src/Filesystem_TEST.cc b/src/Filesystem_TEST.cc
@@ -359,14 +359,8 @@ TEST_F(FilesystemTest, append)
   EXPECT_EQ(path, "base\\before\\after");
 #endif
 
-  // Make sure that the slashes in the middle of string are not altered.
-  // This is actually incorrect behavior
   path = joinPaths("https://fuel.ignitionrobotics.org", "/models", "box");
-#ifndef _WIN32
   EXPECT_EQ(path, "https://fuel.ignitionrobotics.org/models/box");
-#else
-  EXPECT_EQ(path, "https://fuel.ignitionrobotics.org\\models\\box");
-#endif
 }
 
 /////////////////////////////////////////////////

diff --git a/src/TempDirectory_TEST.cc b/src/TempDirectory_TEST.cc
@@ -53,13 +53,13 @@ TEST(TempDirectory, createTempDirectoryEmptyBase)
 {
   // OS TMPDIR should never be empty
   auto tmpDir = ignition::common::tempDirectoryPath();
-  ASSERT_FALSE(tmpDir.empty());
+  ASSERT_FALSE(tmpDir.empty()) << tmpDir;
 
   // Create with an empty basename (eg /tmp/XXXXXX)
   auto tmp = ignition::common::createTempDirectory("", tmpDir);
-  ASSERT_FALSE(tmp.empty());
-  EXPECT_EQ(std::string::npos, tmp.find("foo"));
-  EXPECT_NE(std::string::npos, tmp.find(tmpDir));
+  ASSERT_FALSE(tmp.empty()) << tmp;
+  EXPECT_EQ(std::string::npos, tmp.find("foo")) << tmp;
+  EXPECT_NE(std::string::npos, tmp.find(tmpDir)) << tmp;
 }
 
 /////////////////////////////////////////////////