4 ➡️ 7

Signed-off-by: Louise Poubel <[email protected]>
gazebosim · Jun 7, 2022 · f019a87 · f019a87
2 parents 21c5eb5 + 2010890
commit f019a87
Show file tree

Hide file tree

Showing 4 changed files with 43 additions and 7 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -136,7 +136,28 @@
 1. Fixed test - Download world 2
     * [Pull request 110](https://github.com/ignitionrobotics/ign-fuel-tools/pull/110)
 
-### Ignition Fuel Tools 4.X.X (202X-XX-XX)
+### Ignition Fuel Tools 4.5.0 (2022-06-01)
+
+1. APIs for retrieving models in parallel
+    * [Pull request #199](https://github.com/ignitionrobotics/ign-fuel-tools/pull/199)
+
+1. Add missing header guard for Interface.hh
+    * [Pull request #249](https://github.com/ignitionrobotics/ign-fuel-tools/pull/249)
+
+1. Print error when unzipping fails to save a file
+    * [Pull request #235](https://github.com/ignitionrobotics/ign-fuel-tools/pull/235)
+
+1. More checks when manipulating file system in tests
+    * [Pull request #227](https://github.com/ignitionrobotics/ign-fuel-tools/pull/227)
+
+1. Move test cache to build folder
+    * [Pull request #222](https://github.com/ignitionrobotics/ign-fuel-tools/pull/222)
+
+1. Improve and fix `ResultType` tests
+    * [Pull request #225](https://github.com/ignitionrobotics/ign-fuel-tools/pull/225)
+
+1. FuelClient.cc: include <deque>
+    * [Pull request #213](https://github.com/ignitionrobotics/ign-fuel-tools/pull/213)
 
 ### Ignition Fuel Tools 4.4.0 (2021-06-17)
 

diff --git a/include/ignition/fuel_tools/Interface.hh b/include/ignition/fuel_tools/Interface.hh
@@ -15,6 +15,9 @@
  *
 */
 
+#ifndef IGNITION_FUEL_TOOLS_INTERFACE_HH_
+#define IGNITION_FUEL_TOOLS_INTERFACE_HH_
+
 #include <string>
 #include "ignition/fuel_tools/Export.hh"
 #include "ignition/fuel_tools/FuelClient.hh"
@@ -60,3 +63,5 @@ namespace ignition
         const std::string &_path);
   }
 }
+
+#endif
diff --git a/src/ign_TEST.cc b/src/ign_TEST.cc
@@ -106,8 +106,9 @@ TEST(CmdLine,
 
 /////////////////////////////////////////////////
 // https://github.com/ignitionrobotics/ign-fuel-tools/issues/113
+// https://github.com/gazebosim/gz-fuel-tools/issues/254
 TEST(CmdLine,
-    IGN_UTILS_TEST_ENABLED_ONLY_ON_LINUX(ModelListCustomServerPretty))
+    DETAIL_IGN_UTILS_ADD_DISABLED_PREFIX(ModelListCustomServerPretty))
 {
   auto output = custom_exec_str(
       g_listCmd + " -t model -u https://staging-fuel.ignitionrobotics.org");
@@ -136,8 +137,9 @@ TEST(CmdLine, IGN_UTILS_TEST_ENABLED_ONLY_ON_LINUX(WorldListConfigServerUgly))
 
 /////////////////////////////////////////////////
 // https://github.com/ignitionrobotics/ign-fuel-tools/issues/113
+// https://github.com/gazebosim/gz-fuel-tools/issues/254
 TEST(CmdLine,
-    IGN_UTILS_TEST_ENABLED_ONLY_ON_LINUX(WorldListCustomServerPretty))
+    DETAIL_IGN_UTILS_ADD_DISABLED_PREFIX(WorldListCustomServerPretty))
 {
   auto output = custom_exec_str(
       g_listCmd + " -t world -u https://staging-fuel.ignitionrobotics.org");

diff --git a/src/ign_src_TEST.cc b/src/ign_src_TEST.cc
@@ -104,7 +104,9 @@ TEST_F(CmdLine, ModelListConfigServerUgly)
 /////////////////////////////////////////////////
 // Protocol "https" not supported or disabled in libcurl for Windows
 // https://github.com/ignitionrobotics/ign-fuel-tools/issues/105
-TEST_F(CmdLine, ModelListConfigServerPretty)
+// https://github.com/gazebosim/gz-fuel-tools/issues/254
+TEST_F(CmdLine,
+       DETAIL_IGN_UTILS_ADD_DISABLED_PREFIX(ModelListConfigServerPretty))
 {
   EXPECT_TRUE(listModels("https://staging-fuel.ignitionrobotics.org"));
 
@@ -126,7 +128,9 @@ TEST_F(CmdLine, ModelListConfigServerPretty)
 /////////////////////////////////////////////////
 // Protocol "https" not supported or disabled in libcurl for Windows
 // https://github.com/ignitionrobotics/ign-fuel-tools/issues/105
-TEST_F(CmdLine, ModelListConfigServerPrettyOwner)
+// https://github.com/gazebosim/gz-fuel-tools/issues/254
+TEST_F(CmdLine,
+       DETAIL_IGN_UTILS_ADD_DISABLED_PREFIX(ModelListConfigServerPrettyOwner))
 {
   EXPECT_TRUE(listModels("https://staging-fuel.ignitionrobotics.org",
       "openrobotics"));
@@ -252,7 +256,9 @@ TEST_F(CmdLine, WorldListFail)
 /////////////////////////////////////////////////
 // Protocol "https" not supported or disabled in libcurl for Windows
 // https://github.com/ignitionrobotics/ign-fuel-tools/issues/105
-TEST_F(CmdLine, WorldListConfigServerUgly)
+// https://github.com/gazebosim/gz-fuel-tools/issues/254
+TEST_F(CmdLine,
+       DETAIL_IGN_UTILS_ADD_DISABLED_PREFIX(WorldListConfigServerUgly))
 {
   EXPECT_TRUE(listWorlds(
         "https://staging-fuel.ignitionrobotics.org", "", "true"));
@@ -267,7 +273,9 @@ TEST_F(CmdLine, WorldListConfigServerUgly)
 /////////////////////////////////////////////////
 // Protocol "https" not supported or disabled in libcurl for Windows
 // https://github.com/ignitionrobotics/ign-fuel-tools/issues/105
-TEST_F(CmdLine, WorldListConfigServerPretty)
+// https://github.com/gazebosim/gz-fuel-tools/issues/254
+TEST_F(CmdLine,
+       DETAIL_IGN_UTILS_ADD_DISABLED_PREFIX(WorldListConfigServerPretty))
 {
   EXPECT_TRUE(listWorlds("https://staging-fuel.ignitionrobotics.org"));