Commit 7e358da

stephankaagfloydspace

and

committed

fix(esbuild): use absolute path for esbuild entry (#4)

* fix(esbuild): use absolute path for esbuild entry * test: fix unit test Co-authored-by: Victor Korzunin <ifloydrose@gmail.com>

1 parent 6663596 commit 7e358daCopy full SHA for 7e358da

File tree

-3

lines changed

-3

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ export class NodejsFunction extends lambda.Function {`
`42`	`42`	`es.buildSync({`
`43`	`43`	`...buildOptions,`
`44`	`44`	`external: union(exclude, buildOptions.external \|\| []),`
`45`		`- entryPoints: [entry],`
	`45`	`+ entryPoints: [path.join(projectRoot,entry)],`
`46`	`46`	`outdir: path.join(projectRoot, BUILD_FOLDER, path.dirname(entry)),`
`47`	`47`	`platform: 'node',`
`48`	`48`	`});`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ import { Runtime, RuntimeFamily } from '@aws-cdk/aws-lambda';`
`7`	`7`	`import { Stack } from '@aws-cdk/core';`
`8`	`8`	`import { buildSync } from 'esbuild';`
`9`	`9`	`import mockfs from 'mock-fs';`
`10`		`-import path from 'path';`
	`10`	`+import path,{join} from 'path';`
`11`	`11`
`12`	`12`	`import { NodejsFunction } from '../src';`
`13`	`13`
`@@ -56,7 +56,7 @@ describe('NodejsFunction tests', () => {`
`56`	`56`	`${'a/b/c.h'} \| ${'a/b/c.ts'}`
`57`	`57`	`('Should be valid entry with default rootDir', ({ handler, entry }) => {
`58`	`58`	`new NodejsFunction(new Stack(), 'lambda-function', { handler });`
`59`		`- expect(buildSync).toHaveBeenCalledWith(expect.objectContaining({ entryPoints: [entry] }));`
	`59`	`+ expect(buildSync).toHaveBeenCalledWith(expect.objectContaining({ entryPoints: [join(process.cwd(),entry)] }));`
`60`	`60`	`});`
`61`	`61`
`62`	`62`	`it('Should be valid outdir with custom rootDir', () => {`

Comments

(0)