broker default unit test add

knative · Dec 12, 2022 · c481170 · c481170
1 parent 2ad62d2
commit c481170
Show file tree

Hide file tree

Showing 2 changed files with 353 additions and 51 deletions.
diff --git a/pkg/apis/config/defaults.go b/pkg/apis/config/defaults.go
@@ -181,13 +181,11 @@ func getBrokerClasses(d *Defaults) map[string]*BrokerConfigSpec {
 
 // matchBrokerClass find the corresponding brokerConfig for a given brokerClass
 func matchBrokerClass(brokerClass string, brokerClasses map[string]*BrokerConfigSpec) *BrokerConfig {
-	for bClass, bCSpec := range brokerClasses {
-		if bClass == brokerClass {
-			var bConfig BrokerConfig
-			bConfig.KReference = bCSpec.Spec.Config
-			bConfig.Delivery = bCSpec.Spec.Delivery
-			return &bConfig
-		}
+	var bConfig BrokerConfig
+	if bCSpec, ok := brokerClasses[brokerClass]; ok {
+		bConfig.KReference = bCSpec.Spec.Config
+		bConfig.Delivery = bCSpec.Spec.Delivery
+		return &bConfig
 	}
 	return nil
 }