test: migrate all int test to new util (#635)

test/cli/basic/index.test.ts

@@ -1,9 +1,9 @@
 import fs from 'fs'
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 describe('cli', () => {
   it(`cli basic should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['hello.js', '-o', 'dist/hello.bundle.js'],
     })
@@ -12,7 +12,7 @@ describe('cli', () => {
   })
 
   it(`cli format should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['hello.js', '-f', 'cjs', '-o', 'dist/hello.cjs'],
     })
@@ -24,7 +24,7 @@ describe('cli', () => {
   })
 
   it(`cli compress should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['hello.js', '-m', '-o', 'dist/hello.bundle.min.js'],
     })
@@ -36,7 +36,7 @@ describe('cli', () => {
   })
 
   it(`cli sourcemap should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['hello.js', '--sourcemap', '-o', 'dist/hello.js'],
     })
@@ -51,7 +51,7 @@ describe('cli', () => {
   })
 
   it(`cli minified with sourcemap should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['hello.js', '-m', '--sourcemap', '-o', 'dist/hello.js'],
     })

test/cli/cjs-relative-imports/index.test.ts

@@ -1,8 +1,8 @@
 import fs from 'fs'
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 test(`cli cjs-relative-imports should work properly`, async () => {
-  const { code, distFile } = await createCliJob({
+  const { code, distFile } = await runCli({
     directory: __dirname,
     args: ['index.js', '-o', 'dist/index.js'],
   })

test/cli/dts-bundle/index.test.ts

@@ -1,7 +1,7 @@
 import fs from 'fs'
 import fsp from 'fs/promises'
 import { join } from 'path'
-import { createCliJob, deleteFile } from '../../testing-utils'
+import { runCli, deleteFile } from '../../testing-utils'
 
 describe('cli', () => {
   const dir = __dirname
@@ -16,7 +16,7 @@ describe('cli', () => {
   })
 
   it(`cli dts-bundle option should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['./base.ts', '-o', 'dist/base.js', '--dts-bundle'],
     })
@@ -27,7 +27,7 @@ describe('cli', () => {
   })
 
   it(`cli dts-bundle option should not bundle dts without dts-bundle`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['./base.ts', '-o', 'dist/base.js'],
     })
@@ -42,7 +42,7 @@ describe('cli', () => {
       join(dir, './package.json'),
       '{ "name": "prepare-ts-with-pkg-json", "dependencies": { "@swc/types": "*" } }',
     )
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['./base.ts', '-o', 'dist/base.js'],
     })

test/cli/dts/index.test.ts

@@ -1,10 +1,10 @@
 import fs from 'fs'
 import path from 'path'
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 describe('cli', () => {
   it(`cli dts should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: ['./input.ts', '-o', 'dist/output.js'],
     })

test/cli/env-var/index.test.ts

@@ -1,8 +1,8 @@
 import fs from 'fs'
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 test(`cli env-var should work properly`, async () => {
-  const { code, distFile } = await createCliJob({
+  const { code, distFile } = await runCli({
     directory: __dirname,
     args: ['index.js', '--env', 'MY_TEST_ENV', '-o', 'dist/index.js'],
     options: {

test/cli/externals/index.test.ts

@@ -1,9 +1,9 @@
 import fs from 'fs'
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 describe('cli', () => {
   it(`cli external should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: [
         'with-externals.js',
@@ -21,7 +21,7 @@ describe('cli', () => {
   })
 
   it(`cli no-externals should work properly`, async () => {
-    const { code, distFile } = await createCliJob({
+    const { code, distFile } = await runCli({
       directory: __dirname,
       args: [
         'with-externals.js',

test/cli/no-dts/index.test.ts

@@ -1,9 +1,9 @@
 import fs from 'fs'
 import path from 'path'
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 test(`cli no-dts option should work properly`, async () => {
-  const { code, distFile } = await createCliJob({
+  const { code, distFile } = await runCli({
     directory: __dirname,
     args: ['./base.ts', '-o', 'dist/base.js', '--no-dts'],
   })

test/cli/no-entry/index.test.ts

@@ -1,7 +1,7 @@
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 test(`cli no-entry should work properly`, async () => {
-  const { code, stderr } = await createCliJob({
+  const { code, stderr } = await runCli({
     directory: __dirname,
   })
   expect(

test/cli/output-in-watch/index.test.ts

@@ -1,7 +1,7 @@
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 test('cli - output-in-watch', async () => {
-  const { stdout, signal } = await createCliJob({
+  const { stdout, signal } = await runCli({
     directory: __dirname,
     args: ['hello.js', '-w', '-o', 'dist/hello.bundle.js'],
     abortTimeout: 3000,

test/cli/target/index.test.ts

@@ -1,8 +1,8 @@
 import fs from 'fs'
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 
 test(`cli es2020-target should work properly`, async () => {
-  const { code, distFile } = await createCliJob({
+  const { code, distFile } = await runCli({
     directory: __dirname,
     args: ['es2020.ts', '--target', 'es2020', '-o', 'dist/es2020.js'],
   })

test/cli/workspace/index.test.ts

@@ -1,10 +1,10 @@
 import fs from 'fs'
-import { createCliJob } from '../../testing-utils'
+import { runCli } from '../../testing-utils'
 import path from 'path'
 
 describe('cli', () => {
   it(`cli workspace should work properly`, async () => {
-    const { code, distDir } = await createCliJob({
+    const { code, distDir } = await runCli({
       directory: __dirname,
       args: ['./index.ts'],
     })

test/integration/bin/cts/index.test.ts

@@ -1,19 +1,12 @@
-import {
-  assertFilesContent,
-  createIntegrationTest,
-} from '../../../testing-utils'
+import { assertFilesContent, createJob } from '../../../testing-utils'
 
 describe('integration bin/cts', () => {
+  const { distDir } = createJob({
+    directory: __dirname,
+  })
   it('should work with bin as .cts extension', async () => {
-    await createIntegrationTest(
-      {
-        directory: __dirname,
-      },
-      async ({ distDir }) => {
-        await assertFilesContent(distDir, {
-          'bin/index.cjs': '#!/usr/bin/env node',
-        })
-      },
-    )
+    await assertFilesContent(distDir, {
+      'bin/index.cjs': '#!/usr/bin/env node',
+    })
   })
 })

test/integration/bin/multi-path/index.test.ts

@@ -1,6 +1,6 @@
 import {
   assertFilesContent,
-  createIntegrationTest,
+  createJob,
   isWindows,
 } from '../../../testing-utils'
 
@@ -10,17 +10,13 @@ describe('integration bin/multi-path', () => {
     it('skip test on windows', () => {})
     return
   }
+  const { distDir } = createJob({
+    directory: __dirname,
+  })
   it('should work with bin as multi path', async () => {
-    await createIntegrationTest(
-      {
-        directory: __dirname,
-      },
-      async ({ distDir }) => {
-        await assertFilesContent(distDir, {
-          'bin/a.js': '#!/usr/bin/env node',
-          'bin/b.js': '#!/usr/bin/env node',
-        })
-      },
-    )
+    await assertFilesContent(distDir, {
+      'bin/a.js': '#!/usr/bin/env node',
+      'bin/b.js': '#!/usr/bin/env node',
+    })
   })
 })

test/integration/bin/patch-binary/index.test.ts

@@ -1,22 +1,16 @@
 import {
-  createIntegrationTest,
   assertContainFiles,
   assertFilesContent,
+  createJob,
 } from '../../../testing-utils'
 
 describe('integration bin/patch-binary', () => {
+  const { distDir } = createJob({ directory: __dirname })
   it('should patch binary directive', async () => {
-    await createIntegrationTest(
-      {
-        directory: __dirname,
-      },
-      async ({ distDir }) => {
-        const distFiles = ['bin.js']
-        await assertContainFiles(distDir, distFiles)
-        await assertFilesContent(distDir, {
-          'bin.js': "#!/usr/bin/env node\nconsole.log('Hello, world!');",
-        })
-      },
-    )
+    const distFiles = ['bin.js']
+    await assertContainFiles(distDir, distFiles)
+    await assertFilesContent(distDir, {
+      'bin.js': "#!/usr/bin/env node\nconsole.log('Hello, world!');",
+    })
   })
 })

test/integration/bin/single-path/index.test.ts

@@ -1,8 +1,7 @@
 import {
-  createIntegrationTest,
+  createJob,
   assertContainFiles,
   assertFilesContent,
-  getFileNamesFromDirectory,
   isWindows,
 } from '../../../testing-utils'
 
@@ -12,22 +11,12 @@ describe('integration bin/single-path', () => {
     it('skip test on windows', () => {})
     return
   }
+  const { distDir } = createJob({ directory: __dirname })
   it('should work with bin as single path', async () => {
-    await createIntegrationTest(
-      {
-        directory: __dirname,
-      },
-      async ({ distDir }) => {
-        console.log(
-          'getFileNamesFromDirectory',
-          getFileNamesFromDirectory(distDir),
-        )
-        const distFiles = ['bin.js']
-        await assertContainFiles(distDir, distFiles)
-        await assertFilesContent(distDir, {
-          'bin.js': /console.log\('Hello, world!'\)/,
-        })
-      },
-    )
+    const distFiles = ['bin.js']
+    await assertContainFiles(distDir, distFiles)
+    await assertFilesContent(distDir, {
+      'bin.js': /console.log\('Hello, world!'\)/,
+    })
   })
 })

test/integration/browserslist/index.test.ts

@@ -1,16 +1,10 @@
-import { createIntegrationTest, assertFilesContent } from '../../testing-utils'
+import { assertFilesContent, createJob } from '../../testing-utils'
 
 describe('browserslist', () => {
+  const { distDir } = createJob({ directory: __dirname })
   it('should work with basic JSX format', async () => {
-    await createIntegrationTest(
-      {
-        directory: __dirname,
-      },
-      ({ distDir }) => {
-        assertFilesContent(distDir, {
-          'index.js': `_class_private_field_loose_key`,
-        })
-      },
-    )
+    await assertFilesContent(distDir, {
+      'index.js': `_class_private_field_loose_key`,
+    })
   })
 })

test/integration/conflicted-entry/conflicted-entry.test.ts

@@ -1,17 +1,10 @@
-import { createIntegrationTest } from '../../testing-utils'
+import { createJob } from '../../testing-utils'
 
 describe('integration - conflicted-entry', () => {
+  const { job } = createJob({ directory: __dirname })
   it('should error on conflicted entries', async () => {
-    await createIntegrationTest(
-      {
-        directory: __dirname,
-      },
-      async ({ code, stderr }) => {
-        expect(code).toBe(1)
-        expect(stderr).toContain(
-          'Conflicted entry files found for entries: ./foo',
-        )
-      },
-    )
+    const { code, stderr } = job
+    expect(code).toBe(1)
+    expect(stderr).toContain('Conflicted entry files found for entries: ./foo')
   })
 })