From 3de5feb3e1edb8f84c6cfdac9462de90769816fd Mon Sep 17 00:00:00 2001 From: unional Date: Tue, 3 Jan 2023 01:05:35 -0800 Subject: [PATCH] chore: move test into describe block --- .../src/__tests__/resolve.test.ts | 52 ++++++++++--------- 1 file changed, 27 insertions(+), 25 deletions(-) diff --git a/packages/jest-resolve/src/__tests__/resolve.test.ts b/packages/jest-resolve/src/__tests__/resolve.test.ts index d9bcba08a0c8..e30e9e71f25a 100644 --- a/packages/jest-resolve/src/__tests__/resolve.test.ts +++ b/packages/jest-resolve/src/__tests__/resolve.test.ts @@ -356,38 +356,40 @@ describe('findNodeModule', () => { ); }); - test('supports array pattern - resolve to first found', () => { - const result = Resolver.findNodeModule('#array-import', { - basedir: path.resolve(importsRoot, './array-import/index.cjs'), - conditions: ['import'], + describe('array pattern', () => { + test('supports array pattern - resolve to first found', () => { + const result = Resolver.findNodeModule('#array-import', { + basedir: path.resolve(importsRoot, './array-import/index.cjs'), + conditions: ['import'], + }); + + expect(result).toEqual( + path.resolve(importsRoot, './array-import/node.mjs'), + ); }); - expect(result).toEqual( - path.resolve(importsRoot, './array-import/node.mjs'), - ); - }); + test('supports array pattern - ignore not exist internal file', () => { + const result = Resolver.findNodeModule('#array-import', { + basedir: path.resolve(importsRoot, './array-import/index.cjs'), + conditions: ['browser'], + }); - test('supports array pattern - ignore not exist internal file', () => { - const result = Resolver.findNodeModule('#array-import', { - basedir: path.resolve(importsRoot, './array-import/index.cjs'), - conditions: ['browser'], + expect(result).toEqual( + path.resolve(importsRoot, './array-import/browser.cjs'), + ); }); - expect(result).toEqual( - path.resolve(importsRoot, './array-import/browser.cjs'), - ); - }); + test('supports array pattern - ignore not exist external module', () => { + // this is for optional dependency + const result = Resolver.findNodeModule('#array-import', { + basedir: path.resolve(importsRoot, './array-import/index.cjs'), + conditions: ['require'], + }); - test('supports array pattern - ignore not exist external module', () => { - // this is for optional dependency - const result = Resolver.findNodeModule('#array-import', { - basedir: path.resolve(importsRoot, './array-import/index.cjs'), - conditions: ['require'], + expect(result).toEqual( + path.resolve(importsRoot, './array-import/node.cjs'), + ); }); - - expect(result).toEqual( - path.resolve(importsRoot, './array-import/node.cjs'), - ); }); test('fails for non-existent mapping', () => {