Skip to content

Commit f4fca70

Browse files
committed
Do not mutate originalUserInput
1 parent 51d74ac commit f4fca70

File tree

3 files changed

+6
-16
lines changed

3 files changed

+6
-16
lines changed

lib/init-action.js

Lines changed: 0 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/config-utils.test.ts

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -197,20 +197,18 @@ test("load code quality config", async (t) => {
197197
}),
198198
);
199199

200-
const userConfig: configUtils.UserConfig = {
201-
"disable-default-queries": true,
202-
queries: [{ uses: "code-quality" }],
203-
"query-filters": [],
204-
};
205-
206200
// And the config we expect it to result in
207201
const expectedConfig: configUtils.Config = {
208202
analysisKinds: [AnalysisKind.CodeQuality],
209203
languages: [KnownLanguage.actions],
210204
buildMode: undefined,
205+
originalUserInput: {},
211206
// This gets set because we only have `AnalysisKind.CodeQuality`
212-
originalUserInput: userConfig,
213-
computedConfig: userConfig,
207+
computedConfig: {
208+
"disable-default-queries": true,
209+
queries: [{ uses: "code-quality" }],
210+
"query-filters": [],
211+
},
214212
tempDir,
215213
codeQLCmd: codeql.getPath(),
216214
gitHubVersion: githubVersion,

src/config-utils.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1145,11 +1145,6 @@ export async function initConfig(inputs: InitConfigInputs): Promise<Config> {
11451145
const queries = codeQualityQueries.map((v) => ({ uses: v }));
11461146

11471147
// Set the query customisation options for Code Quality only analysis.
1148-
config.originalUserInput["disable-default-queries"] = true;
1149-
config.originalUserInput.queries = queries;
1150-
config.originalUserInput["query-filters"] = [];
1151-
1152-
// Update the computed configuration for the call to `getOverlayDatabaseMode`.
11531148
config.computedConfig["disable-default-queries"] = true;
11541149
config.computedConfig.queries = queries;
11551150
config.computedConfig["query-filters"] = [];

0 commit comments

Comments
 (0)