fix: only strip optional '.git' suffix from https server remote name …

…and not 'Xgit' (peter-evans#1257)
aleksandrychev · Mar 4, 2024 · 232667d · 232667d
1 parent b5840e6
commit 232667d
Show file tree

Hide file tree

Showing 3 changed files with 1,319 additions and 1,301 deletions.
diff --git a/__test__/utils.unit.test.ts b/__test__/utils.unit.test.ts
@@ -56,6 +56,24 @@ describe('utils tests', () => {
     )
     expect(remote4.protocol).toEqual('HTTPS')
     expect(remote4.repository).toEqual('peter-evans/create-pull-request')
+
+    const remote5 = utils.getRemoteDetail(
+      'https://github.com/peter-evans/ungit'
+    )
+    expect(remote5.protocol).toEqual('HTTPS')
+    expect(remote5.repository).toEqual('peter-evans/ungit')
+
+    const remote6 = utils.getRemoteDetail(
+      'https://github.com/peter-evans/ungit.git'
+    )
+    expect(remote6.protocol).toEqual('HTTPS')
+    expect(remote6.repository).toEqual('peter-evans/ungit')
+
+    const remote7 = utils.getRemoteDetail(
+      'git@github.com:peter-evans/ungit.git'
+    )
+    expect(remote7.protocol).toEqual('SSH')
+    expect(remote7.repository).toEqual('peter-evans/ungit')
   })
 
   test('getRemoteDetail fails to parse a remote URL', async () => {