Skip to content

Commit

Permalink
Fix build
Browse files Browse the repository at this point in the history
  • Loading branch information
rix0rrr committed Dec 2, 2022
1 parent d95c6e8 commit 7ab3e49
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
1 change: 0 additions & 1 deletion packages/cdk-assets/lib/private/handlers/files.ts
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,6 @@ export class FileAssetHandler implements IAssetHandler {
if (this.host.aborted) { return; }
const publishFile = this.asset.source.executable ?
await this.externalPackageFile(this.asset.source.executable) : await this.packageFile(this.asset.source);
}

this.host.emitMessage(EventType.UPLOAD, `Upload ${s3Url}`);

Expand Down
12 changes: 0 additions & 12 deletions packages/cdk-assets/test/files.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -310,18 +310,6 @@ test('correctly identify asset path if path is absolute', async () => {
expect(true).toBeTruthy(); // No exception, satisfy linter
});

test('empty directory prints failures', async () => {
const progressListener = new FakeListener();
const pub = new AssetPublishing(AssetManifest.fromPath('/emptyzip/cdk.out'), { aws, progressListener });

aws.mockS3.listObjectsV2 = mockedApiResult({ Contents: undefined });
aws.mockS3.upload = mockUpload(); // Should not be hit

await pub.publish();

expect(progressListener.messages).toContainEqual(expect.stringContaining('EMPTY ZIP FILE'));
});

describe('external assets', () => {
let pub: AssetPublishing;
beforeEach(() => {
Expand Down

0 comments on commit 7ab3e49

Please sign in to comment.