Keep a trailing slash in URL.joinpath

Issue: #862

CHANGES/862.bugfix.rst

@@ -0,0 +1 @@
+Stopped dropping trailing slashes in ``URL.joinpath``.

tests/test_url.py

@@ -797,6 +797,21 @@ def test_div_with_dots():
             "http://example.com/path/to",
             id="cleanup-query-and-fragment",
         ),
+        pytest.param("", ("path/",), "http://example.com/path/", id="trailing-slash"),
+        pytest.param(
+            "", ("path/", "to/"), "http://example.com/path/to/", id="duplicate-slash"
+        ),
+        pytest.param("", (), "http://example.com", id="empty-segments"),
+        pytest.param(
+            "/", ("path/",), "http://example.com/path/", id="base-slash-trailing-slash"
+        ),
+        pytest.param(
+            "/",
+            ("path/", "to/"),
+            "http://example.com/path/to/",
+            id="base-slash-duplicate-slash",
+        ),
+        pytest.param("/", (), "http://example.com", id="base-slash-empty-segments"),
     ],
 )
 def test_joinpath(base, to_join, expected):
@@ -811,6 +826,9 @@ def test_joinpath(base, to_join, expected):
         pytest.param(URL("a"), ("b",), ("a", "b"), id="relative-path"),
         pytest.param(URL("a"), ("b", "", "c"), ("a", "b", "c"), id="empty-element"),
         pytest.param(URL("/a"), ("b"), ("/", "a", "b"), id="absolute-path"),
+        pytest.param(URL(), ("a/",), ("a", ""), id="trailing-slash"),
+        pytest.param(URL(), ("a/", "b/"), ("a", "b", ""), id="duplicate-slash"),
+        pytest.param(URL(), (), ("",), id="empty-segments"),
     ],
 )
 def test_joinpath_relative(url, to_join, expected):

yarl/_url.py

@@ -713,6 +713,7 @@ def _validate_authority_uri_abs_path(host, path):
 
     def _make_child(self, segments, encoded=False):
         """add segments to self._val.path, accounting for absolute vs relative paths"""
+        need_trailing_slash = segments and segments[-1].endswith("/")
         parsed = []
         for seg in reversed(segments):
             if not seg:
@@ -729,6 +730,8 @@ def _make_child(self, segments, encoded=False):
             elif seg != ".":
                 parsed.append(seg)
         parsed.reverse()
+        if need_trailing_slash:
+            parsed.append("")
         old_path = self._val.path
         if old_path:
             parsed = [*old_path.rstrip("/").split("/"), *parsed]