Clean up test suite by removing duplicates and improving clarity

- Consolidate 3 redundant URL authority tests into 1 comprehensive test
- Rename ambiguous test descriptions for better clarity
- Reduce test count from 783 to 781 with no coverage loss
- Maintain 100% code coverage across all metrics

Author: jdalton

test/json-export.test.mts

@@ -240,7 +240,7 @@ describe('PackageURL JSON/dict export', () => {
       expect(partialPurl.subpath).toBeUndefined()
     })
 
-    it('should validate input and throw appropriate errors', () => {
+    it('should validate object input and throw appropriate errors', () => {
       // Test non-object inputs
       expect(() => PackageURL.fromObject('not an object')).toThrow(
         'Object argument is required.',
@@ -290,7 +290,7 @@ describe('PackageURL JSON/dict export', () => {
       expect(completePurl.subpath).toBe('lib/fs.d.ts')
     })
 
-    it('should validate input and throw appropriate errors', () => {
+    it('should validate JSON string input and throw appropriate errors', () => {
       // Test non-string inputs
       expect(() => PackageURL.fromJSON(123)).toThrow(
         'JSON string argument is required.',

test/package-url.test.mts

@@ -921,42 +921,31 @@ describe('PackageURL', () => {
         )
       })
 
-      it('should reject URLs with authority components', () => {
+      it('should reject all URL authority component combinations', () => {
+        // Full credentials with host and port
         expect(() =>
           PackageURL.fromString('pkg://user:pass@host:8080/type/name'),
         ).toThrow(/cannot contain a "user:pass@host:port"/)
+
+        // Username with host
         expect(() =>
           PackageURL.fromString('pkg://user@host/type/name'),
         ).toThrow(/cannot contain a "user:pass@host:port"/)
-      })
 
-      it('should reject URLs with password auth component', () => {
-        // Test password-only auth (empty username, non-empty password)
+        // Password only (empty username, non-empty password)
         expect(() =>
           PackageURL.fromString('pkg://:password@type/name'),
         ).toThrow(/cannot contain a "user:pass@host:port"/)
-      })
 
-      it('should reject all combinations of username and password auth', () => {
-        // Test username only (already covered but for completeness)
+        // Username only
         expect(() => PackageURL.fromString('pkg://user@type/name')).toThrow(
           /cannot contain a "user:pass@host:port"/,
         )
 
-        // Test password only (already covered but for completeness)
-        expect(() =>
-          PackageURL.fromString('pkg://:password@type/name'),
-        ).toThrow(/cannot contain a "user:pass@host:port"/)
-
-        // Test both username and password (full auth credentials)
+        // Full credentials without host
         expect(() =>
           PackageURL.fromString('pkg://user:password@type/name'),
         ).toThrow(/cannot contain a "user:pass@host:port"/)
-
-        // Test with host component too (complete authority format)
-        expect(() =>
-          PackageURL.fromString('pkg://user:pass@host:8080/type/name'),
-        ).toThrow(/cannot contain a "user:pass@host:port"/)
       })
 
       it('should handle URL parsing failures gracefully', () => {

test/result.test.mts

@@ -300,7 +300,7 @@ describe('PackageURL Result methods', () => {
   })
 
   describe('tryParseString', () => {
-    it('should return Ok for valid purl string', () => {
+    it('should return Ok with parsed components for valid purl string', () => {
       const result = PackageURL.tryParseString('pkg:npm/[email protected]')
 
       expect(result.isOk()).toBe(true)
@@ -314,7 +314,7 @@ describe('PackageURL Result methods', () => {
       expect(components[3]).toBe('4.17.21')
     })
 
-    it('should return Err for invalid purl string', () => {
+    it('should return Err when parsing invalid purl string', () => {
       const result = PackageURL.tryParseString('invalid-purl')
 
       expect(result.isErr()).toBe(true)