tiny cleanup

Signed-off-by: Sunjay Bhatia <[email protected]>

Author: sunjayBhatia

internal/dag/accessors_test.go

@@ -291,6 +291,9 @@ func TestGetServiceClusters(t *testing.T) {
 			},
 		},
 	}
+	// We should only get one cluster since the other is for an ExternalName
+	// service.
+	assert.Len(t, d.GetServiceClusters(), 1)
 
 	dagWithMirror := &DAG{
 		Listeners: map[string]*Listener{
@@ -300,7 +303,6 @@ func TestGetServiceClusters(t *testing.T) {
 						Routes: map[string]*Route{
 							"foo": {
 								Clusters: []*Cluster{
-									{Upstream: &Service{ExternalName: "bar.com"}},
 									{Upstream: &Service{}},
 								},
 								MirrorPolicies: []*MirrorPolicy{
@@ -317,10 +319,6 @@ func TestGetServiceClusters(t *testing.T) {
 			},
 		},
 	}
-	// We should only get one cluster since the other is for an ExternalName
-	// service.
-	assert.Len(t, d.GetServiceClusters(), 1)
-
-	// We should get one two clusters since we have mirrorPolicies.
+	// We should get two clusters since we have mirrorPolicies.
 	assert.Len(t, dagWithMirror.GetServiceClusters(), 2)
 }