cleanup

Author: ivpusic

cmd/query/archive.go

@@ -0,0 +1,46 @@
+package query
+
+import (
+	"fmt"
+	"strconv"
+
+	"github.com/duneanalytics/cli/cmdutil"
+	"github.com/duneanalytics/cli/output"
+	"github.com/spf13/cobra"
+)
+
+func newArchiveCmd() *cobra.Command {
+	cmd := &cobra.Command{
+		Use:   "archive <query-id>",
+		Short: "Archive a saved query",
+		Args:  cobra.ExactArgs(1),
+		RunE:  runArchive,
+	}
+
+	output.AddFormatFlag(cmd, "text")
+
+	return cmd
+}
+
+func runArchive(cmd *cobra.Command, args []string) error {
+	queryID, err := strconv.Atoi(args[0])
+	if err != nil {
+		return fmt.Errorf("invalid query ID %q: must be an integer", args[0])
+	}
+
+	client := cmdutil.ClientFromCmd(cmd)
+
+	resp, err := client.ArchiveQuery(queryID)
+	if err != nil {
+		return err
+	}
+
+	w := cmd.OutOrStdout()
+	switch output.FormatFromCmd(cmd) {
+	case output.FormatJSON:
+		return output.PrintJSON(w, resp)
+	default:
+		fmt.Fprintf(w, "Archived query %d\n", resp.QueryID)
+		return nil
+	}
+}

cmd/query/archive_test.go

@@ -0,0 +1,71 @@
+package query_test
+
+import (
+	"encoding/json"
+	"errors"
+	"testing"
+
+	"github.com/duneanalytics/duneapi-client-go/models"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+func TestArchiveSuccess(t *testing.T) {
+	mock := &mockClient{
+		archiveQueryFn: func(id int) (*models.UpdateQueryResponse, error) {
+			assert.Equal(t, 4125432, id)
+			return &models.UpdateQueryResponse{QueryID: 4125432}, nil
+		},
+	}
+
+	root, buf := newTestRoot(mock)
+	root.SetArgs([]string{"query", "archive", "4125432"})
+	require.NoError(t, root.Execute())
+	assert.Equal(t, "Archived query 4125432\n", buf.String())
+}
+
+func TestArchiveJSONOutput(t *testing.T) {
+	mock := &mockClient{
+		archiveQueryFn: func(_ int) (*models.UpdateQueryResponse, error) {
+			return &models.UpdateQueryResponse{QueryID: 4125432}, nil
+		},
+	}
+
+	root, buf := newTestRoot(mock)
+	root.SetArgs([]string{"query", "archive", "4125432", "-o", "json"})
+	require.NoError(t, root.Execute())
+
+	var got map[string]int
+	require.NoError(t, json.Unmarshal(buf.Bytes(), &got))
+	assert.Equal(t, 4125432, got["query_id"])
+}
+
+func TestArchiveMissingArgument(t *testing.T) {
+	root, _ := newTestRoot(&mockClient{})
+	root.SetArgs([]string{"query", "archive"})
+	err := root.Execute()
+	require.Error(t, err)
+	assert.Contains(t, err.Error(), "accepts 1 arg(s)")
+}
+
+func TestArchiveNonIntegerID(t *testing.T) {
+	root, _ := newTestRoot(&mockClient{})
+	root.SetArgs([]string{"query", "archive", "abc"})
+	err := root.Execute()
+	require.Error(t, err)
+	assert.Contains(t, err.Error(), "invalid query ID")
+}
+
+func TestArchiveAPIError(t *testing.T) {
+	mock := &mockClient{
+		archiveQueryFn: func(_ int) (*models.UpdateQueryResponse, error) {
+			return nil, errors.New("api: not found")
+		},
+	}
+
+	root, _ := newTestRoot(mock)
+	root.SetArgs([]string{"query", "archive", "999"})
+	err := root.Execute()
+	require.Error(t, err)
+	assert.Contains(t, err.Error(), "api: not found")
+}

cmd/query/query.go

@@ -11,5 +11,6 @@ func NewQueryCmd() *cobra.Command {
 	cmd.AddCommand(newCreateCmd())
 	cmd.AddCommand(newGetCmd())
 	cmd.AddCommand(newUpdateCmd())
+	cmd.AddCommand(newArchiveCmd())
 	return cmd
 }

cmd/query/testutil_test.go

@@ -14,9 +14,10 @@ import (
 // mockClient embeds the interface so unimplemented methods panic.
 type mockClient struct {
 	dune.DuneClient
-	createQueryFn func(models.CreateQueryRequest) (*models.CreateQueryResponse, error)
-	getQueryFn    func(int) (*models.GetQueryResponse, error)
-	updateQueryFn func(int, models.UpdateQueryRequest) (*models.UpdateQueryResponse, error)
+	createQueryFn  func(models.CreateQueryRequest) (*models.CreateQueryResponse, error)
+	getQueryFn     func(int) (*models.GetQueryResponse, error)
+	updateQueryFn  func(int, models.UpdateQueryRequest) (*models.UpdateQueryResponse, error)
+	archiveQueryFn func(int) (*models.UpdateQueryResponse, error)
 }
 
 func (m *mockClient) CreateQuery(req models.CreateQueryRequest) (*models.CreateQueryResponse, error) {
@@ -31,6 +32,10 @@ func (m *mockClient) UpdateQuery(queryID int, req models.UpdateQueryRequest) (*m
 	return m.updateQueryFn(queryID, req)
 }
 
+func (m *mockClient) ArchiveQuery(queryID int) (*models.UpdateQueryResponse, error) {
+	return m.archiveQueryFn(queryID)
+}
+
 // newTestRoot builds a root → query command tree with the mock injected.
 func newTestRoot(mock dune.DuneClient) (*cobra.Command, *bytes.Buffer) {
 	root := &cobra.Command{