diff --git a/.betterer.results b/.betterer.results index d2598bb0bf6..477d19d5d82 100644 --- a/.betterer.results +++ b/.betterer.results @@ -44,7 +44,7 @@ exports[`no enzyme tests`] = { "packages/jaeger-ui-components/src/TraceTimelineViewer/ListView/index.test.js:1734982398": [ [14, 26, 13, "RegExp match", "2409514259"] ], - "packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.test.js:700147304": [ + "packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.test.js:1241675783": [ [16, 19, 13, "RegExp match", "2409514259"] ], "packages/jaeger-ui-components/src/TraceTimelineViewer/SpanTreeOffset.test.js:174536706": [ @@ -7044,12 +7044,7 @@ exports[`better eslint`] = { [0, 0, 0, "Unexpected any. Specify a different type.", "1"] ], "public/app/features/explore/TraceView/TraceView.test.tsx:5381": [ - [0, 0, 0, "Unexpected any. Specify a different type.", "0"], - [0, 0, 0, "Unexpected any. Specify a different type.", "1"], - [0, 0, 0, "Unexpected any. Specify a different type.", "2"], - [0, 0, 0, "Unexpected any. Specify a different type.", "3"], - [0, 0, 0, "Unexpected any. Specify a different type.", "4"], - [0, 0, 0, "Unexpected any. Specify a different type.", "5"] + [0, 0, 0, "Parsing error: Merge conflict marker encountered.", "0"] ], "public/app/features/explore/TraceView/TraceView.tsx:5381": [ [0, 0, 0, "Do not use any type assertions.", "0"], diff --git a/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/AccordianLogs.test.js b/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/AccordianLogs.test.js index 947a805b633..592af162af1 100644 --- a/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/AccordianLogs.test.js +++ b/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/AccordianLogs.test.js @@ -56,7 +56,7 @@ describe('AccordianLogs tests', () => { it('shows the number of log entries', () => { setup(); - expect(screen.getByRole('switch', { name: 'Logs (2)' })).toBeInTheDocument(); + expect(screen.getByRole('switch', { name: 'Events (2)' })).toBeInTheDocument(); }); it('hides log entries when not expanded', () => { diff --git a/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/AccordianLogs.tsx b/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/AccordianLogs.tsx index 268f617802b..065fda8cceb 100644 --- a/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/AccordianLogs.tsx +++ b/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/AccordianLogs.tsx @@ -90,7 +90,7 @@ export default function AccordianLogs(props: AccordianLogsProps) { return (
- {arrow} Logs ({logs.length}) + {arrow} Events ({logs.length}) {isOpen && (
diff --git a/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.test.js b/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.test.js index e13a3550f97..b9423169915 100644 --- a/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.test.js +++ b/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.test.js @@ -146,14 +146,14 @@ describe('', () => { }); it('renders the span tags', () => { - const target = ; + const target = ; expect(wrapper.containsMatchingElement(target)).toBe(true); wrapper.find({ data: span.tags }).simulate('toggle'); expect(props.tagsToggle).toHaveBeenLastCalledWith(span.spanID); }); it('renders the process tags', () => { - const target = ; + const target = ; expect(wrapper.containsMatchingElement(target)).toBe(true); wrapper.find({ data: span.process.tags }).simulate('toggle'); expect(props.processToggle).toHaveBeenLastCalledWith(span.spanID); diff --git a/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.tsx b/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.tsx index 8f4b5e5cfae..5bb0ceacf17 100644 --- a/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.tsx +++ b/packages/jaeger-ui-components/src/TraceTimelineViewer/SpanDetail/index.tsx @@ -213,7 +213,7 @@ export default function SpanDetail(props: SpanDetailProps) {
tagsToggle(spanID)} @@ -222,7 +222,7 @@ export default function SpanDetail(props: SpanDetailProps) { processToggle(spanID)} diff --git a/public/app/features/explore/TraceView/TraceView.test.tsx b/public/app/features/explore/TraceView/TraceView.test.tsx index 3cdb9e82994..c50f6207d63 100644 --- a/public/app/features/explore/TraceView/TraceView.test.tsx +++ b/public/app/features/explore/TraceView/TraceView.test.tsx @@ -92,14 +92,14 @@ describe('TraceView', () => { it('toggles detailState', async () => { renderTraceViewNew(); - expect(screen.queryByText(/Tags/)).toBeFalsy(); + expect(screen.queryByText(/Attributes/)).toBeFalsy(); const spanView = screen.getAllByText('', { selector: 'div[data-testid="span-view"]' })[0]; await userEvent.click(spanView); - expect(screen.queryByText(/Tags/)).toBeTruthy(); + expect(screen.queryByText(/Attributes/)).toBeTruthy(); await userEvent.click(spanView); - screen.debug(screen.queryAllByText(/Tags/)); - expect(screen.queryByText(/Tags/)).toBeFalsy(); + screen.debug(screen.queryAllByText(/Attributes/)); + expect(screen.queryByText(/Attributes/)).toBeFalsy(); }); it('shows timeline ticks', () => { @@ -118,21 +118,21 @@ describe('TraceView', () => { const firstSpan = screen.getAllByText('', { selector: 'div[data-testid="span-view"]' })[0]; await userEvent.click(firstSpan); - await userEvent.click(screen.getByText(/Process/)); + await userEvent.click(screen.getByText(/Resource/)); table = screen.getByText('', { selector: 'div[data-testid="KeyValueTable"]' }); expect(table.innerHTML).toContain('client-uuid-1'); await userEvent.click(firstSpan); const secondSpan = screen.getAllByText('', { selector: 'div[data-testid="span-view"]' })[1]; await userEvent.click(secondSpan); - await userEvent.click(screen.getByText(/Process/)); + await userEvent.click(screen.getByText(/Resource/)); table = screen.getByText('', { selector: 'div[data-testid="KeyValueTable"]' }); expect(table.innerHTML).toContain('client-uuid-2'); await userEvent.click(secondSpan); const thirdSpan = screen.getAllByText('', { selector: 'div[data-testid="span-view"]' })[2]; await userEvent.click(thirdSpan); - await userEvent.click(screen.getByText(/Process/)); + await userEvent.click(screen.getByText(/Resource/)); table = screen.getByText('', { selector: 'div[data-testid="KeyValueTable"]' }); expect(table.innerHTML).toContain('client-uuid-3'); }); @@ -142,10 +142,10 @@ describe('TraceView', () => { const span = screen.getAllByText('', { selector: 'div[data-testid="span-view"]' })[2]; await userEvent.click(span); //Process is in detail view - expect(screen.getByText(/Process/)).toBeInTheDocument(); + expect(screen.getByText(/Resource/)).toBeInTheDocument(); rerender(getTraceView([frameNew])); - expect(screen.queryByText(/Process/)).not.toBeInTheDocument(); + expect(screen.queryByText(/Resource/)).not.toBeInTheDocument(); }); });