fix e2e test

slalombuild · Mar 7, 2024 · 92e577f · 92e577f
1 parent 040902d
commit 92e577f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/tests/end-to-end/testpreservehooks.bats b/tests/end-to-end/testpreservehooks.bats
@@ -9,18 +9,21 @@ setup() {
   mkdir -p $MOCK_REPO
   echo '# Existing YAML Contents should persist' > $MOCK_REPO/.pre-commit-config.yaml
   echo 'repos:' >> $MOCK_REPO/.pre-commit-config.yaml
-  echo '-   repo: https://github.com/hhatto/autopep8'
+  echo '-   repo: https://github.com/hhatto/autopep8' >> $MOCK_REPO/.pre-commit-config.yaml
   echo '    rev: v2.0.4' >> $MOCK_REPO/.pre-commit-config.yaml
   echo '    hooks:' >> $MOCK_REPO/.pre-commit-config.yaml
   echo '    -   id: autopep8' >> $MOCK_REPO/.pre-commit-config.yaml
+  echo 'fail_fast: false' >> $MOCK_REPO/.pre-commit-config.yaml
   echo 'print("hello world!")' > $MOCK_REPO/hw.py
   run git init $MOCK_REPO
 }
 
 @test "can preserve pre-existing hooks" {
     run python secureli/main.py init -y  --directory $MOCK_REPO
-    run grep '# Existing YAML Contents should persist' $MOCK_REPO/.secureli/.pre-commit-config.yaml
-    assert_output --partial '# Existing YAML Contents should persist'
+    run grep 'https://github.com/hhatto/autopep8' $MOCK_REPO/.secureli/.pre-commit-config.yaml
+    assert_output --partial 'https://github.com/hhatto/autopep8'
+    run grep 'fail_fast: false' $MOCK_REPO/.secureli/.pre-commit-config.yaml
+    assert_output --partial 'fail_fast: false'
 }
 
 teardown() {

diff --git a/tests/services/test_language_support.py b/tests/services/test_language_support.py
@@ -459,7 +459,7 @@ def test_build_pre_commit_respects_existing_pre_commit_config(
     with (
         patch("builtins.open", mock_open(read_data="data")),
         patch.object(
-            yaml, "safe_load", return_value={"repos": [{"autoPep8": {"version": 0}}]}
+            yaml, "safe_load", return_value={"repos": [{"autopep8": {"version": 0}}]}
         ),
     ):
         result = language_support_service.build_pre_commit_config(
@@ -468,9 +468,9 @@ def test_build_pre_commit_respects_existing_pre_commit_config(
         assert result.successful == True
         assert result.config_data == {
             "repos": [
-                {"autoPep8": {"version": 0}},
-                {"autoPep8": {"version": 0}},
-                {"autoPep8": {"version": 0}},
+                {"autopep8": {"version": 0}},
+                {"autopep8": {"version": 0}},
+                {"autopep8": {"version": 0}},
             ]
         }