From 0759c98504fba882928d53137491abf06e608619 Mon Sep 17 00:00:00 2001 From: Chris Hodges Date: Tue, 25 Jun 2024 06:45:16 -0500 Subject: [PATCH] Tempo: Fix user facing typo in pipeline operator error message (#89638) Tempo: Fix user facing typo in TraceQL pipeline operator error message --- .../plugins/datasource/tempo/traceql/highlighting.test.ts | 8 ++++---- .../app/plugins/datasource/tempo/traceql/highlighting.ts | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/public/app/plugins/datasource/tempo/traceql/highlighting.test.ts b/public/app/plugins/datasource/tempo/traceql/highlighting.test.ts index e9b1d2a80ed..335983a824b 100644 --- a/public/app/plugins/datasource/tempo/traceql/highlighting.test.ts +++ b/public/app/plugins/datasource/tempo/traceql/highlighting.test.ts @@ -102,7 +102,7 @@ describe('Highlighting', () => { ['{span.http.status_code = 200} &&', 'Invalid spanset expression after spanset combining operator.'], [ '{span.http.status_code = 200} && {span.http.status_code = 200} | foo() > 3', - 'Invalid aggregation operator after pipepile operator.', + 'Invalid aggregation operator after pipeline operator.', ], [ '{span.http.status_code = 200} && {span.http.status_code = 200} | avg() > 3', @@ -134,9 +134,9 @@ describe('Highlighting', () => { ['{ span. }', 'Invalid expression for spanset.'], ['{.foo=}', 'Invalid value after comparison or arithmetic operator.'], ['{.foo="}', 'Invalid value after comparison or arithmetic operator.'], - ['{.foo=300} |', 'Invalid aggregation operator after pipepile operator.'], - ['{.foo=300} && {.bar=200} |', 'Invalid aggregation operator after pipepile operator.'], - ['{.foo=300} && {.bar=300} && {.foo=300} |', 'Invalid aggregation operator after pipepile operator.'], + ['{.foo=300} |', 'Invalid aggregation operator after pipeline operator.'], + ['{.foo=300} && {.bar=200} |', 'Invalid aggregation operator after pipeline operator.'], + ['{.foo=300} && {.bar=300} && {.foo=300} |', 'Invalid aggregation operator after pipeline operator.'], ['{.foo=300} | avg(.value)', 'Invalid comparison operator after aggregator operator.'], ['{.foo=300} && {.foo=300} | avg(.value)', 'Invalid comparison operator after aggregator operator.'], ['{.foo=300} | avg(.value) =', 'Invalid value after comparison operator.'], diff --git a/public/app/plugins/datasource/tempo/traceql/highlighting.ts b/public/app/plugins/datasource/tempo/traceql/highlighting.ts index a8fbefc23ce..21e17c9aba8 100644 --- a/public/app/plugins/datasource/tempo/traceql/highlighting.ts +++ b/public/app/plugins/datasource/tempo/traceql/highlighting.ts @@ -52,7 +52,7 @@ export const computeErrorMessage = (errorNode: SyntaxNode) => { case SpansetPipelineExpression: return 'Invalid spanset combining operator after spanset expression.'; case Pipe: - return 'Invalid aggregation operator after pipepile operator.'; + return 'Invalid aggregation operator after pipeline operator.'; default: return 'Invalid spanset expression after spanset combining operator.'; }