From b40f49a50c47256fe044d95be0a9f920066dddb6 Mon Sep 17 00:00:00 2001
From: Linda_pp <rhysd@users.noreply.github.com>
Date: Sun, 11 Jun 2023 01:09:46 +0900
Subject: [PATCH] Revert "Stronger external checks"

---
 linter.go      |  6 ++--
 linter_test.go | 96 --------------------------------------------------
 2 files changed, 2 insertions(+), 100 deletions(-)

diff --git a/linter.go b/linter.go
index a1ddccdeb..33923d3f4 100644
--- a/linter.go
+++ b/linter.go
@@ -509,8 +509,7 @@ func (l *Linter) check(
 			if err == nil {
 				rules = append(rules, r)
 			} else {
-				l.log("Error in rule \"shellcheck\":", err)
-				return nil, err
+				l.log("Rule \"shellcheck\" was disabled:", err)
 			}
 		} else {
 			l.log("Rule \"shellcheck\" was disabled since shellcheck command name was empty")
@@ -520,8 +519,7 @@ func (l *Linter) check(
 			if err == nil {
 				rules = append(rules, r)
 			} else {
-				l.log("Error in rule \"pyflakes\":", err)
-				return nil, err
+				l.log("Rule \"pyflakes\" was disabled:", err)
 			}
 		} else {
 			l.log("Rule \"pyflakes\" was disabled since pyflakes command name was empty")
diff --git a/linter_test.go b/linter_test.go
index d971617d2..73d47278c 100644
--- a/linter_test.go
+++ b/linter_test.go
@@ -16,102 +16,6 @@ import (
 	"golang.org/x/sys/execabs"
 )
 
-func TestLinterShellcheckCommandNotFound(t *testing.T) {
-	dir := filepath.Join("testdata", "ok")
-
-	es, err := os.ReadDir(dir)
-	if err != nil {
-		panic(err)
-	}
-
-	fs := make([]string, 0, len(es))
-	for _, e := range es {
-		if e.IsDir() {
-			continue
-		}
-		n := e.Name()
-		if strings.HasSuffix(n, ".yaml") || strings.HasSuffix(n, ".yml") {
-			fs = append(fs, filepath.Join(dir, n))
-		}
-	}
-
-	proj := &Project{root: dir}
-
-	shellcheck := "i_am_not_a_program"
-	pyflakes := ""
-
-	for _, f := range fs {
-		t.Run(filepath.Base(f), func(t *testing.T) {
-			opts := LinterOptions{
-				Shellcheck: shellcheck,
-				Pyflakes:   pyflakes,
-			}
-
-			linter, err := NewLinter(io.Discard, &opts)
-			if err != nil {
-				t.Fatal(err)
-			}
-
-			config := Config{}
-			linter.defaultConfig = &config
-
-			errs, err := linter.LintFile(f, proj)
-			if err == nil {
-				t.Fatal("Non existing shellcheck command should fail")
-			}
-			_ = errs
-		})
-	}
-}
-
-func TestLinterPyflakesCommandNotFound(t *testing.T) {
-	dir := filepath.Join("testdata", "ok")
-
-	es, err := os.ReadDir(dir)
-	if err != nil {
-		panic(err)
-	}
-
-	fs := make([]string, 0, len(es))
-	for _, e := range es {
-		if e.IsDir() {
-			continue
-		}
-		n := e.Name()
-		if strings.HasSuffix(n, ".yaml") || strings.HasSuffix(n, ".yml") {
-			fs = append(fs, filepath.Join(dir, n))
-		}
-	}
-
-	proj := &Project{root: dir}
-
-	shellcheck := ""
-	pyflakes := "i_am_not_a_program"
-
-	for _, f := range fs {
-		t.Run(filepath.Base(f), func(t *testing.T) {
-			opts := LinterOptions{
-				Shellcheck: shellcheck,
-				Pyflakes:   pyflakes,
-			}
-
-			linter, err := NewLinter(io.Discard, &opts)
-			if err != nil {
-				t.Fatal(err)
-			}
-
-			config := Config{}
-			linter.defaultConfig = &config
-
-			errs, err := linter.LintFile(f, proj)
-			if err == nil {
-				t.Fatal("Non existing shellcheck command should fail")
-			}
-			_ = errs
-		})
-	}
-}
-
 func TestLinterLintOK(t *testing.T) {
 	dir := filepath.Join("testdata", "ok")