Skip to content

Commit 54de91a

Browse files
committed
test: remove skipped tests requiring external @socketsecurity/registry module
Remove 6 skipped tests that depend on @socketsecurity/registry: - resolveRegistryPackageName describe block (4 tests) - PackageURL lazy loading test - scoped package name conversion test These tests cannot run without the external module and were permanently skipped, providing no value in the test suite. Test count: 78 tests (6 skipped) → 72 tests (0 skipped)
1 parent 3197f92 commit 54de91a

1 file changed

Lines changed: 0 additions & 49 deletions

File tree

test/packages/operations.test.ts

Lines changed: 0 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -124,35 +124,6 @@ describe('packages/operations', () => {
124124
})
125125
})
126126

127-
describe.skip('resolveRegistryPackageName', () => {
128-
// These tests are skipped because they require @socketsecurity/registry module
129-
// which may not be available in all test environments
130-
it('should convert unscoped package name', () => {
131-
const result = resolveRegistryPackageName('package')
132-
expect(result).toBe('package')
133-
})
134-
135-
it('should convert scoped package name with registry delimiter', () => {
136-
const result = resolveRegistryPackageName('@scope/package')
137-
// Should use REGISTRY_SCOPE_DELIMITER which is typically '--'
138-
expect(result).toContain('scope')
139-
expect(result).toContain('package')
140-
})
141-
142-
it('should handle various scoped package formats', () => {
143-
const result1 = resolveRegistryPackageName('@org/pkg')
144-
expect(result1).toBeTruthy()
145-
146-
const result2 = resolveRegistryPackageName('@socketregistry/test')
147-
expect(result2).toBeTruthy()
148-
})
149-
150-
it('should return unscoped name unchanged', () => {
151-
const result = resolveRegistryPackageName('simple-package')
152-
expect(result).toBe('simple-package')
153-
})
154-
})
155-
156127
describe('readPackageJson', () => {
157128
it('should read and parse package.json from directory', async () => {
158129
await runWithTempDir(async tmpDir => {
@@ -706,13 +677,6 @@ describe('packages/operations', () => {
706677
await expect(packPackage('/non/existent')).rejects.toThrow()
707678
}, 30000)
708679

709-
it.skip('should lazy load PackageURL on first use', () => {
710-
// Skipped: requires @socketsecurity/registry module
711-
// resolveRegistryPackageName should lazy load PackageURL
712-
const result = resolveRegistryPackageName('package')
713-
expect(result).toBe('package')
714-
})
715-
716680
it('should lazy load pacote on first use', async () => {
717681
// extractPackage should lazy load pacote
718682
await expect(
@@ -827,19 +791,6 @@ describe('packages/operations', () => {
827791
}, 'integration-editable-workflow-')
828792
})
829793

830-
it.skip('should handle scoped package name conversions', () => {
831-
// Skipped: requires @socketsecurity/registry module
832-
const scopedName = '@scope/package'
833-
const registryName = resolveRegistryPackageName(scopedName)
834-
835-
// Should contain both parts
836-
expect(registryName).toContain('scope')
837-
expect(registryName).toContain('package')
838-
839-
// Should be different from original
840-
expect(registryName !== scopedName || !scopedName.startsWith('@')).toBe(true)
841-
})
842-
843794
it('should handle release tag extraction for various formats', () => {
844795
const testCases = [
845796
{ input: 'pkg@1.0.0', expected: '1.0.0' },

0 commit comments

Comments
 (0)