Fix bug in URIPath assignment operator (#275)

Signed-off-by: Michael Carroll <[email protected]>
gazebosim · Dec 6, 2021 · f75ad3d · f75ad3d
1 parent 5d84d4a
commit f75ad3d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/URI.cc b/src/URI.cc
@@ -587,6 +587,7 @@ URIPath &URIPath::operator=(const URIPath &_path)
 {
   this->dataPtr->path = _path.dataPtr->path;
   this->dataPtr->isAbsolute = _path.dataPtr->isAbsolute;
+  this->dataPtr->trailingSlash = _path.dataPtr->trailingSlash;
   return *this;
 }
 

diff --git a/src/URI_TEST.cc b/src/URI_TEST.cc
@@ -890,8 +890,8 @@ TEST(URITEST, HasAuthority)
     // Modifyng path updates string
     uri.Path() = URIPath("new_authority.com/another/path");
 
-    EXPECT_EQ("new_authority.com/another/path/", uri.Path().Str());
-    EXPECT_EQ("https://new_authority.com/another/path/", uri.Str());
+    EXPECT_EQ("new_authority.com/another/path", uri.Path().Str());
+    EXPECT_EQ("https://new_authority.com/another/path", uri.Str());
 
     // Clearing keeps false authority
     uri.Clear();