Skip to content

Commit d423bc5

Browse files
fix: input queries and bom params (#165)
1 parent 7429734 commit d423bc5

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

‎entrypoint.sh‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ fi
2727
[[ ! -z "$INPUT_FAIL_ON" ]] && FAIL_ON_PARAM="--fail-on $INPUT_FAIL_ON"
2828
[[ ! -z "$INPUT_TIMEOUT" ]] && TIMEOUT_PARAM="--timeout $INPUT_TIMEOUT"
2929
[[ ! -z "$INPUT_PROFILING" ]] && PROFILING_PARAM="--profiling $INPUT_PROFILING"
30-
[[ ! -z "$INPUT_BOM" ]] && BOM_PARAM="-m $INPUT_PROFILING"
31-
[[ ! -z "$INPUT_INCLUDE_QUERIES" ]] && INCLUDE_QUERIES_PARAM="-i $INPUT_PROFILING"
30+
[[ ! -z "$INPUT_BOM" ]] && BOM_PARAM="-m"
31+
[[ ! -z "$INPUT_INCLUDE_QUERIES" ]] && INCLUDE_QUERIES_PARAM="-i $INPUT_INCLUDE_QUERIES"
3232
[[ ! -z "$INPUT_DISABLE_SECRETS" ]] && DISABLE_SECRETS_PARAM="--disable-secrets"
3333
[[ ! -z "$INPUT_DISABLE_FULL_DESCRIPTIONS" ]] && DISABLE_FULL_DESCRIPTIONS_PARAM="--disable-full-descriptions"
3434
[[ ! -z "$INPUT_LIBRARIES_PATH" ]] && LIBRARIES_PATH_PARAM="-b $INPUT_LIBRARIES_PATH"

‎test/e2e/entrypoint.test.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ test('inputs mapped to KICS flags', async (t) => {
102102
})
103103
}
104104

105-
await t.test('include_queries -> -i <ids>', knownBug('the flag is built from $INPUT_PROFILING instead of $INPUT_INCLUDE_QUERIES (#98)'), async (t) => {
105+
await t.test('include_queries -> -i <ids>', async (t) => {
106106
const run = await runEntrypoint(t, { inputs: { include_queries: '229588ef-8fde-40c8-8756-f4f2b5825ded' } })
107107
assert.equal(flagValue(run.args, '-i'), '229588ef-8fde-40c8-8756-f4f2b5825ded')
108108
})
@@ -112,7 +112,7 @@ test('inputs mapped to KICS flags', async (t) => {
112112
assert.ok(run.args.includes('-m'), run.args.join(' '))
113113
})
114114

115-
await t.test('bom and include_queries do not borrow the value of profiling', knownBug('both flags are built from $INPUT_PROFILING (see #98)'), async (t) => {
115+
await t.test('bom and include_queries do not borrow the value of profiling', async (t) => {
116116
const run = await runEntrypoint(t, { inputs: { bom: 'true', include_queries: 'q-1', profiling: 'CPU' } })
117117
assert.equal(flagValue(run.args, '--profiling'), 'CPU')
118118
assert.notEqual(flagValue(run.args, '-m'), 'CPU', 'bom is a switch, it takes no value')

0 commit comments

Comments
 (0)