fix: et v1beta3 does not have broker set by default (#8079)

* fix: et v1beta3 does not have broker set by default

Signed-off-by: Calum Murray <[email protected]>

* fix(text): unit tests no longer expect default broker

Signed-off-by: Calum Murray <[email protected]>

* cleanup: test name reflects new behaviour of allowing nil references

Signed-off-by: Calum Murray <[email protected]>

---------

Signed-off-by: Calum Murray <[email protected]>

Author: Cali0707

pkg/apis/eventing/v1beta1/eventtype_defaults.go

@@ -28,9 +28,6 @@ func (et *EventType) SetDefaults(ctx context.Context) {
 }
 
 func (ets *EventTypeSpec) SetDefaults(ctx context.Context) {
-	if ets.Reference == nil && ets.Broker == "" {
-		ets.Broker = "default"
-	}
 	if ets.Reference != nil {
 		ets.Reference.SetDefaults(ctx)
 	}

pkg/apis/eventing/v1beta1/eventtype_defaults_test.go

@@ -35,12 +35,10 @@ func TestEventTypeDefaults(t *testing.T) {
 		"nil spec": {
 			initial: EventType{},
 			expected: EventType{
-				Spec: EventTypeSpec{
-					Broker: "default",
-				},
+				Spec: EventTypeSpec{},
 			},
 		},
-		"broker empty": {
+		"default broker reference": {
 			initial: EventType{
 				Spec: EventTypeSpec{
 					Type:   "test-type",
@@ -53,7 +51,6 @@ func TestEventTypeDefaults(t *testing.T) {
 				Spec: EventTypeSpec{
 					Type:   "test-type",
 					Source: testSource,
-					Broker: "default",
 					Schema: testSchema,
 				},
 			},
@@ -70,7 +67,6 @@ func TestEventTypeDefaults(t *testing.T) {
 				Spec: EventTypeSpec{
 					Type:   "test-type",
 					Source: testSource,
-					Broker: "default",
 					Schema: testSchema,
 				},
 			},