chore: remove unused trailing slash utility (#3280)

nuxt-modules · Dec 24, 2024 · 4d93e07 · 4d93e07
1 parent 1262163
commit 4d93e07
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 52 deletions.
diff --git a/src/runtime/routing/utils.ts b/src/runtime/routing/utils.ts
@@ -18,14 +18,6 @@ export function getNormalizedLocales(locales: Locale[] | LocaleObject[]): Locale
   return normalized
 }
 
-export function adjustRoutePathForTrailingSlash(
-  pagePath: string,
-  trailingSlash: boolean,
-  isChildWithRelativePath: boolean
-) {
-  return pagePath.replace(/\/+$/, '') + (trailingSlash ? '/' : '') || (isChildWithRelativePath ? '' : '/')
-}
-
 export function getRouteName(routeName?: string | symbol | null) {
   if (isString(routeName)) return routeName
   if (isSymbol(routeName)) return routeName.toString()

diff --git a/test/routing-utils.test.ts b/test/routing-utils.test.ts
@@ -1,50 +1,6 @@
 import { describe, it, assert, test } from 'vitest'
 import * as utils from '../src/runtime/routing/utils'
 
-describe('adjustRouteDefinitionForTrailingSlash', function () {
-  describe('pagePath: /foo/bar', function () {
-    describe('trailingSlash: false, isChildWithRelativePath: true', function () {
-      it('should be trailed with slash: /foo/bar/', function () {
-        assert.equal(utils.adjustRoutePathForTrailingSlash('/foo/bar', true, true), '/foo/bar/')
-      })
-    })
-
-    describe('trailingSlash: false, isChildWithRelativePath: true', function () {
-      it('should not be trailed with slash: /foo/bar/', function () {
-        assert.equal(utils.adjustRoutePathForTrailingSlash('/foo/bar', false, true), '/foo/bar')
-      })
-    })
-
-    describe('trailingSlash: false, isChildWithRelativePath: false', function () {
-      it('should be trailed with slash: /foo/bar/', function () {
-        assert.equal(utils.adjustRoutePathForTrailingSlash('/foo/bar', true, false), '/foo/bar/')
-      })
-    })
-
-    describe('trailingSlash: false, isChildWithRelativePath: false', function () {
-      it('should not be trailed with slash: /foo/bar/', function () {
-        assert.equal(utils.adjustRoutePathForTrailingSlash('/foo/bar', false, false), '/foo/bar')
-      })
-    })
-  })
-
-  describe('pagePath: /', function () {
-    describe('trailingSlash: false, isChildWithRelativePath: true', function () {
-      it('should not be trailed with slash: empty', function () {
-        assert.equal(utils.adjustRoutePathForTrailingSlash('/', false, true), '')
-      })
-    })
-  })
-
-  describe('pagePath: empty', function () {
-    describe('trailingSlash: true, isChildWithRelativePath: true', function () {
-      it('should not be trailed with slash: /', function () {
-        assert.equal(utils.adjustRoutePathForTrailingSlash('', true, true), '/')
-      })
-    })
-  })
-})
-
 describe('getLocaleRouteName', () => {
   describe('strategy: prefix_and_default', () => {
     it('should be `route1___en___default`', () => {