[rayci] support select+tag together as a filter

Signed-off-by: can <[email protected]>

raycicmd/make.go

@@ -92,7 +92,11 @@ func makePipeline(repoDir string, config *config, info *buildInfo) (
 
 	var filter *stepFilter
 	if len(info.selects) > 0 && len(config.TagFilterCommand) > 0 {
-		f, err := newSelectAndTagsStepFilter(config.SkipTags, info.selects, config.TagFilterCommand)
+		f, err := newSelectAndTagsStepFilter(
+			config.SkipTags,
+			info.selects,
+			config.TagFilterCommand,
+		)
 		if err != nil {
 			return nil, fmt.Errorf("run tag filter command: %w", err)
 		}

raycicmd/step_filter_test.go

@@ -204,3 +204,26 @@ func TestStepFilter_selects(t *testing.T) {
 		}
 	}
 }
+
+func TestStepFilter_selectsAndTags(t *testing.T) {
+	filter, _ := newSelectAndTagsStepFilter([]string{"disabled"}, []string{"foo", "bar"}, []string{"echo", "tune"})
+	for _, node := range []*stepNode{
+		{key: "foo"},
+		{id: "foo", tags: []string{"tune"}},
+		{id: "bar"},
+	} {
+		if !filter.accept(node) {
+			t.Errorf("miss %+v", node)
+		}
+	}
+
+	for _, node := range []*stepNode{
+		{id: "foo", tags: []string{"not_tune"}},
+		{id: "bar", tags: []string{"tune_not"}},
+		{key: "w00t"},
+	} {
+		if filter.accept(node) {
+			t.Errorf("miss %+v", node)
+		}
+	}
+}