cleanup debug logs

Author: lahirumaramba

auth/user_mgt.go

@@ -1158,8 +1158,6 @@ func (c *baseClient) QueryUsers(ctx context.Context, query *QueryUsersRequest) (
 		return nil, err
 	}
 
-	//log.Printf("QueryUsers() with response = %d, %d", parsed.Count, len(parsed.Users))
-
 	var userRecords []*UserRecord
 	for _, user := range parsed.Users {
 		userRecord, err := user.makeUserRecord()
@@ -1525,8 +1523,6 @@ func (c *baseClient) post(
 		URL:    url,
 		Body:   internal.NewJSONEntity(payload),
 	}
-	//log.Printf("%+v \n", req.Body)
-	//log.Printf("%+v \n", resp)
 	return c.httpClient.DoAndUnmarshal(ctx, req, resp)
 }
 

auth/user_mgt_test.go

@@ -2048,6 +2048,36 @@ func TestQueryUsersMalformedLastRefreshTimestamp(t *testing.T) {
 	}
 }
 
+func TestQueryUsersDefaultReturnUserInfo(t *testing.T) {
+	resp := `{
+		"userInfo": [{
+			"localId": "testuser"
+		}],
+		"recordsCount": "1"
+	}`
+	s := echoServer([]byte(resp), t)
+	defer s.Close()
+
+	// ReturnUserInfo is nil, should default to true in build()
+	query := &QueryUsersRequest{
+		Limit: 1,
+	}
+
+	_, err := s.Client.QueryUsers(context.Background(), query)
+	if err != nil {
+		t.Fatalf("QueryUsers() = %v", err)
+	}
+
+	var got map[string]interface{}
+	if err := json.Unmarshal(s.Rbody, &got); err != nil {
+		t.Fatal(err)
+	}
+
+	if got["returnUserInfo"] != true {
+		t.Errorf("QueryUsers() request[\"returnUserInfo\"] = %v; want true", got["returnUserInfo"])
+	}
+}
+
 func TestMakeExportedUser(t *testing.T) {
 	queryResponse := &userQueryResponse{
 		UID:                "testuser",
@@ -2507,33 +2537,3 @@ func echoServer(resp interface{}, t *testing.T) *mockAuthServer {
 func (s *mockAuthServer) Close() {
 	s.Srv.Close()
 }
-
-func TestQueryUsersDefaultReturnUserInfo(t *testing.T) {
-	resp := `{
-		"userInfo": [{
-			"localId": "testuser"
-		}],
-		"recordsCount": "1"
-	}`
-	s := echoServer([]byte(resp), t)
-	defer s.Close()
-
-	// ReturnUserInfo is nil, should default to true in build()
-	query := &QueryUsersRequest{
-		Limit: 1,
-	}
-
-	_, err := s.Client.QueryUsers(context.Background(), query)
-	if err != nil {
-		t.Fatalf("QueryUsers() = %v", err)
-	}
-
-	var got map[string]interface{}
-	if err := json.Unmarshal(s.Rbody, &got); err != nil {
-		t.Fatal(err)
-	}
-
-	if got["returnUserInfo"] != true {
-		t.Errorf("QueryUsers() request[\"returnUserInfo\"] = %v; want true", got["returnUserInfo"])
-	}
-}

internal/http_client.go

@@ -167,8 +167,6 @@ func (c *HTTPClient) DoAndUnmarshal(ctx context.Context, req *Request, v interfa
 		return nil, err
 	}
 
-	//log.Printf("%+v \n", string(resp.Body))
-
 	if v != nil {
 		if err := json.Unmarshal(resp.Body, v); err != nil {
 			return nil, fmt.Errorf("error while parsing response: %v", err)