fix: fail if ProviderVersion not provided but BrokerURL is supplied

types/verify_request.go

@@ -150,6 +150,10 @@ func (v *VerifyRequest) Validate() error {
 		v.Args = append(v.Args, "--broker-token", v.BrokerToken)
 	}
 
+	if v.BrokerURL != "" && v.ProviderVersion == "" {
+		return errors.New("both 'ProviderVersion' must be supplied if 'BrokerURL' given")
+	}
+
 	if v.ProviderVersion != "" {
 		v.Args = append(v.Args, "--provider_app_version", v.ProviderVersion)
 	}

types/verify_request_test.go

@@ -40,6 +40,22 @@ func TestVerifyRequest_NoBaseURL(t *testing.T) {
 }
 
 func TestVerifyRequest_BrokerUsernameWithoutPassword(t *testing.T) {
+	r := VerifyRequest{
+		PactURLs:        []string{"http://localhost:1234/path/to/pact"},
+		ProviderBaseURL: "http://localhost:8080",
+		BrokerURL:       "http://localhost:1234",
+		ProviderVersion: "1.0.0.",
+		BrokerPassword:  "1234",
+	}
+
+	err := r.Validate()
+
+	if err == nil {
+		t.Fatal("want error, got nil")
+	}
+}
+
+func TestVerifyRequest_BrokerURLWithoutVersion(t *testing.T) {
 	r := VerifyRequest{
 		PactURLs:        []string{"http://localhost:1234/path/to/pact"},
 		ProviderBaseURL: "http://localhost:8080",