minor lint fixes

pkg/registry/apis/dashboard/legacy/sql_dashboards.go

@@ -115,7 +115,7 @@ func (a *dashboardSqlAccess) getRows(ctx context.Context, query *DashboardQuery)
 		// }
 	}
 
-	sqlcmd := selector
+	var sqlcmd string
 	args := []any{query.OrgID}
 
 	limit := query.Limit

pkg/services/apiserver/options/storage.go

@@ -49,7 +49,7 @@ func (o *StorageOptions) Validate() []error {
 	case StorageTypeFile, StorageTypeEtcd, StorageTypeLegacy, StorageTypeUnified, StorageTypeUnifiedGrpc, StorageTypeUnifiedNext, StorageTypeUnifiedNextGrpc:
 		// no-op
 	default:
-		errs = append(errs, fmt.Errorf("--grafana-apiserver-storage-type must be one of %s, %s, %s, %s, %s, %s", StorageTypeFile, StorageTypeEtcd, StorageTypeLegacy, StorageTypeUnified, StorageTypeUnifiedGrpc, StorageTypeUnifiedNext, StorageTypeUnifiedNextGrpc))
+		errs = append(errs, fmt.Errorf("--grafana-apiserver-storage-type must be one of %s, %s, %s, %s, %s, %s, %s", StorageTypeFile, StorageTypeEtcd, StorageTypeLegacy, StorageTypeUnified, StorageTypeUnifiedGrpc, StorageTypeUnifiedNext, StorageTypeUnifiedNextGrpc))
 	}
 
 	if _, _, err := net.SplitHostPort(o.Address); err != nil {

pkg/tests/apis/dashboard/dashboards_test.go

@@ -1,6 +1,7 @@
 package dashboards
 
 import (
+	"fmt"
 	"testing"
 
 	"github.com/stretchr/testify/require"
@@ -47,7 +48,7 @@ func TestIntegrationDashboardsApp(t *testing.T) {
 
 	t.Run("Check discovery client", func(t *testing.T) {
 		disco := helper.GetGroupVersionInfoJSON("dashboard.grafana.app")
-		//fmt.Printf("%s", string(disco))
+		fmt.Printf("%s", string(disco))
 
 		require.JSONEq(t, `[
   {
@@ -77,10 +78,10 @@ func TestIntegrationDashboardsApp(t *testing.T) {
           {
             "responseKind": {
               "group": "",
-						"kind": "DashboardVersionList",
+              "kind": "PartialObjectMetadataList",
               "version": ""
             },
-						"subresource": "versions",
+            "subresource": "history",
             "verbs": [
               "get"
             ]
@@ -93,7 +94,8 @@ func TestIntegrationDashboardsApp(t *testing.T) {
           "get",
           "list",
           "patch",
-					"update"
+          "update",
+          "watch"
         ]
       }
     ],

public/app/features/dashboard-scene/settings/version-history/HistorySrv.ts

@@ -33,7 +33,7 @@ class LegacyHistorySrv implements HistorySrv {
 
   async getDashboardVersion(dashboardUID: string, version: number): Promise<Dashboard | {}> {
     if (typeof dashboardUID !== 'string') {
-      return Promise.resolve({})
+      return Promise.resolve({});
     }
 
     const info = await getBackendSrv().get(`api/dashboards/uid/${dashboardUID}/versions/${version}`);
@@ -42,7 +42,7 @@ class LegacyHistorySrv implements HistorySrv {
 
   restoreDashboard(dashboardUID: string, version: number): Promise<SaveDashboardResponseDTO | {}> {
     if (typeof dashboardUID !== 'string') {
-      return Promise.resolve({})
+      return Promise.resolve({});
     }
 
     const url = `api/dashboards/uid/${dashboardUID}/restore`;
@@ -51,12 +51,11 @@ class LegacyHistorySrv implements HistorySrv {
   }
 }
 
-
 let historySrv: HistorySrv | undefined = undefined;
 
 export function getHistorySrv(): HistorySrv {
   if (!historySrv) {
-    historySrv = new LegacyHistorySrv()
+    historySrv = new LegacyHistorySrv();
   }
-  return historySrv
+  return historySrv;
 }

public/app/features/dashboard/components/DashboardSettings/VersionsSettings.test.tsx

@@ -135,8 +135,8 @@ describe('VersionSettings', () => {
   });
 
   test('clicking show more appends results to the table', async () => {
-    getHistorySrv().getHistoryList
+    getHistorySrv()
-      // @ts-ignore
+      .getHistoryList// @ts-ignore
       .mockImplementationOnce(() => Promise.resolve(versions.slice(0, VERSIONS_FETCH_LIMIT)))
       .mockImplementationOnce(
         () => new Promise((resolve) => setTimeout(() => resolve(versions.slice(VERSIONS_FETCH_LIMIT)), 1000))
@@ -166,8 +166,8 @@ describe('VersionSettings', () => {
   test('selecting two versions and clicking compare button should render compare view', async () => {
     // @ts-ignore
     historySrv.getHistoryList.mockResolvedValue(versions.slice(0, VERSIONS_FETCH_LIMIT));
-    getHistorySrv().getDashboardVersion
+    getHistorySrv()
-      // @ts-ignore
+      .getDashboardVersion// @ts-ignore
       .mockImplementationOnce(() => Promise.resolve(diffs.lhs))
       .mockImplementationOnce(() => Promise.resolve(diffs.rhs));