Skip to content

Commit

Permalink
fix(instrumentation-generic-pool): update span name typo (#2405)
Browse files Browse the repository at this point in the history
Should be 'generic-pool.acquire' instead of 'generic-pool.aquire'

Signed-off-by: Kaung Zin Hein <kaungzinhein113@gmail.com>
Co-authored-by: Marc Pichler <marc.pichler@dynatrace.com>
  • Loading branch information
Zen-cronic and pichlermarc committed Aug 27, 2024
1 parent d7a5bd4 commit 2105609
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ export class GenericPoolInstrumentation extends InstrumentationBase {
) {
const parent = api.context.active();
const span = instrumentation.tracer.startSpan(
'generic-pool.aquire',
'generic-pool.acquire',
{},
parent
);
Expand Down Expand Up @@ -156,7 +156,7 @@ export class GenericPoolInstrumentation extends InstrumentationBase {
}
const parent = api.context.active();
const span = instrumentation.tracer.startSpan(
'generic-pool.aquire',
'generic-pool.acquire',
{},
parent
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ describe('GenericPool instrumentation', () => {
assert.strictEqual(await acquire(), CLIENT);
const [span] = memoryExporter.getFinishedSpans();
assert.strictEqual(memoryExporter.getFinishedSpans().length, 1);
assert.strictEqual(span.name, 'generic-pool.aquire');
assert.strictEqual(span.name, 'generic-pool.acquire');
});

it('should attach it to the parent span', async () => {
Expand All @@ -107,7 +107,7 @@ describe('GenericPool instrumentation', () => {
assert.strictEqual(memoryExporter.getFinishedSpans().length, 2);

const [span] = memoryExporter.getFinishedSpans();
assert.strictEqual(span.name, 'generic-pool.aquire');
assert.strictEqual(span.name, 'generic-pool.acquire');
assert.strictEqual(span.parentSpanId, rootSpan.spanContext().spanId);
});
});
Expand Down

0 comments on commit 2105609

Please sign in to comment.