Fix warnings reported by ineffassign. (#3147)

hypermodeinc · Mar 21, 2019 · 9ec098c · 9ec098c
1 parent 3d6df60
commit 9ec098c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/ee/acl/acl_curl_test.go b/ee/acl/acl_curl_test.go
@@ -90,7 +90,7 @@ func TestCurlAuthorization(t *testing.T) {
 		failMsg:    "Token is expired",
 	})
 	// login again using the refreshJwt
-	accessJwt, refreshJwt, err = z.HttpLogin(&z.LoginParams{
+	_, refreshJwt, err = z.HttpLogin(&z.LoginParams{
 		Endpoint:   loginEndpoint,
 		RefreshJwt: refreshJwt,
 	})
@@ -109,7 +109,7 @@ func TestCurlAuthorization(t *testing.T) {
 		failMsg:    "Token is expired",
 	})
 	// refresh the jwts again
-	accessJwt, refreshJwt, err = z.HttpLogin(&z.LoginParams{
+	_, refreshJwt, err = z.HttpLogin(&z.LoginParams{
 		Endpoint:   loginEndpoint,
 		RefreshJwt: refreshJwt,
 	})
@@ -133,7 +133,7 @@ func TestCurlAuthorization(t *testing.T) {
 	glog.Infof("Sleeping for 6 seconds for acl caches to be refreshed")
 	time.Sleep(6 * time.Second)
 	// refresh the jwts again
-	accessJwt, refreshJwt, err = z.HttpLogin(&z.LoginParams{
+	_, _, err = z.HttpLogin(&z.LoginParams{
 		Endpoint:   loginEndpoint,
 		RefreshJwt: refreshJwt,
 	})

diff --git a/systest/21million/run_test.go b/systest/21million/run_test.go
@@ -29,6 +29,7 @@ import (
 	"github.com/dgraph-io/dgraph/chunker"
 	"github.com/dgraph-io/dgraph/x"
 	"github.com/dgraph-io/dgraph/z"
+	"github.com/stretchr/testify/require"
 )
 
 func TestQueries(t *testing.T) {
@@ -53,6 +54,7 @@ func TestQueries(t *testing.T) {
 		// The test query and expected result are separated by a delimiter.
 		bodies := strings.SplitN(contents, "\n---\n", 2)
 		resp, err := dg.NewTxn().Query(context.Background(), bodies[0])
+		require.NoError(err)
 
 		t.Logf("running %s", file.Name())
 		if len(resp.GetJson()) > 0 {