test(@modulify/conventional-git): URL's of remotes, edge cases

Author: cmath10

packages/conventional-git/tests/Client.test.ts

@@ -268,4 +268,31 @@ describe('Client', () => {
       expect(await client.version({ prefix: 'pkg@' })).toBe('1.1.0')
     })
   })
+
+  describe('url', () => {
+    it('should return remote url', async () => {
+      exec('git remote add origin https://github.com/fork/conventional.git')
+      exec('git remote add upstream https://github.com/modulify/conventional.git')
+
+      const client = new Client({ cwd })
+
+      expect(await client.url()).toBe('https://github.com/fork/conventional.git')
+      expect(await client.url('upstream')).toBe('https://github.com/modulify/conventional.git')
+    })
+
+    it('should return empty string if remote does not exist', async () => {
+      const client = new Client({ cwd })
+
+      expect(await client.url('non-existent')).toBe('')
+    })
+
+    it('should throw other errors', async () => {
+      const client = new Client({ cwd })
+
+      // @ts-expect-error accessing private for test
+      vi.spyOn(client._git.cmd, 'exec').mockRejectedValue(new Error('Unexpected error'))
+
+      await expect(client.url()).rejects.toThrow('Unexpected error')
+    })
+  })
 })

packages/conventional-git/tests/parse.test.ts

@@ -295,5 +295,57 @@ describe('parse', () => {
       expect(commit.hash).toBe('1234567')
       expect(commit.type).toBe('feat')
     })
+
+    it('should handle notes followed by other notes', () => {
+      const commit = parse('feat: subject\n\nBREAKING CHANGE: note 1\nBREAKING CHANGE: note 2')
+
+      expect(commit.notes).toHaveLength(2)
+      expect(commit.notes[0].text).toBe('note 1')
+      expect(commit.notes[1].text).toBe('note 2')
+    })
+
+    it('should handle notes followed by references', () => {
+      const commit = parse('feat: subject\n\nBREAKING CHANGE: note 1\nCloses #123')
+      expect(commit.notes[0].text).toBe('note 1')
+      expect(commit.references[0].issue).toBe('123')
+    })
+
+    it('should handle merge commits with correspondence that are not manageable', () => {
+      const parse = createParser({
+        mergePattern: /^Merge branch '([\w-]+)'/,
+        mergeCorrespondence: ['nonManageable'],
+      })
+
+      const commit = parse('Merge branch \'feature\'\nheader')
+
+      expect(commit.meta.nonManageable).toBe('feature')
+    })
+
+    it('should handle parseReference returning null for malformed reference', () => {
+      const parse = createParser({
+        issuePrefixes: ['#'],
+      })
+
+      const commit = parse('fix: something with # but no number #abc')
+
+      expect(commit.references).toEqual([])
+    })
+
+    it('should handle parseRevert with missing correspondence fields', () => {
+      const parse = createParser({
+        revertPattern: /^Revert (.*)/,
+        revertCorrespondence: ['customField'],
+      })
+
+      const commit = parse('Revert some subject')
+
+      expect(commit.revert?.customField).toBe('some subject')
+    })
+
+    it('should handle trimLineBreaks for only line breaks', () => {
+      const commit = parse('\n\n')
+
+      expect(commit.header).toBe(null)
+    })
   })
 })