Merge pull request #8 from storyblok/EXT-1526-trim-slashes-always-ret…

…urn-string

refactor: trimSlashes returns string

src/storyblok-auth-api/refreshToken.ts

@@ -1,6 +1,5 @@
 import { token_endpoint } from './storyblok-oauth-api-endpoints'
-import { hasKey } from '../utils'
-import { trimSlashes } from '../utils/trimSlashes/trimSlashes'
+import { hasKey, trimSlashes } from '../utils'
 import { AuthHandlerParams } from './AuthHandlerParams'
 
 export type RefreshTokenWithFetchParams = Pick<

src/utils/index.ts

@@ -3,3 +3,4 @@ export * from './hasKey'
 export * from './GetCookie'
 export * from './SetCookie'
 export * from './numberFromString'
+export * from './trimSlashes'

src/utils/trimSlashes/index.ts

@@ -0,0 +1 @@
+export * from './trimSlashes'

src/utils/trimSlashes/trimSlashes.test.ts

@@ -15,4 +15,13 @@ describe('trimSlashes', () => {
       'hello/my/fellow/bloker',
     )
   })
+  it('should trim double slashes', () => {
+    expect(trimSlashes('//hello//')).toEqual('hello')
+  })
+  it('should trim a slash character to an empty string', () => {
+    expect(trimSlashes('/')).toEqual('')
+  })
+  it('should trim multiple slash characters to an empty string', () => {
+    expect(trimSlashes('//')).toEqual('')
+  })
 })

src/utils/trimSlashes/trimSlashes.ts

@@ -1,2 +1,6 @@
-export const trimSlashes = (slugs: string): string | undefined =>
-  slugs.match(/^\/*(.*?)\/*$/)?.[1] ?? undefined
+/**
+ * Removes leading and trailing slashes (`/`) from a string.
+ * @param slugs
+ */
+export const trimSlashes = (slugs: string): string =>
+  slugs.match(/^\/*(.*?)\/*$/)?.[1] ?? ''