From c89aa60986f99011ad69f5e7d57cf1cb34593069 Mon Sep 17 00:00:00 2001 From: CrazyMax Date: Wed, 1 Feb 2023 16:47:01 +0100 Subject: [PATCH] docker: configDir Signed-off-by: CrazyMax --- __tests__/docker.test.ts | 26 +++++++++++++++++++++++++- src/docker.ts | 6 ++++++ 2 files changed, 31 insertions(+), 1 deletion(-) diff --git a/__tests__/docker.test.ts b/__tests__/docker.test.ts index 1ee1dfd..9735031 100644 --- a/__tests__/docker.test.ts +++ b/__tests__/docker.test.ts @@ -14,8 +14,10 @@ * limitations under the License. */ -import {beforeEach, describe, expect, it, jest} from '@jest/globals'; +import {afterEach, beforeEach, describe, expect, it, jest} from '@jest/globals'; import * as exec from '@actions/exec'; +import path from 'path'; +import osm = require('os'); import {Docker} from '../src/docker'; @@ -23,6 +25,28 @@ beforeEach(() => { jest.clearAllMocks(); }); +describe('configDir', () => { + const originalEnv = process.env; + beforeEach(() => { + jest.resetModules(); + process.env = { + ...originalEnv, + DOCKER_CONFIG: '/var/docker/config' + }; + }); + afterEach(() => { + process.env = originalEnv; + }); + it('returns default', async () => { + process.env.DOCKER_CONFIG = ''; + jest.spyOn(osm, 'homedir').mockImplementation(() => path.join('/tmp', 'home')); + expect(Docker.configDir).toEqual(path.join('/tmp', 'home', '.docker')); + }); + it('returns from env', async () => { + expect(Docker.configDir).toEqual('/var/docker/config'); + }); +}); + describe('isAvailable', () => { it('cli', () => { const execSpy = jest.spyOn(exec, 'getExecOutput'); diff --git a/src/docker.ts b/src/docker.ts index dca238d..186c43e 100644 --- a/src/docker.ts +++ b/src/docker.ts @@ -14,9 +14,15 @@ * limitations under the License. */ +import os from 'os'; +import path from 'path'; import * as exec from '@actions/exec'; export class Docker { + static get configDir(): string { + return process.env.DOCKER_CONFIG || path.join(os.homedir(), '.docker'); + } + public static isAvailable(): boolean { let dockerAvailable = false; exec