[GitHub] [trafficcontrol] lactose commented on a change in pull request #4099: Correctly handle assigning org servers to ds that have req caps

[GitBox <gi...@apache.org>]

lactose commented on a change in pull request #4099: Correctly handle assigning org servers to ds that have req caps
URL: https://github.com/apache/trafficcontrol/pull/4099#discussion_r344279424
 
 

 ##########
 File path: traffic_ops/traffic_ops_golang/deliveryservice/deliveryservices_required_capabilities_test.go
 ##########
 @@ -78,7 +78,7 @@ func TestCreateDeliveryServicesRequiredCapability(t *testing.T) {
 	mockTenantID(t, mock, 1)
 
 	arrayRows := sqlmock.NewRows([]string{"array"})
-	mock.ExpectQuery("SELECT server FROM deliveryservice_server").WillReturnRows(arrayRows)
+	mock.ExpectQuery("SELECT ds.server FROM deliveryservice_server").WillReturnRows(arrayRows)
 
 Review comment:
   no real comment here, just curious why this needed to be done? should it be documented?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services