From 04b4619314d73f7fd66ba47713de1780d9ba6789 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 12 Oct 2026 08:54:08 +0000 Subject: [PATCH 1/2] test: cover parseDifcProxyHost via docker-manager barrel Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- src/docker-manager.test.ts | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/src/docker-manager.test.ts b/src/docker-manager.test.ts index fbdd79e26..b05e0ad97 100644 --- a/src/docker-manager.test.ts +++ b/src/docker-manager.test.ts @@ -171,6 +171,32 @@ describe('docker-manager (barrel re-exports)', () => { }); }); + describe('parseDifcProxyHost via barrel', () => { + it('defaults for empty input', () => { + expect(dockerManager.parseDifcProxyHost(' ')).toEqual({ host: 'host.docker.internal', port: '18443' }); + }); + + it('parses host:port and strips scheme', () => { + expect(dockerManager.parseDifcProxyHost('tcp://example.com:9000')).toEqual({ host: 'example.com', port: '9000' }); + }); + + it('defaults the port and keeps 443 for https scheme', () => { + expect(dockerManager.parseDifcProxyHost('example.com')).toEqual({ host: 'example.com', port: '18443' }); + expect(dockerManager.parseDifcProxyHost('https://example.com:443').port).toBe('443'); + }); + + it('strips IPv6 brackets', () => { + expect(dockerManager.parseDifcProxyHost('[::1]:8080')).toEqual({ host: '::1', port: '8080' }); + }); + + it('rejects malformed values and out-of-range ports', () => { + expect(() => dockerManager.parseDifcProxyHost('http://')).not.toThrow(); + expect(() => dockerManager.parseDifcProxyHost('host:abc')).toThrow(/Invalid --difc-proxy-host/); + expect(() => dockerManager.parseDifcProxyHost('host:99999')).toThrow(/Invalid --difc-proxy-host/); + expect(() => dockerManager.parseDifcProxyHost('host:0')).toThrow(/Must be between 1 and 65535/); + }); + }); + describe('filterCapDrop via barrel', () => { it('returns empty for undefined or empty lists', () => { expect(dockerManager.filterCapDrop(undefined, null)).toEqual([]); From f4fa8216e8b068c506a24b170eb037dd027094d5 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 12 Oct 2026 14:25:54 +0000 Subject: [PATCH 2/2] test: remove duplicate parser barrel coverage --- src/docker-manager.test.ts | 26 -------------------------- 1 file changed, 26 deletions(-) diff --git a/src/docker-manager.test.ts b/src/docker-manager.test.ts index b05e0ad97..fbdd79e26 100644 --- a/src/docker-manager.test.ts +++ b/src/docker-manager.test.ts @@ -171,32 +171,6 @@ describe('docker-manager (barrel re-exports)', () => { }); }); - describe('parseDifcProxyHost via barrel', () => { - it('defaults for empty input', () => { - expect(dockerManager.parseDifcProxyHost(' ')).toEqual({ host: 'host.docker.internal', port: '18443' }); - }); - - it('parses host:port and strips scheme', () => { - expect(dockerManager.parseDifcProxyHost('tcp://example.com:9000')).toEqual({ host: 'example.com', port: '9000' }); - }); - - it('defaults the port and keeps 443 for https scheme', () => { - expect(dockerManager.parseDifcProxyHost('example.com')).toEqual({ host: 'example.com', port: '18443' }); - expect(dockerManager.parseDifcProxyHost('https://example.com:443').port).toBe('443'); - }); - - it('strips IPv6 brackets', () => { - expect(dockerManager.parseDifcProxyHost('[::1]:8080')).toEqual({ host: '::1', port: '8080' }); - }); - - it('rejects malformed values and out-of-range ports', () => { - expect(() => dockerManager.parseDifcProxyHost('http://')).not.toThrow(); - expect(() => dockerManager.parseDifcProxyHost('host:abc')).toThrow(/Invalid --difc-proxy-host/); - expect(() => dockerManager.parseDifcProxyHost('host:99999')).toThrow(/Invalid --difc-proxy-host/); - expect(() => dockerManager.parseDifcProxyHost('host:0')).toThrow(/Must be between 1 and 65535/); - }); - }); - describe('filterCapDrop via barrel', () => { it('returns empty for undefined or empty lists', () => { expect(dockerManager.filterCapDrop(undefined, null)).toEqual([]);