fix fuzzing tests broken by API changes

rhysd · Feb 25, 2024 · b27d697 · b27d697
1 parent c6b0678
commit b27d697
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/fuzz/check.go b/fuzz/check.go
@@ -24,14 +24,18 @@ func FuzzCheck(data []byte) int {
 		actionlint.NewRuleMatrix(),
 		actionlint.NewRuleCredentials(),
 		actionlint.NewRuleShellName(),
-		actionlint.NewRuleRunnerLabel([]string{}),
+		actionlint.NewRuleRunnerLabel(),
 		actionlint.NewRuleEvents(),
 		actionlint.NewRuleGlob(),
 		actionlint.NewRuleJobNeeds(),
 		actionlint.NewRuleAction(ac),
 		actionlint.NewRuleEnvVar(),
 		actionlint.NewRuleID(),
 		actionlint.NewRuleExpression(ac, wc),
+		actionlint.NewRuleWorkflowCall("test.yaml", wc),
+		actionlint.NewRulePermissions(),
+		actionlint.NewRuleDeprecatedCommands(),
+		actionlint.NewRuleIfCond(),
 	}
 
 	v := actionlint.NewVisitor()

diff --git a/fuzz/expr.go b/fuzz/expr.go
@@ -20,7 +20,7 @@ func FuzzExprParse(data []byte) int {
 		return 0
 	}
 
-	c := actionlint.NewExprSemanticsChecker(true)
+	c := actionlint.NewExprSemanticsChecker(true, nil)
 	c.Check(e)
 
 	return 1

diff --git a/fuzz/parse.go b/fuzz/parse.go
@@ -8,14 +8,15 @@ import (
 )
 
 func canParseByGoYAML(data []byte) (ret bool) {
+	ret = true
 	defer func() {
 		if err := recover(); err != nil {
 			ret = false
 		}
 	}()
 	var n yaml.Node
 	yaml.Unmarshal(data, &n)
-	return true
+	return
 }
 
 func FuzzParse(data []byte) int {