Support multiple HSes on runtime.SkipIf

Author: kegsay

runtime/hs.go

@@ -9,7 +9,7 @@ const (
 
 var Homeserver string
 
-// Skip the test (via t.Skipf) if the homeserver being tested matches `hs`, else return.
+// Skip the test (via t.Skipf) if the homeserver being tested matches one of the homeservers, else return.
 //
 // The homeserver being tested is detected via the presence of a `*_blacklist` tag e.g:
 //   go test -tags="dendrite_blacklist"
@@ -18,17 +18,20 @@ var Homeserver string
 // implementation is added, a respective `hs_$name.go` needs to be created in this directory. This
 // file pairs together the tag name with a string constant declared in this package
 // e.g. dendrite_blacklist == runtime.Dendrite
-func SkipIf(t *testing.T, hs string) {
+func SkipIf(t *testing.T, hses ...string) {
 	t.Helper()
-	if Homeserver == hs {
-		t.Skipf("skipped on %s", hs)
+	for _, hs := range hses {
+		if Homeserver == hs {
+			t.Skipf("skipped on %s", hs)
+			return
+		}
 	}
 	if Homeserver == "" {
 		// they ran Complement without a blacklist so it's impossible to know what HS they are
 		// running, warn them.
 		t.Logf(
-			"WARNING: %s called runtime.SkipIf(%s) but Complement doesn't know which HS is running as it was run without a *_blacklist tag: executing test.",
-			t.Name(), hs,
+			"WARNING: %s called runtime.SkipIf(%v) but Complement doesn't know which HS is running as it was run without a *_blacklist tag: executing test.",
+			t.Name(), hses,
 		)
 	}
 }