test(purl-types): fix incomplete assertion in legacy npm names test

jdalton · jdalton · commit 3f8d16891415 · 2025-11-04T16:05:56.000-05:00
Add missing .toBe(true) to assertion that validates npm legacy package
names have at least one characteristic (builtin, mixed case, or illegal
characters). Without this, the test would pass even if the assertion
failed.
diff --git a/test/purl-types.test.mts b/test/purl-types.test.mts
@@ -62,7 +62,7 @@ describe('PackageURL type-specific tests', () => {
         expect(
           isBuiltin || isMixedCased || containsIllegalCharacters,
           `assert for ${legacyName}`,
-        )
+        ).toBe(true)
       }
     })
 

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ describe('PackageURL type-specific tests', () => {`
`62`	`62`	`expect(`
`63`	`63`	`isBuiltin \|\| isMixedCased \|\| containsIllegalCharacters,`
`64`	`64`	`assert for ${legacyName}`,
`65`		`- )`
	`65`	`+ ).toBe(true)`
`66`	`66`	`}`
`67`	`67`	`})`
`68`	`68`