chore: remove --skip-deps flags from existing test cases

Trong Nhan Mai · Trong Nhan Mai · commit c02e99360063 · 2024-08-21T14:46:48.000+10:00
diff --git a/tests/integration/cases/all_checks_excluded/test.yaml b/tests/integration/cases/all_checks_excluded/test.yaml
@@ -15,5 +15,4 @@ steps:
     command_args:
     - -rp
     - https://github.com/apache/maven
-    - --skip-deps
   expect_fail: true
diff --git a/tests/integration/cases/apache_maven_analyzing_a_clone_from_another_local_repo/test.yaml b/tests/integration/cases/apache_maven_analyzing_a_clone_from_another_local_repo/test.yaml
@@ -34,7 +34,6 @@ steps:
     - master
     - -d
     - 3fc399318edef0d5ba593723a24fff64291d6f9b
-    - --skip-deps
 # We don't compare the report content because the remote_path fields in the reports are nondeterministic when running
 # this test locally and running it in the GitHub Actions runner. We only check if the reports are generated as
 # expected without the issue described in https://github.com/oracle/macaron/issues/116.
diff --git a/tests/integration/cases/apache_maven_local_paths_in_configuration_without_dep_resolution/test.yaml b/tests/integration/cases/apache_maven_local_paths_in_configuration_without_dep_resolution/test.yaml
@@ -29,7 +29,6 @@ steps:
     command_args:
     - -c
     - config.yaml
-    - --skip-deps
 - name: Run verify-policy for apache/maven
   kind: verify
   options:
diff --git a/tests/integration/cases/apache_maven_local_repo/test.yaml b/tests/integration/cases/apache_maven_local_repo/test.yaml
@@ -30,7 +30,6 @@ steps:
     - master
     - -d
     - 3fc399318edef0d5ba593723a24fff64291d6f9b
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/apache_maven_purl_repo_path/test.yaml b/tests/integration/cases/apache_maven_purl_repo_path/test.yaml
@@ -20,7 +20,6 @@ steps:
     - master
     - -d
     - 3fc399318edef0d5ba593723a24fff64291d6f9b
-    - --skip-deps
 # This to check if run_macaron.sh cleans up the token file after the test case runs for the docker image.
 - name: Check the token file doesn't exist.
   kind: shell
diff --git a/tests/integration/cases/apache_maven_using_default_template_file_as_input_template/test.yaml b/tests/integration/cases/apache_maven_using_default_template_file_as_input_template/test.yaml
@@ -20,7 +20,6 @@ steps:
     - 3fc399318edef0d5ba593723a24fff64291d6f9b
     - -g
     - macaron.html
-    - --skip-deps
 - name: Run verify-policy
   kind: verify
   options:
diff --git a/tests/integration/cases/apache_maven_yaml_input_no_deps_and_skip_deps/test.yaml b/tests/integration/cases/apache_maven_yaml_input_no_deps_and_skip_deps/test.yaml
@@ -14,7 +14,6 @@ steps:
     command_args:
     - -c
     - maven_config.yaml
-    - --skip-deps
 - name: Compare dependency report
   kind: compare
   options:
diff --git a/tests/integration/cases/apache_maven_yaml_input_skip_deps/test.yaml b/tests/integration/cases/apache_maven_yaml_input_skip_deps/test.yaml
@@ -15,7 +15,6 @@ steps:
     command_args:
     - -c
     - maven_config.yaml
-    - --skip-deps
 - name: Run macaron verify-policy for maven
   kind: verify
   options:
diff --git a/tests/integration/cases/behnazh-w_example-maven-app/test.yaml b/tests/integration/cases/behnazh-w_example-maven-app/test.yaml
@@ -24,7 +24,6 @@ steps:
     - pkg:maven/io.github.behnazh-w.demo/example-maven-app@1.0-SNAPSHOT?type=jar
     - --repo-path
     - example-maven-app
-    - --skip-deps
     expectation: witness_provenance_expectation.cue
     provenance: witness_provenance.jsonl
 - name: Run macaron analyze on the remote repository with GitHub provenance.
@@ -33,7 +32,6 @@ steps:
     command_args:
     - --package-url
     - pkg:maven/io.github.behnazh-w.demo/example-maven-app@1.0?type=jar
-    - --skip-deps
     expectation: github_provenance_expectation.cue
     provenance: github_provenance.jsonl
 - name: Run macaron verify-policy
@@ -60,21 +58,18 @@ steps:
     - https://github.com/behnazh-w/example-maven-app
     - -d
     - 2deca75ed5dd365eaf1558a82347b1f11306135f
-    - --skip-deps
     provenance: github_provenance.jsonl
 - name: Run macaron analyze, validate user input of repo and commit (via purl) vs provenance
   kind: analyze
   options:
     command_args:
     - -purl
     - pkg:github/behnazh-w/example-maven-app@2deca75
-    - --skip-deps
     provenance: github_provenance.jsonl
 - name: Run macaron analyze, validate user input of repo and commit (via purl with tag) vs provenance
   kind: analyze
   options:
     command_args:
     - -purl
     - pkg:github/behnazh-w/example-maven-app@1.0
-    - --skip-deps
     provenance: github_provenance.jsonl
diff --git a/tests/integration/cases/facebook_yoga_yarn_classic/test.yaml b/tests/integration/cases/facebook_yoga_yarn_classic/test.yaml
@@ -19,7 +19,6 @@ steps:
     - main
     - -d
     - f8e2bc0875c145c429d0e865c9b83a40f65b3070
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/gitlab_tinyMediaManager/test.yaml b/tests/integration/cases/gitlab_tinyMediaManager/test.yaml
@@ -18,7 +18,6 @@ steps:
     - main
     - -d
     - cca6b67a335074eca42136556f0a321f75dc4f48
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/gitlab_tinyMediaManager_purl/test.yaml b/tests/integration/cases/gitlab_tinyMediaManager_purl/test.yaml
@@ -20,7 +20,6 @@ steps:
     - main
     - -d
     - cca6b67a335074eca42136556f0a321f75dc4f48
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/google_guava/test.yaml b/tests/integration/cases/google_guava/test.yaml
@@ -14,7 +14,6 @@ steps:
     command_args:
     - -purl
     - pkg:maven/com.google.guava/guava@32.1.2-jre?type=jar
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/invalid_provenance_file/test.yaml b/tests/integration/cases/invalid_provenance_file/test.yaml
@@ -15,5 +15,4 @@ steps:
     command_args:
     - -rp
     - https://github.com/apache/maven
-    - --skip-deps
   expect_fail: true
diff --git a/tests/integration/cases/invalid_purl/test.yaml b/tests/integration/cases/invalid_purl/test.yaml
@@ -17,5 +17,4 @@ steps:
     - invalid-purl
     - --repo-path
     - https://github.com/apache/maven
-    - --skip-deps
   expect_fail: true
diff --git a/tests/integration/cases/jackson_databind_with_purl_and_no_deps/test.yaml b/tests/integration/cases/jackson_databind_with_purl_and_no_deps/test.yaml
@@ -14,7 +14,6 @@ steps:
     command_args:
     - -purl
     - pkg:maven/com.fasterxml.jackson.core/jackson-databind@2.14.0-rc1
-    - --skip-deps
 - name: Run verify-policy
   kind: verify
   options:
diff --git a/tests/integration/cases/jenkinsci_plotplugin/test.yaml b/tests/integration/cases/jenkinsci_plotplugin/test.yaml
@@ -18,7 +18,6 @@ steps:
     - master
     - -d
     - 55b059187e252b35ac0d6cb52268833ee1bb7380
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/micronaut-projects_micronaut-core/test.yaml b/tests/integration/cases/micronaut-projects_micronaut-core/test.yaml
@@ -16,7 +16,6 @@ steps:
     command_args:
     - -purl
     - pkg:maven/io.micronaut/micronaut-core@4.2.3
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/micronaut-projects_micronaut-test/test.yaml b/tests/integration/cases/micronaut-projects_micronaut-test/test.yaml
@@ -14,7 +14,6 @@ steps:
     command_args:
     - -c
     - micronaut_test_config.yaml
-    - --skip-deps
 - name: Compare dependency report
   kind: compare
   options:
diff --git a/tests/integration/cases/missing_template_file/test.yaml b/tests/integration/cases/missing_template_file/test.yaml
@@ -14,7 +14,6 @@ steps:
     command_args:
     - -rp
     - https://github.com/apache/maven
-    - --skip-deps
     - -g
     - does/not/exist
   expect_fail: true
diff --git a/tests/integration/cases/no_branch_or_commit/test.yaml b/tests/integration/cases/no_branch_or_commit/test.yaml
@@ -17,5 +17,4 @@ steps:
     - pkg:maven/apache/maven
     - --repo-path
     - https://github.com/apache/maven
-    - --skip-deps
   expect_fail: true
diff --git a/tests/integration/cases/no_github_token/test.yaml b/tests/integration/cases/no_github_token/test.yaml
@@ -14,8 +14,7 @@ steps:
   options:
     command_args:
     - --repo-path
-    - https://github.com/apache/maven --skip-deps
-    - --skip-deps
+    - https://github.com/apache/maven
   env:
     GITHUB_TOKEN:
   expect_fail: true
diff --git a/tests/integration/cases/onu-ui_onu-ui_pnpm/test.yaml b/tests/integration/cases/onu-ui_onu-ui_pnpm/test.yaml
@@ -19,7 +19,6 @@ steps:
     - main
     - -d
     - e3f2825c3940002a920d65476116a64684b3d95e
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/ossf_scorecard/test.yaml b/tests/integration/cases/ossf_scorecard/test.yaml
@@ -17,7 +17,6 @@ steps:
     command_args:
     - --package-url
     - pkg:github/ossf/scorecard@v4.13.1
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/purl_of_nonexistent_artifact/test.yaml b/tests/integration/cases/purl_of_nonexistent_artifact/test.yaml
@@ -14,7 +14,6 @@ steps:
     command_args:
     - -purl
     - pkg:maven/com.example/nonexistent@1.0.0
-    - --skip-deps
 - name: Run macaron verify-policy
   kind: verify
   options:
diff --git a/tests/integration/cases/sigstore_mock/test.yaml b/tests/integration/cases/sigstore_mock/test.yaml
@@ -21,7 +21,6 @@ steps:
     - main
     - -d
     - ebdcfdfbdfeb9c9aeee6df53674ef230613629f5
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/sigstore_sget/test.yaml b/tests/integration/cases/sigstore_sget/test.yaml
@@ -19,7 +19,6 @@ steps:
     - main
     - -d
     - 99e7b91204d391ccc76507f7079b6d2a7957489e
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/slsa-framework_slsa-verifier/test.yaml b/tests/integration/cases/slsa-framework_slsa-verifier/test.yaml
@@ -21,7 +21,6 @@ steps:
     - main
     - -d
     - e6428d7da594455a4c2b7f24907fec421a5e0e95
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/slsa-framework_slsa-verifier_explicit_provenance_provided/test.yaml b/tests/integration/cases/slsa-framework_slsa-verifier_explicit_provenance_provided/test.yaml
@@ -19,7 +19,6 @@ steps:
     - https://github.com/slsa-framework/slsa-verifier
     - -d
     - 6fb4f7e2dd9c2f5d4f55fa88f6796278a7bba6d6
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
@@ -35,7 +34,6 @@ steps:
     - https://github.com/slsa-framework/slsa-verifier
     - -d
     - 6fb4f7e2dd9c2f5d4f55fa88f6796278a7bba6d6
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/snakeyaml_unsupported_git_service/test.yaml b/tests/integration/cases/snakeyaml_unsupported_git_service/test.yaml
@@ -28,7 +28,6 @@ steps:
     - snakeyaml
     - -d
     - a34989252e6f59e36a3aaf788a903b7a37a73d33
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/test_analyzing_an_invalid_local_repo_path/test.yaml b/tests/integration/cases/test_analyzing_an_invalid_local_repo_path/test.yaml
@@ -17,5 +17,4 @@ steps:
     command_args:
     - -rp
     - path/to/invalid/repo
-    - --skip-deps
   expect_fail: true
diff --git a/tests/integration/cases/test_using_a_repo_path_outside_of_local_repos_dir/test.yaml b/tests/integration/cases/test_using_a_repo_path_outside_of_local_repos_dir/test.yaml
@@ -21,5 +21,4 @@ steps:
     command_args:
     - -rp
     - ../
-    - --skip-deps
   expect_fail: true
diff --git a/tests/integration/cases/test_using_invalid_local_repos_dir/test.yaml b/tests/integration/cases/test_using_invalid_local_repos_dir/test.yaml
@@ -17,5 +17,4 @@ steps:
     command_args:
     - -rp
     - apache/maven
-    - --skip-deps
   expect_fail: true
diff --git a/tests/integration/cases/test_using_local_repo_with_no_commit/test.yaml b/tests/integration/cases/test_using_local_repo_with_no_commit/test.yaml
@@ -21,5 +21,4 @@ steps:
     command_args:
     - -rp
     - empty_repo
-    - --skip-deps
   expect_fail: true
diff --git a/tests/integration/cases/timyarkov_docker_test/test.yaml b/tests/integration/cases/timyarkov_docker_test/test.yaml
@@ -19,7 +19,6 @@ steps:
     - main
     - -d
     - 404a51a2f38c4470af6b32e4e00b5318c2d7c0cc
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/tutorial_npm_verify_provenance_semver/test.yaml b/tests/integration/cases/tutorial_npm_verify_provenance_semver/test.yaml
@@ -14,7 +14,6 @@ steps:
     command_args:
     - -purl
     - pkg:npm/semver@7.6.2
-    - --skip-deps
 - name: Verify checks for semver@7.6.2
   kind: verify
   options:
@@ -25,7 +24,6 @@ steps:
     command_args:
     - -purl
     - pkg:npm/semver@7.6.0
-    - --skip-deps
 - name: Verify checks for all 7.6.x semver runs
   kind: verify
   options:
@@ -36,7 +34,6 @@ steps:
     command_args:
     - -purl
     - pkg:npm/semver@1.0.0
-    - --skip-deps
 - name: Verify checks for all semver runs
   kind: verify
   options:
diff --git a/tests/integration/cases/uiv-lib_uiv/test.yaml b/tests/integration/cases/uiv-lib_uiv/test.yaml
@@ -19,7 +19,6 @@ steps:
     - dev
     - -d
     - 057b25b4db0913edab4cf728c306085e6fc20d49
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/urllib3_expectation_dir/test.yaml b/tests/integration/cases/urllib3_expectation_dir/test.yaml
@@ -18,7 +18,6 @@ steps:
     - pkg:pypi/urllib3@2.0.0a1
     - --provenance-expectation
     - expectation
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/urllib3_expectation_file/test.yaml b/tests/integration/cases/urllib3_expectation_file/test.yaml
@@ -17,7 +17,6 @@ steps:
     command_args:
     - -purl
     - pkg:pypi/urllib3@2.0.0a1
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/urllib3_invalid_expectation/test.yaml b/tests/integration/cases/urllib3_invalid_expectation/test.yaml
@@ -17,7 +17,6 @@ steps:
     command_args:
     - -purl
     - pkg:pypi/urllib3@2.0.0a1
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
diff --git a/tests/integration/cases/urllib3_no_tag/test.yaml b/tests/integration/cases/urllib3_no_tag/test.yaml
@@ -18,7 +18,6 @@ steps:
     - main
     - --digest
     - 87a0ecee6e691fe5ff93cd000c0158deebef763b
-    - --skip-deps
 - name: Run macaron verify-policy to verify failed check
   kind: verify
   options:
diff --git a/tests/integration/cases/wojtekmaj_reactpdf_yarn_modern/test.yaml b/tests/integration/cases/wojtekmaj_reactpdf_yarn_modern/test.yaml
@@ -19,7 +19,6 @@ steps:
     - main
     - -d
     - be18436b7be827eb993b2e1e4bd9230dd835a9a3
-    - --skip-deps
 - name: Run macaron verify-policy to verify passed/failed checks
   kind: verify
   options:
