FuelClient.cc: include <deque> (#213)

* FuelClient.cc: include <deque> Fixes the Windows build. Signed-off-by: Steve Peters <scpeters@openrobotics.org> * ign_src_TEST: print error messages Signed-off-by: Steve Peters <scpeters@openrobotics.org> * print one more error message Signed-off-by: Steve Peters <scpeters@openrobotics.org> * ign.cc: remove console error message This was introduced in #199. Signed-off-by: Steve Peters <scpeters@openrobotics.org>
gazebosim · Nov 8, 2021 · 4edeed5 · 4edeed5
1 parent c680ee8
commit 4edeed5
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/FuelClient.cc b/src/FuelClient.cc
@@ -25,6 +25,7 @@
 #endif
 
 #include <algorithm>
+#include <deque>
 #include <iomanip>
 #include <iostream>
 #include <memory>

diff --git a/src/ign.cc b/src/ign.cc
@@ -638,9 +638,6 @@ extern "C" IGNITION_FUEL_TOOLS_VISIBLE int downloadUrl(const char *_url,
     if (downloadWorlds)
     {
       auto result = client.DownloadWorlds(worldIds, _jobs);
-      ignerr << "Failed to download worlds for collection ["
-          << collection.Name()
-          << "]" << std::endl;
     }
   }
   else

diff --git a/src/ign_src_TEST.cc b/src/ign_src_TEST.cc
@@ -492,7 +492,8 @@ TEST_P(DownloadCollectionTest, AllItems)
   // Check output
   EXPECT_NE(stdOutBuffer.str().find("Download succeeded"), std::string::npos)
       << stdOutBuffer.str();
-  EXPECT_TRUE(stdErrBuffer.str().empty());
+  EXPECT_TRUE(stdErrBuffer.str().empty())
+      << stdErrBuffer.str();
 
   // Check files
   // Model: Backpack
@@ -630,7 +631,8 @@ TEST_P(DownloadCollectionTest, Worlds)
   // Check output
   EXPECT_NE(stdOutBuffer.str().find("Download succeeded"), std::string::npos)
       << stdOutBuffer.str();
-  EXPECT_TRUE(stdErrBuffer.str().empty());
+  EXPECT_TRUE(stdErrBuffer.str().empty())
+      << stdErrBuffer.str();
 
   // Check files
   // Model: Backpack