tolerate codelinks with trailing slashes

Author: tayloraswift

Sources/CodelinkTests/Main.swift

@@ -298,41 +298,49 @@ enum Main:SyncTests
             tests.expect(codelink.path.components ..? ["Sloth", "update(_:)"])
             tests.expect(codelink.hash?.value ==? .init("4ko57", radix: 36))
         }
-        if  let tests:TestGroup = tests / "codelinks" / "legacy-docc"
+        if  let tests:TestGroup = tests / "Codelinks" / "LegacyDocC"
         {
-            if  let tests:TestGroup = tests / "slashes",
+            if  let tests:TestGroup = tests / "Slashes",
                 let codelink:Codelink = .parse("Sloth/Color", for: tests)
             {
                 tests.expect(nil: codelink.filter)
                 tests.expect(nil: codelink.scope)
                 tests.expect(codelink.path.components ..? ["Sloth", "Color"])
                 tests.expect(nil: codelink.hash)
             }
-            if  let tests:TestGroup = tests / "filter",
+            if  let tests:TestGroup = tests / "TrailingSlashes",
+                let codelink:Codelink = .parse("Sloth/Color/", for: tests)
+            {
+                tests.expect(nil: codelink.filter)
+                tests.expect(nil: codelink.scope)
+                tests.expect(codelink.path.components ..? ["Sloth", "Color"])
+                tests.expect(nil: codelink.hash)
+            }
+            if  let tests:TestGroup = tests / "Filter",
                 let codelink:Codelink = .parse("Sloth/Color-swift.enum", for: tests)
             {
                 tests.expect(codelink.filter ==? .enum)
                 tests.expect(nil: codelink.scope)
                 tests.expect(codelink.path.components ..? ["Sloth", "Color"])
                 tests.expect(nil: codelink.hash)
             }
-            if  let tests:TestGroup = tests / "hash",
+            if  let tests:TestGroup = tests / "Hash",
                 let codelink:Codelink = .parse("Sloth/update(_:)-4ko57", for: tests)
             {
                 tests.expect(nil: codelink.filter)
                 tests.expect(nil: codelink.scope)
                 tests.expect(codelink.path.components ..? ["Sloth", "update(_:)"])
                 tests.expect(codelink.hash?.value ==? .init("4KO57", radix: 36))
             }
-            if  let tests:TestGroup = tests / "hash" / "minus",
+            if  let tests:TestGroup = tests / "Hash" / "Minus",
                 let codelink:Codelink = .parse("Sloth/-(_:)-4ko57", for: tests)
             {
                 tests.expect(nil: codelink.filter)
                 tests.expect(nil: codelink.scope)
                 tests.expect(codelink.path.components ..? ["Sloth", "-(_:)"])
                 tests.expect(codelink.hash?.value ==? .init("4KO57", radix: 36))
             }
-            if  let tests:TestGroup = tests / "hash" / "slinging" / "slasher",
+            if  let tests:TestGroup = tests / "Hash" / "Slinging" / "Slasher",
                 let codelink:Codelink = .parse("Sloth//(_:)-4ko57", for: tests)
             {
                 tests.expect(nil: codelink.filter)

Sources/Codelinks/Codelink.Path.swift

@@ -37,12 +37,21 @@ extension Codelink.Path
             case (_,                "-", nil):
                 format = .legacy
                 suffix = .init(.init(codepoints))
-                //  we know we already consumed all remaining input
+                //  We know we already consumed all remaining input.
                 return
 
             case (.nominal(_, nil), "/", nil):
                 format = .legacy
-                fallthrough
+
+                //  Tolerate trailing slash.
+                if  codepoints.isEmpty
+                {
+                    return
+                }
+                else
+                {
+                    fallthrough
+                }
 
             case (.nominal(_, nil), ".", _):
                 if  let next:Component = .init(parsing: &codepoints)