docker/actions-toolkit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #51 from crazy-max/fix-docker-exporter
Some checks failed
publish / publish (push) Has been cancelled
Details

Browse Source 
buildx: fix docker exporter check
This commit is contained in:
CrazyMax
2023-02-20 10:28:01 +01:00
committed by GitHub
parent d153cfaf3c 1acd6c2fc0
commit 1383a2bcaf
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
__tests__/buildx/inputs.test.ts
View File

@@ -228,6 +228,8 @@ describe('hasDockerExporter', () => {
[['type=docker', 'type=tar,dest=/tmp/image.tar'], true, undefined], [['type=docker', 'type=tar,dest=/tmp/image.tar'], true, undefined],
[['"type=tar","dest=/tmp/image.tar"'], false, undefined], [['"type=tar","dest=/tmp/image.tar"'], false, undefined],
[['" type= local" , dest=./release-out'], false, undefined], [['" type= local" , dest=./release-out'], false, undefined],
[['type=docker'], true, false],
[['type=docker'], true, true],
[['.'], true, true], [['.'], true, true],
])('given %p returns %p', async (exporters: Array<string>, expected: boolean, load: boolean | undefined) => { ])('given %p returns %p', async (exporters: Array<string>, expected: boolean, load: boolean | undefined) => {
expect(Inputs.hasDockerExporter(exporters, load)).toEqual(expected); expect(Inputs.hasDockerExporter(exporters, load)).toEqual(expected);

2
src/buildx/inputs.ts
View File

@@ -134,7 +134,7 @@ export class Inputs {
} }
public static hasDockerExporter(exporters: string[], load?: boolean): boolean { public static hasDockerExporter(exporters: string[], load?: boolean): boolean {
return load ?? Inputs.hasExporterType('docker', exporters); return load || Inputs.hasExporterType('docker', exporters);
} }
public static hasExporterType(name: string, exporters: string[]): boolean { public static hasExporterType(name: string, exporters: string[]): boolean {