From 97c94e60ce107b9d10396b767690a4d16a16082d Mon Sep 17 00:00:00 2001 From: ayushsingh82 Date: Wed, 19 Aug 2026 09:03:46 +0530 Subject: [PATCH] Add regression test for server.signingWorker being fully optional server.signingWorker (wallet-gateway/remote/src/config/Config.ts) is already wrapped in z.preprocess((val) => val ?? {}, ...) with pollInterval defaulting to 5000, so the whole block can already be omitted from config.yaml/json. That behavior had no direct test coverage, so add one. Note: this repo has no helm chart, so the "propagate to the helm chart" part of #2176 doesn't apply here -- it likely refers to a different/deployment repo. Fixes #2176 Signed-off-by: ayushsingh82 --- wallet-gateway/remote/src/config/Config.test.ts | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/wallet-gateway/remote/src/config/Config.test.ts b/wallet-gateway/remote/src/config/Config.test.ts index 146374d91..010d139ad 100644 --- a/wallet-gateway/remote/src/config/Config.test.ts +++ b/wallet-gateway/remote/src/config/Config.test.ts @@ -2,6 +2,7 @@ // SPDX-License-Identifier: Apache-2.0 import { expect, test } from 'vitest' +import { serverConfigSchema } from './Config.js' import { ConfigUtils } from './ConfigUtils.js' test('config from json file', async () => { @@ -28,3 +29,8 @@ test('config from json file', async () => { ) } }) + +test('server.signingWorker is fully optional and defaults when omitted', () => { + const parsed = serverConfigSchema.parse({}) + expect(parsed.signingWorker.pollInterval).toBe(5000) +})