Update test imports to use dist directory

Author: jdalton

test/benchmark.test.mts

@@ -6,7 +6,7 @@ import { describe, expect, it } from 'vitest'
 import { readJson } from '@socketsecurity/registry/lib/fs'
 import { isObject } from '@socketsecurity/registry/lib/objects'
 
-import { PackageURL } from '../src/package-url.js'
+import { PackageURL } from '../dist/package-url.js'
 
 describe('PackageURL', () => {
   it('Benchmarking the library', async () => {

test/json-export.test.mts

@@ -25,7 +25,7 @@ SOFTWARE.
  */
 import { describe, expect, it } from 'vitest'
 
-import { PackageURL } from '../src/package-url.js'
+import { PackageURL } from '../dist/package-url.js'
 
 describe('PackageURL JSON/dict export', () => {
   describe('toObject', () => {

test/package-url-builder.test.mts

@@ -25,21 +25,28 @@ SOFTWARE.
  */
 import { describe, expect, it } from 'vitest'
 
-import { PackageURLBuilder } from '../src/package-url-builder.js'
-import { PackageURL } from '../src/package-url.js'
+import { PackageURL, PackageURLBuilder } from '../dist/package-url.js'
 
 describe('PackageURLBuilder', () => {
   describe('basic construction', () => {
     it('should build a simple PackageURL', () => {
       const purl = PackageURLBuilder.create().type('npm').name('lodash').build()
 
-      expect(purl).toBeInstanceOf(PackageURL)
+      // Cannot use instanceof due to ESM/CJS interop: test imports ESM wrapper,
+      // but PackageURLBuilder uses CommonJS require(), creating different class references.
+      // Verify constructor name instead.
+      expect(purl.constructor.name).toBe('PackageURL')
       expect(purl.type).toBe('npm')
       expect(purl.name).toBe('lodash')
       expect(purl.namespace).toBeUndefined()
       expect(purl.version).toBeUndefined()
       expect(purl.qualifiers).toBeUndefined()
       expect(purl.subpath).toBeUndefined()
+      // Verify it has PackageURL methods
+      expect(typeof purl.toString).toBe('function')
+      expect(purl.toString()).toBe('pkg:npm/lodash')
+      expect(typeof purl.toJSON).toBe('function')
+      expect(typeof purl.toObject).toBe('function')
     })
 
     it('should build a complete PackageURL with all fields', () => {

test/package-url.test.mts

@@ -33,7 +33,12 @@ import {
 
 import npmBuiltinNames from '../data/npm/builtin-names.json'
 import npmLegacyNames from '../data/npm/legacy-names.json'
-import { LOOP_SENTINEL } from '../src/constants.js'
+import { LOOP_SENTINEL } from '../dist/constants.js'
+import { PurlError, formatPurlErrorMessage } from '../dist/error.js'
+import { recursiveFreeze } from '../dist/objects.js'
+import { PackageURL } from '../dist/package-url.js'
+import { PurlQualifierNames } from '../dist/purl-qualifier-names.js'
+import { PurlType } from '../dist/purl-type.js'
 import {
   encodeComponent,
   encodeNamespace,
@@ -42,7 +47,6 @@ import {
   encodeSubpath,
   encodeVersion,
 } from '../src/encode.js'
-import { PurlError, formatPurlErrorMessage } from '../src/error.js'
 import {
   normalizeName,
   normalizeNamespace,
@@ -51,8 +55,6 @@ import {
   normalizeType,
   normalizeVersion,
 } from '../src/normalize.js'
-import { recursiveFreeze } from '../src/objects.js'
-import { PackageURL } from '../src/package-url.js'
 import {
   PurlComponent,
   PurlComponentEncoder,
@@ -61,8 +63,6 @@ import {
   componentComparator,
   componentSortOrder,
 } from '../src/purl-component.js'
-import { PurlQualifierNames } from '../src/purl-qualifier-names.js'
-import { PurlType } from '../src/purl-type.js'
 import {
   validateEmptyByType,
   validateQualifierKey,
@@ -1254,7 +1254,9 @@ describe('PackageURL', () => {
 
       it('should have proper prototype for PackageURL instances', () => {
         const purl = new PackageURL('type', null, 'name')
-        expect(Object.getPrototypeOf(Object.getPrototypeOf(purl))).toBe(null)
+        // Verify the instance is properly constructed with null prototype for security
+        expect(Object.getPrototypeOf(purl)).toBe(PackageURL.prototype)
+        expect(Object.getPrototypeOf(PackageURL.prototype)).toBe(null)
       })
     })
 

test/result.test.mts

@@ -25,8 +25,8 @@ SOFTWARE.
  */
 import { describe, expect, it } from 'vitest'
 
-import { PackageURL } from '../src/package-url.js'
-import { Err, Ok, ResultUtils, err, ok } from '../src/result.js'
+import { PackageURL } from '../dist/package-url.js'
+import { Err, Ok, ResultUtils, err, ok } from '../dist/result.js'
 
 describe('Result types', () => {
   describe('Ok', () => {

test/url-converter.test.mts

@@ -25,8 +25,8 @@ SOFTWARE.
  */
 import { describe, expect, it } from 'vitest'
 
-import { PackageURL } from '../src/package-url.js'
-import { UrlConverter } from '../src/url-converter.js'
+import { PackageURL } from '../dist/package-url.js'
+import { UrlConverter } from '../dist/url-converter.js'
 
 describe('UrlConverter', () => {
   describe('toRepositoryUrl', () => {