From 7f51e58a06566d211b949aaa64f03fe20658d1a8 Mon Sep 17 00:00:00 2001 From: Gabriel-Ladzaretti <97394622+Gabriel-Ladzaretti@users.noreply.github.com> Date: Sat, 18 Jun 2022 14:35:43 +0300 Subject: [PATCH] fix(core/config): Log cached repo config filename and content (#16134) --- .../repository/onboarding/branch/check.ts | 4 +++ .../onboarding/branch/index.spec.ts | 29 +++++++++++++++++++ 2 files changed, 33 insertions(+) diff --git a/lib/workers/repository/onboarding/branch/check.ts b/lib/workers/repository/onboarding/branch/check.ts index 7717a1ae85..911092d953 100644 --- a/lib/workers/repository/onboarding/branch/check.ts +++ b/lib/workers/repository/onboarding/branch/check.ts @@ -76,6 +76,10 @@ export const isOnboarded = async (config: RenovateConfig): Promise<boolean> => { configFileContent.renovate ) { logger.debug('Existing config file confirmed'); + logger.debug( + { fileName: cache.configFileName, config: configFileContent }, + 'Repository config' + ); return true; } } diff --git a/lib/workers/repository/onboarding/branch/index.spec.ts b/lib/workers/repository/onboarding/branch/index.spec.ts index 98a55a94d0..ad8d1074a7 100644 --- a/lib/workers/repository/onboarding/branch/index.spec.ts +++ b/lib/workers/repository/onboarding/branch/index.spec.ts @@ -12,6 +12,7 @@ import { REPOSITORY_FORKED, REPOSITORY_NO_PACKAGE_FILES, } from '../../../../constants/error-messages'; +import { logger } from '../../../../logger'; import type { Pr } from '../../../../modules/platform'; import { PrState } from '../../../../types'; import * as _cache from '../../../../util/cache/repository'; @@ -165,6 +166,19 @@ describe('workers/repository/onboarding/branch/index', () => { cache.getCache.mockReturnValue({ configFileName: '.renovaterc' }); platform.getJsonFile.mockResolvedValueOnce({}); const res = await checkOnboardingBranch(config); + expect(logger.debug).toHaveBeenCalledWith( + 'Checking cached config file name' + ); + expect(logger.debug).toHaveBeenCalledWith( + 'Existing config file confirmed' + ); + expect(logger.debug).toHaveBeenCalledWith( + { + fileName: '.renovaterc', + config: {}, + }, + 'Repository config' + ); expect(res.repoIsOnboarded).toBeTrue(); }); @@ -173,6 +187,21 @@ describe('workers/repository/onboarding/branch/index', () => { platform.getJsonFile.mockResolvedValueOnce({ renovate: {} }); fs.readLocalFile.mockResolvedValueOnce('{}'); const res = await checkOnboardingBranch(config); + expect(logger.debug).toHaveBeenCalledWith( + 'Checking cached config file name' + ); + expect(logger.debug).toHaveBeenCalledWith( + 'Existing config file confirmed' + ); + expect(logger.debug).toHaveBeenCalledWith( + { + fileName: 'package.json', + config: { + renovate: {}, + }, + }, + 'Repository config' + ); expect(res.repoIsOnboarded).toBeTrue(); }); -- GitLab