diff --git a/.github/workflows/pr-test.yml b/.github/workflows/pr-test.yml index 28f4e320f6..e3ecec4603 100644 --- a/.github/workflows/pr-test.yml +++ b/.github/workflows/pr-test.yml @@ -139,6 +139,7 @@ jobs: - uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4 - uses: azure/setup-helm@5119fcb9089d432beecbf79bb2c7915207344b78 # v3 with: + version: "v3.19.1" token: ${{ secrets.GITHUB_TOKEN }} - uses: imranismail/setup-kustomize@a76db1c6419124d51470b1e388c4b29476f495f1 # v2 if: ${{ runner.os != 'windows' }} diff --git a/checkov/terraform/graph_builder/variable_rendering/evaluate_terraform.py b/checkov/terraform/graph_builder/variable_rendering/evaluate_terraform.py index 0e6f3b3918..87e97a3410 100644 --- a/checkov/terraform/graph_builder/variable_rendering/evaluate_terraform.py +++ b/checkov/terraform/graph_builder/variable_rendering/evaluate_terraform.py @@ -126,6 +126,7 @@ def replace_string_value(original_str: Any, str_to_replace: str, replaced_value: return original_str if keep_origin else str_to_replace string_without_interpolation = remove_interpolation(original_str, str_to_replace, escape_unrendered=False) + return string_without_interpolation.replace(str_to_replace, str(replaced_value)) diff --git a/integration_tests/test_checkov_json_report.py b/integration_tests/test_checkov_json_report.py index 6fef287dc0..cb33bd3c1b 100644 --- a/integration_tests/test_checkov_json_report.py +++ b/integration_tests/test_checkov_json_report.py @@ -54,6 +54,9 @@ def validate_report(self, report_path): self.validate_report_not_empty(data) def validate_report_not_empty(self, report): + if "summary" not in report: + print("ERROR - summary not found in report, found keys: {keys}".format(keys=report)) + self.assertEqual(list(report.keys()), ["summary", "results"]) self.assertEqual(report["summary"]["parsing_errors"], 0, f"expecting 0 parsing errors but got: {report['results']['parsing_errors']}") self.assertGreater(report["summary"]["failed"], 1,