Fixing report B

Author: obay

cmd/contacts.go

@@ -390,4 +390,3 @@ func getStringValue(v interface{}) string {
 		return fmt.Sprintf("%v", val)
 	}
 }
-

cmd/root.go

@@ -56,4 +56,3 @@ func initConfig() {
 		fmt.Fprintln(os.Stderr, "Using config file:", viper.ConfigFileUsed())
 	}
 }
-

cmd/version.go

@@ -31,4 +31,3 @@ var versionCmd = &cobra.Command{
 func init() {
 	rootCmd.AddCommand(versionCmd)
 }
-

internal/hubspot/client.go

@@ -217,7 +217,7 @@ func (c *Client) SearchContacts(query string, limit int) (*ContactResponse, erro
 
 	// Parse query - if it contains "=", treat as property=value, otherwise search in common fields
 	var filterGroups []map[string]interface{}
-	
+
 	if strings.Contains(query, "=") {
 		// Property-based search
 		parts := strings.SplitN(query, "=", 2)
@@ -287,4 +287,3 @@ func (c *Client) ListProperties() ([]Property, error) {
 
 	return propsResp.Results, nil
 }
-

internal/hubspot/client_test.go

@@ -0,0 +1,67 @@
+package hubspot
+
+import (
+	"net/http"
+	"net/http/httptest"
+	"testing"
+)
+
+func TestNewClient(t *testing.T) {
+	apiKey := "test-api-key"
+	client := NewClient(apiKey)
+
+	if client == nil {
+		t.Fatal("NewClient returned nil")
+	}
+
+	if client.apiKey != apiKey {
+		t.Errorf("Expected API key %s, got %s", apiKey, client.apiKey)
+	}
+
+	if client.baseURL != baseURL {
+		t.Errorf("Expected base URL %s, got %s", baseURL, client.baseURL)
+	}
+
+	if client.client == nil {
+		t.Fatal("HTTP client is nil")
+	}
+}
+
+func TestClient_doRequest(t *testing.T) {
+	// Create a test server
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.Header.Get("Authorization") != "Bearer test-api-key" {
+			t.Error("Missing or incorrect Authorization header")
+		}
+		w.WriteHeader(http.StatusOK)
+		w.Write([]byte(`{"test": "data"}`))
+	}))
+	defer server.Close()
+
+	client := NewClient("test-api-key")
+	client.baseURL = server.URL
+
+	_, err := client.doRequest("GET", "/test", nil)
+	if err != nil {
+		t.Errorf("doRequest failed: %v", err)
+	}
+}
+
+func TestContact_String(t *testing.T) {
+	contact := Contact{
+		ID: "123",
+		Properties: map[string]interface{}{
+			"email": "test@example.com",
+		},
+	}
+
+	if contact.ID != "123" {
+		t.Errorf("Expected ID 123, got %s", contact.ID)
+	}
+
+	email, ok := contact.Properties["email"].(string)
+	if !ok || email != "test@example.com" {
+		t.Errorf("Expected email test@example.com, got %v", contact.Properties["email"])
+	}
+}
+