You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficcontrol.apache.org by mi...@apache.org on 2018/04/02 14:00:03 UTC

[incubator-trafficcontrol] branch master updated: fixed the API tests

This is an automated email from the ASF dual-hosted git repository.

mitchell852 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-trafficcontrol.git


The following commit(s) were added to refs/heads/master by this push:
     new fdb38dc  fixed the API tests
fdb38dc is described below

commit fdb38dc998d7b422260d044a40c388bd2ee00dc9
Author: Dewayne Richardson <de...@apache.org>
AuthorDate: Mon Apr 2 07:47:31 2018 -0600

    fixed the API tests
---
 traffic_ops/testing/api/v13/cdns_test.go                            | 4 ++--
 .../testing/api/v13/deliveryservice_request_comments_test.go        | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/traffic_ops/testing/api/v13/cdns_test.go b/traffic_ops/testing/api/v13/cdns_test.go
index 2c9ab4c..a3fe7f5 100644
--- a/traffic_ops/testing/api/v13/cdns_test.go
+++ b/traffic_ops/testing/api/v13/cdns_test.go
@@ -49,7 +49,7 @@ func UpdateTestCDNs(t *testing.T) {
 	// Retrieve the CDN by name so we can get the id for the Update
 	resp, _, err := TOSession.GetCDNByName(firstCDN.Name)
 	if err != nil {
-		t.Errorf("cannot GET CDN by name: '%s' %v - %v\n", firstCDN.Name, err)
+		t.Errorf("cannot GET CDN by name: '%s', %v\n", firstCDN.Name, err)
 	}
 	remoteCDN := resp[0]
 	expectedCDNDomain := "domain2"
@@ -63,7 +63,7 @@ func UpdateTestCDNs(t *testing.T) {
 	// Retrieve the CDN to check CDN name got updated
 	resp, _, err = TOSession.GetCDNByID(remoteCDN.ID)
 	if err != nil {
-		t.Errorf("cannot GET CDN by name: '$%s' %v - %v\n", firstCDN.Name, err)
+		t.Errorf("cannot GET CDN by name: '$%s', %v\n", firstCDN.Name, err)
 	}
 	respCDN := resp[0]
 	if respCDN.DomainName != expectedCDNDomain {
diff --git a/traffic_ops/testing/api/v13/deliveryservice_request_comments_test.go b/traffic_ops/testing/api/v13/deliveryservice_request_comments_test.go
index b5eda00..40fda20 100644
--- a/traffic_ops/testing/api/v13/deliveryservice_request_comments_test.go
+++ b/traffic_ops/testing/api/v13/deliveryservice_request_comments_test.go
@@ -72,7 +72,7 @@ func UpdateTestDeliveryServiceRequestComments(t *testing.T) {
 	// Retrieve the delivery service request comment to check that the value got updated
 	resp, _, err := TOSession.GetDeliveryServiceRequestCommentByID(firstComment.ID)
 	if err != nil {
-		t.Errorf("cannot GET delivery service request comment by id: '$%s' %v - %v\n", firstComment.ID, err)
+		t.Errorf("cannot GET delivery service request comment by id: '$%d', %v\n", firstComment.ID, err)
 	}
 	respDSRC := resp[0]
 	if respDSRC.Value != newFirstCommentValue {
@@ -100,7 +100,7 @@ func DeleteTestDeliveryServiceRequestComments(t *testing.T) {
 	for _, comment := range comments {
 		_, _, err := TOSession.DeleteDeliveryServiceRequestCommentByID(comment.ID)
 		if err != nil {
-			t.Errorf("cannot DELETE delivery service request comment by id: '%s' %v\n", comment.ID, err)
+			t.Errorf("cannot DELETE delivery service request comment by id: '%d' %v\n", comment.ID, err)
 		}
 
 		// Retrieve the delivery service request comment to see if it got deleted
@@ -109,7 +109,7 @@ func DeleteTestDeliveryServiceRequestComments(t *testing.T) {
 			t.Errorf("error deleting delivery service request comment: %s\n", err.Error())
 		}
 		if len(comments) > 0 {
-			t.Errorf("expected delivery service request comment: %s to be deleted\n", comment.ID)
+			t.Errorf("expected delivery service request comment: %d to be deleted\n", comment.ID)
 		}
 	}
 }

-- 
To stop receiving notification emails like this one, please contact
mitchell852@apache.org.