fix behavior of slashJoin if only one path segment given

simbo · Feb 1, 2021 · 2eebeb5 · 2eebeb5
1 parent 2297f63
commit 2eebeb5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/path-slashes.test.ts b/src/path-slashes.test.ts
@@ -73,6 +73,9 @@ describe('withoutSlashes', () => {
 describe('slashJoin', () => {
   it('should join path parts and add slashes if necessary', () => {
     expect(slashJoin('foo/', '/bar', 'baz', '/boom')).toBe('foo/bar/baz/boom');
+    expect(slashJoin('foo/')).toBe('foo/');
+    expect(slashJoin('/foo/')).toBe('/foo/');
+    expect(slashJoin('/foo')).toBe('/foo');
     expect(slashJoin('/foo/', '/bar', 'baz', '/boom')).toBe('/foo/bar/baz/boom');
     expect(slashJoin(['foo/', '/bar', 'baz', '/boom'])).toBe('foo/bar/baz/boom');
     expect(slashJoin(['foo/', '/bar', 'baz', '/boom/'])).toBe('foo/bar/baz/boom/');

diff --git a/src/path-slashes.ts b/src/path-slashes.ts
@@ -70,7 +70,7 @@ export function slashJoin(...strings: (string | string[])[]): string {
   return parts
     .map((part, i) => {
       if (i === 0) {
-        return withoutTrailingSlash(part);
+        return parts.length === 1 ? part : withoutTrailingSlash(part);
       } else if (i === parts.length - 1) {
         return withoutLeadingSlash(part);
       } else {