fix: remove colons from depNameSanitized (#33672)

renovatebot · Jan 17, 2025 · 3e74602 · 3e74602
1 parent 6964458
commit 3e74602
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/lib/workers/repository/updates/flatten.spec.ts b/lib/workers/repository/updates/flatten.spec.ts
@@ -1,7 +1,7 @@
 import is from '@sindresorhus/is';
 import type { RenovateConfig } from '../../../../test/util';
 import { getConfig } from '../../../config/defaults';
-import { flattenUpdates } from './flatten';
+import { flattenUpdates, sanitizeDepName } from './flatten';
 
 jest.mock('../../../util/git/semantic');
 
@@ -14,6 +14,14 @@ beforeEach(() => {
 });
 
 describe('workers/repository/updates/flatten', () => {
+  describe('sanitizeDepName()', () => {
+    it('sanitizes urls', () => {
+      expect(sanitizeDepName('https://some.host.name/a/path/to.git')).toBe(
+        'https-some.host.name-a-path-to.git',
+      );
+    });
+  });
+
   describe('flattenUpdates()', () => {
     it('flattens', async () => {
       // TODO #22198

diff --git a/lib/workers/repository/updates/flatten.ts b/lib/workers/repository/updates/flatten.ts
@@ -17,12 +17,13 @@ import { generateBranchName } from './branch-name';
 const upper = (str: string): string =>
   str.charAt(0).toUpperCase() + str.substring(1);
 
-function sanitizeDepName(depName: string): string {
+export function sanitizeDepName(depName: string): string {
   return depName
     .replace('@types/', '')
     .replace('@', '')
     .replace(regEx(/\//g), '-')
     .replace(regEx(/\s+/g), '-')
+    .replace(regEx(/:/g), '-')
     .replace(regEx(/-+/), '-')
     .toLowerCase();
 }