fixup! Fix engine test by using query hash instead of serialized params

Signed-off-by: Christian Haudum <christian.haudum@gmail.com>
grafana · Jul 9, 2024 · c778d52 · c778d52
1 parent b7cb579
commit c778d52
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/pkg/logql/engine_test.go b/pkg/logql/engine_test.go
@@ -2738,14 +2738,12 @@ func paramsID(p interface{}) string {
 	case SelectLogParams:
 	case SelectSampleParams:
 		return fmt.Sprintf("%d", params.Plan.Hash())
-	default:
-		b, err := json.Marshal(p)
-		if err != nil {
-			panic(err)
-		}
-		return strings.ReplaceAll(string(b), " ", "")
 	}
-	panic("invalid params type")
+	b, err := json.Marshal(p)
+	if err != nil {
+		panic(err)
+	}
+	return strings.ReplaceAll(string(b), " ", "")
 }
 
 type logData struct {