add more normal test cases for local actions check

action_metadata_test.go

@@ -97,11 +97,31 @@ func TestLocalActionsFindMetadata(t *testing.T) {
 		if err != nil {
 			t.Fatal(err)
 		}
+		testCachedFlag(t, cached, false)
 		if !cmp.Equal(want, have) {
 			t.Fatal(cmp.Diff(want, have))
 		}
-		testCachedFlag(t, cached, false)
 	})
+
+	for _, using := range []string{"docker", "composite"} {
+		a, u := "./"+using, using
+		t.Run(a, func(t *testing.T) {
+			want := &ActionMetadata{
+				Name: "My action",
+				Runs: ActionMetadataRuns{
+					Using: u,
+				},
+			}
+			have, cached, err := c.FindMetadata(a)
+			if err != nil {
+				t.Fatal(err)
+			}
+			testCachedFlag(t, cached, false)
+			if !cmp.Equal(want, have) {
+				t.Fatal(cmp.Diff(want, have))
+			}
+		})
+	}
 }
 
 func TestLocalActionsFindConcurrently(t *testing.T) {

rule_action.go

@@ -187,6 +187,7 @@ func (rule *RuleAction) checkLocalAction(path string, action *ExecAction) {
 	}
 
 	if !cached {
+		rule.Debug("Checking runner metadata of %s action %q at %q", meta.Runs, meta.Name, path)
 		rule.checkLocalActionRunner(path, meta, action.Uses.Pos)
 	}
 

testdata/action_metadata/composite/action.yaml

@@ -0,0 +1,8 @@
+name: 'My action'
+author: 'rhysd <https://rhysd.github.io>'
+description: 'my action'
+
+runs:
+  using: composite
+  steps:
+    - run: echo hello

testdata/action_metadata/docker/Dockerfile

@@ -0,0 +1 @@
+FROM alpine:latest

testdata/action_metadata/docker/action.yaml

@@ -0,0 +1,7 @@
+name: 'My action'
+author: 'rhysd <https://rhysd.github.io>'
+description: 'my action'
+
+runs:
+  using: docker
+  image: Dockerfile