test: reorganise npm unit tests (#876)

Author: 43081j

test/unit/app/utils/npm.spec.ts

@@ -0,0 +1,22 @@
+import { describe, expect, it } from 'vitest'
+
+import { buildScopeTeam } from '../../../../../app/utils/npm/common'
+import { validateScopeTeam } from '../../../../../cli/src/npm-client'
+
+describe('buildScopeTeam', () => {
+  it('constructs scope:team with @ prefix', () => {
+    expect(buildScopeTeam('netlify', 'developers')).toBe('@netlify:developers')
+    expect(buildScopeTeam('nuxt', 'core')).toBe('@nuxt:core')
+  })
+
+  it('strips existing @ prefix from orgName', () => {
+    expect(buildScopeTeam('@netlify', 'developers')).toBe('@netlify:developers')
+    expect(buildScopeTeam('@nuxt', 'core')).toBe('@nuxt:core')
+  })
+
+  it('produces format accepted by validateScopeTeam', () => {
+    expect(() => validateScopeTeam(buildScopeTeam('netlify', 'developers'))).not.toThrow()
+    expect(() => validateScopeTeam(buildScopeTeam('nuxt', 'core'))).not.toThrow()
+    expect(() => validateScopeTeam(buildScopeTeam('my-org', 'my-team'))).not.toThrow()
+  })
+})

test/unit/app/utils/npm/common.spec.ts

@@ -25,19 +25,25 @@ describe('spdx utilities', () => {
   })
 
   describe('getSpdxLicenseUrl', () => {
-    it('returns URL for valid SPDX licenses', () => {
+    it('returns URL for valid license identifiers', () => {
       expect(getSpdxLicenseUrl('MIT')).toBe('https://spdx.org/licenses/MIT.html')
+      expect(getSpdxLicenseUrl('ISC')).toBe('https://spdx.org/licenses/ISC.html')
       expect(getSpdxLicenseUrl('Apache-2.0')).toBe('https://spdx.org/licenses/Apache-2.0.html')
+      expect(getSpdxLicenseUrl('GPL-3.0-only')).toBe('https://spdx.org/licenses/GPL-3.0-only.html')
+      expect(getSpdxLicenseUrl('BSD-2-Clause')).toBe('https://spdx.org/licenses/BSD-2-Clause.html')
+      expect(getSpdxLicenseUrl('GPL-3.0+')).toBe('https://spdx.org/licenses/GPL-3.0+.html')
     })
 
     it('returns null for invalid licenses', () => {
       expect(getSpdxLicenseUrl('CustomLicense')).toBeNull()
       expect(getSpdxLicenseUrl('INVALID')).toBeNull()
+      expect(getSpdxLicenseUrl('MIT OR Apache-2.0')).toBeNull()
     })
 
     it('returns null for undefined or empty', () => {
       expect(getSpdxLicenseUrl(undefined)).toBeNull()
       expect(getSpdxLicenseUrl('')).toBeNull()
+      expect(getSpdxLicenseUrl('   ')).toBeNull()
     })
 
     it('trims whitespace', () => {