Merge pull request #147 from sgotti/fix_when_reftype_tag

when: fix typo in tag matching

services/types/when.go

@@ -58,7 +58,7 @@ func MatchWhen(when *When, refType itypes.RunRefType, branch, tag, ref string) b
 			}
 		}
 		// test only if tag is not empty, if empty mean that we are not in a tag
-		if refType == itypes.RunRefTypeBranch && when.Tag != nil && tag != "" {
+		if refType == itypes.RunRefTypeTag && when.Tag != nil && tag != "" {
 			// first check includes and override with excludes
 			if matchCondition(when.Tag.Include, tag) {
 				include = true

services/types/when_test.go

@@ -284,7 +284,7 @@ func TestMatchWhen(t *testing.T) {
 			out:     false,
 		},
 		{
-			name: "test only matching reftype",
+			name: "test only matching branch reftype",
 			when: &When{
 				Branch: &WhenConditions{
 					Include: []WhenCondition{
@@ -298,6 +298,21 @@ func TestMatchWhen(t *testing.T) {
 			tag:    "master",
 			out:    false,
 		},
+		{
+			name: "test only matching tag reftype",
+			when: &When{
+				Tag: &WhenConditions{
+					Include: []WhenCondition{
+						{Type: WhenConditionTypeSimple, Match: "master"},
+					},
+				},
+			},
+			refType: itypes.RunRefTypeBranch,
+			branch:  "master",
+			// we provide also a value to tag (should not be done)
+			tag: "master",
+			out: false,
+		},
 	}
 
 	for _, tt := range tests {