TINY-13550: Refactor create shared function type

Co-authored-by: ltrouton <46954949+ltrouton@users.noreply.github.com>

Author: shanmen-tiny

modules/server/src/main/ts/bedrock/compiler/Compiler.ts

@@ -1,13 +1,26 @@
 import * as fs from 'fs';
 import * as Webpack from '../compiler/Webpack';
 import * as Rspack from '../compiler/Rspack';
+import * as Types from './Types';
 
 export interface Compiler {
   readonly generate: () => Promise<Buffer | string>;
 }
 
-export const compile = (bundler: 'webpack' | 'rspack', tsConfigFile: string, scratchDir: string, basedir: string, exitOnCompileError: boolean, files: string[], coverage: string[], polyfills: string[]): Compiler => {
-  const getCompileFunc = () => {
+export const compile = (args: { bundler: Types.Bundler; tsConfigFile: string; scratchDir: string; basedir: string; exitOnCompileError: boolean; files: string[]; coverage: string[]; polyfills: string[];
+}): Compiler => {
+  const {
+    bundler,
+    tsConfigFile,
+    scratchDir,
+    basedir,
+    exitOnCompileError,
+    files,
+    coverage,
+    polyfills
+  } = args;
+
+  const getCompileFunc = (): Types.CompileFn => {
     switch (bundler) {
       case 'rspack':
         return Rspack.compile;

modules/server/src/main/ts/bedrock/compiler/Rspack.ts

@@ -6,7 +6,7 @@ import { ExitCodes } from '../util/ExitCodes';
 import * as Imports from './Imports';
 import { rspack, RspackOptions } from '@rspack/core';
 import { RspackDevServer } from '@rspack/dev-server';
-import { RspackCompileInfo, DevServerServeSettings } from './Types';
+import { RspackCompileInfo, DevServerServeSettings, CompileFn } from './Types';
 
 const getWebPackConfigTs = (tsConfigFile: string, scratchFile: string, dest: string, manualMode: boolean, basedir: string): RspackOptions => {
   // eslint-disable-next-line @typescript-eslint/no-var-requires
@@ -168,7 +168,7 @@ const getCompileInfo = (tsConfigFile: string, scratchDir: string, basedir: strin
   return getTsCompileInfo(tsConfigFile, scratchDir, basedir, manualMode);
 };
 
-export const compile = async (tsConfigFile: string, scratchDir: string, basedir: string, exitOnCompileError: boolean, srcFiles: string[], _coverage: string[], polyfills: string[]): Promise<string> => {
+export const compile: CompileFn = async (tsConfigFile: string, scratchDir: string, basedir: string, exitOnCompileError: boolean, srcFiles: string[], _coverage: string[], polyfills: string[]): Promise<string> => {
   const compileInfo = await getCompileInfo(tsConfigFile, scratchDir, basedir, false);
   return compileTests(compileInfo, exitOnCompileError, srcFiles, polyfills);
 };

modules/server/src/main/ts/bedrock/compiler/Types.ts

@@ -17,3 +17,14 @@ export interface CompileInfo<T> {
   readonly config: T;
 }
 
+export type CompileFn = (
+  tsConfigFile: string,
+  scratchDir: string,
+  basedir: string,
+  exitOnCompileError: boolean,
+  srcFiles: string[],
+  coverage: string[],
+  polyfills: string[]
+) => Promise<string>;
+
+export type Bundler = 'rspack' | 'webpack';

modules/server/src/main/ts/bedrock/server/RunnerRoutes.ts

@@ -6,6 +6,7 @@ import * as Routes from './Routes';
 import * as Compiler from '../compiler/Compiler';
 import * as FileUtils from '../util/FileUtils';
 import * as Arr from '../util/Arr';
+import * as Types from '../compiler/Types';
 
 interface PackageJson {
   readonly name: string;
@@ -17,22 +18,22 @@ interface WorkspaceRoot {
   folder: string;
 }
 
-export const generate = async (mode: string, projectdir: string, basedir: string, configFile: string, bundler: 'webpack' | 'rspack', testfiles: string[], chunk: number,
+export const generate = async (mode: string, projectdir: string, basedir: string, configFile: string, bundler: Types.Bundler, testfiles: string[], chunk: number,
                                retries: number, singleTimeout: number, stopOnFailure: boolean, basePage: string, coverage: string[], polyfills: string[]): Promise<Routes.Runner> => {
   const files = testfiles.map((filePath) => {
     return path.relative(projectdir, filePath);
   });
 
-  const testGenerator = Compiler.compile(
+  const testGenerator = Compiler.compile({
     bundler,
-    path.join(projectdir, configFile),
-    path.join(projectdir, 'scratch'),
+    tsConfigFile: path.join(projectdir, configFile),
+    scratchDir: path.join(projectdir, 'scratch'),
     basedir,
-    mode === 'auto',
+    exitOnCompileError: mode === 'auto',
     files,
     coverage,
     polyfills
-  );
+  });
 
   // read the project json file to determine the project name to expose resources as `/project/${name}`
   const pkjson: PackageJson = FileUtils.readFileAsJson(`${projectdir}/package.json`);