[trafficcontrol] branch master updated: Fix Unit Test for PR (#3211) - ADDSSLKeys API - Remove mandatory SKI/AKI (#3255)

[mi...@apache.org]

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/trafficcontrol.git


The following commit(s) were added to refs/heads/master by this push:
     new e9a9fd0  Fix Unit Test for PR (#3211) - ADDSSLKeys API - Remove mandatory SKI/AKI (#3255)
e9a9fd0 is described below

commit e9a9fd057543ab5133077717859921a2bc23e22d
Author: Jeffrey Bevill <40...@users.noreply.github.com>
AuthorDate: Wed Jan 23 14:04:14 2019 -0700

    Fix Unit Test for PR (#3211) - ADDSSLKeys API - Remove mandatory SKI/AKI (#3255)
    
    Test has been updated to reflect that rootCA is included in certificate chain output.
---
 traffic_ops/traffic_ops_golang/deliveryservice/keys_test.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/traffic_ops/traffic_ops_golang/deliveryservice/keys_test.go b/traffic_ops/traffic_ops_golang/deliveryservice/keys_test.go
index 1bc2bd2..659d221 100644
--- a/traffic_ops/traffic_ops_golang/deliveryservice/keys_test.go
+++ b/traffic_ops/traffic_ops_golang/deliveryservice/keys_test.go
@@ -227,7 +227,7 @@ func TestVerifyAndEncodeCertificate(t *testing.T) {
 
 	certs := strings.SplitAfter(dat, PemCertEndMarker)
 	length := len(certs) - 1
-	if length != 2 {
+	if length != 3 { // rootCA now included in certChain
 		t.Errorf("Test failure: expected 2 certs from verifyCertificate(), got: %d ", length)
 	}
 }