[ARVADOS] updated: 1.3.0-2859-g04ab4d444

[Git user]

Summary of changes:
 lib/controller/integration_test.go      | 1 +
 sdk/go/httpserver/logger_test.go        | 4 ++++
 services/keepstore/s3aws_volume_test.go | 3 +++
 3 files changed, 8 insertions(+)

       via  04ab4d44425ff9cd5556c5978d85da3b6750e91c (commit)
      from  4901a3c5f9442e8b884eb73b37da6272150c57a2 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 04ab4d44425ff9cd5556c5978d85da3b6750e91c
Author: Ward Vandewege <ward at curii.com>
Date:   Fri Aug 14 22:22:44 2020 -0400

    Fix more ineffassign warnings.
    
    No issue #
    
    Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <ward at curii.com>

diff --git a/lib/controller/integration_test.go b/lib/controller/integration_test.go
index 3bf64771d..a73f5f9f8 100644
--- a/lib/controller/integration_test.go
+++ b/lib/controller/integration_test.go
@@ -300,6 +300,7 @@ func (s *IntegrationSuite) TestCreateContainerRequestWithFedToken(c *check.C) {
 	resp, err = arvados.InsecureHTTPClient.Do(req)
 	if c.Check(err, check.IsNil) {
 		err = json.NewDecoder(resp.Body).Decode(&cr)
+		c.Check(err, check.IsNil)
 		c.Check(cr.UUID, check.Matches, "z2222-.*")
 	}
 }
diff --git a/sdk/go/httpserver/logger_test.go b/sdk/go/httpserver/logger_test.go
index 32802b6a9..7d5eb2b64 100644
--- a/sdk/go/httpserver/logger_test.go
+++ b/sdk/go/httpserver/logger_test.go
@@ -58,6 +58,7 @@ func (s *Suite) TestLogRequests(c *check.C) {
 
 	gotReq := make(map[string]interface{})
 	err = dec.Decode(&gotReq)
+	c.Check(err, check.IsNil)
 	c.Logf("%#v", gotReq)
 	c.Check(gotReq["RequestID"], check.Matches, "req-[a-z0-9]{20}")
 	c.Check(gotReq["reqForwardedFor"], check.Equals, "1.2.3.4:12345")
@@ -65,6 +66,7 @@ func (s *Suite) TestLogRequests(c *check.C) {
 
 	gotResp := make(map[string]interface{})
 	err = dec.Decode(&gotResp)
+	c.Check(err, check.IsNil)
 	c.Logf("%#v", gotResp)
 	c.Check(gotResp["RequestID"], check.Equals, gotReq["RequestID"])
 	c.Check(gotResp["reqForwardedFor"], check.Equals, "1.2.3.4:12345")
@@ -111,9 +113,11 @@ func (s *Suite) TestLogErrorBody(c *check.C) {
 
 		gotReq := make(map[string]interface{})
 		err = dec.Decode(&gotReq)
+		c.Check(err, check.IsNil)
 		c.Logf("%#v", gotReq)
 		gotResp := make(map[string]interface{})
 		err = dec.Decode(&gotResp)
+		c.Check(err, check.IsNil)
 		c.Logf("%#v", gotResp)
 		if trial.expectLog {
 			c.Check(gotResp["respBody"], check.Equals, trial.expectBody, comment)
diff --git a/services/keepstore/s3aws_volume_test.go b/services/keepstore/s3aws_volume_test.go
index 97045a660..d9886c07f 100644
--- a/services/keepstore/s3aws_volume_test.go
+++ b/services/keepstore/s3aws_volume_test.go
@@ -166,7 +166,9 @@ func (s *StubbedS3AWSSuite) TestIAMRoleCredentials(c *check.C) {
 		metrics: newVolumeMetricsVecs(prometheus.NewRegistry()),
 	}
 	err := v.check(s.metadata.URL + "/latest")
+	c.Check(err, check.IsNil)
 	creds, err := v.bucket.svc.Client.Config.Credentials.Retrieve(context.Background())
+	c.Check(err, check.IsNil)
 	c.Check(creds.AccessKeyID, check.Equals, "ASIAIOSFODNN7EXAMPLE")
 	c.Check(creds.SecretAccessKey, check.Equals, "wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY")
 
@@ -185,6 +187,7 @@ func (s *StubbedS3AWSSuite) TestIAMRoleCredentials(c *check.C) {
 		metrics: newVolumeMetricsVecs(prometheus.NewRegistry()),
 	}
 	err = deadv.check(s.metadata.URL + "/latest")
+	c.Check(err, check.IsNil)
 	_, err = deadv.bucket.svc.Client.Config.Credentials.Retrieve(context.Background())
 	c.Check(err, check.ErrorMatches, `(?s).*EC2RoleRequestError: no EC2 instance role found.*`)
 	c.Check(err, check.ErrorMatches, `(?s).*404.*`)

-----------------------------------------------------------------------


hooks/post-receive
--