1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330 |
- import * as Sentry from '@sentry/react';
- import MockDate from 'mockdate';
- import {DetailedEventsFixture} from 'sentry-fixture/events';
- import {ProjectFixture} from 'sentry-fixture/project';
- import {initializeOrg} from 'sentry-test/initializeOrg';
- import {
- act,
- findByText,
- fireEvent,
- render,
- screen,
- userEvent,
- waitFor,
- } from 'sentry-test/reactTestingLibrary';
- import type {RawSpanType} from 'sentry/components/events/interfaces/spans/types';
- import {EntryType, type Event, type EventTransaction} from 'sentry/types';
- import type {TraceFullDetailed} from 'sentry/utils/performance/quickTrace/types';
- import {TraceView} from 'sentry/views/performance/newTraceDetails/index';
- import type {TraceTree} from 'sentry/views/performance/newTraceDetails/traceModels/traceTree';
- import {RouteContext} from 'sentry/views/routeContext';
- jest.mock('screenfull', () => ({
- enabled: true,
- get isFullscreen() {
- return false;
- },
- request: jest.fn(),
- exit: jest.fn(),
- on: jest.fn(),
- off: jest.fn(),
- }));
- class MockResizeObserver {
- callback: ResizeObserverCallback;
- constructor(callback: ResizeObserverCallback) {
- this.callback = callback;
- }
- unobserve(_element: HTMLElement) {
- return;
- }
- observe(element: HTMLElement) {
- // Executes in sync so we dont have to
- this.callback(
- [
- {
- target: element,
- // @ts-expect-error partial mock
- contentRect: {width: 1000, height: 24 * 10 - 1},
- },
- ],
- this
- );
- }
- disconnect() {}
- }
- function TraceViewWithProviders({traceSlug}: {traceSlug: string}) {
- const {router} = initializeOrg({
- project: ProjectFixture(),
- });
- return (
- <RouteContext.Provider
- value={{
- router,
- location: router.location,
- params: {...router.params, traceSlug},
- routes: router.routes,
- }}
- >
- <TraceView />
- </RouteContext.Provider>
- );
- }
- type Arguments<F extends Function> = F extends (...args: infer A) => any ? A : never;
- type ResponseType = Arguments<typeof MockApiClient.addMockResponse>[0];
- function mockTraceResponse(resp?: Partial<ResponseType>) {
- MockApiClient.addMockResponse({
- url: '/organizations/org-slug/events-trace/trace-id/',
- method: 'GET',
- asyncDelay: 1,
- ...(resp ?? {}),
- });
- }
- function mockTraceMetaResponse(resp?: Partial<ResponseType>) {
- MockApiClient.addMockResponse({
- url: '/organizations/org-slug/events-trace-meta/trace-id/',
- method: 'GET',
- asyncDelay: 1,
- ...(resp ?? {}),
- });
- }
- function mockTraceTagsResponse(resp?: Partial<ResponseType>) {
- MockApiClient.addMockResponse({
- url: '/organizations/org-slug/events-facets/',
- method: 'GET',
- asyncDelay: 1,
- ...(resp ?? []),
- });
- }
- // function _mockTraceDetailsResponse(id: string, resp?: Partial<ResponseType>) {
- // MockApiClient.addMockResponse({
- // url: `/organizations/org-slug/events/project_slug:transaction-${id}`,
- // method: 'GET',
- // asyncDelay: 1,
- // ...(resp ?? {}),
- // });
- // }
- function mockTransactionDetailsResponse(id: string, resp?: Partial<ResponseType>) {
- MockApiClient.addMockResponse({
- url: `/organizations/org-slug/events/project_slug:${id}/`,
- method: 'GET',
- asyncDelay: 1,
- ...(resp ?? {body: DetailedEventsFixture()[0]}),
- });
- }
- function mockTraceRootEvent(id: string, resp?: Partial<ResponseType>) {
- MockApiClient.addMockResponse({
- url: `/organizations/org-slug/events/project_slug:${id}/`,
- method: 'GET',
- asyncDelay: 1,
- ...(resp ?? {body: DetailedEventsFixture()[0]}),
- });
- }
- function mockTraceRootFacets(resp?: Partial<ResponseType>) {
- MockApiClient.addMockResponse({
- url: `/organizations/org-slug/events-facets/`,
- method: 'GET',
- asyncDelay: 1,
- body: {},
- ...(resp ?? {}),
- });
- }
- function mockTraceEventDetails(resp?: Partial<ResponseType>) {
- MockApiClient.addMockResponse({
- url: `/organizations/org-slug/events/`,
- method: 'GET',
- asyncDelay: 1,
- body: {},
- ...(resp ?? {}),
- });
- }
- function mockSpansResponse(
- id: string,
- resp?: Partial<ResponseType>,
- body: Partial<EventTransaction> = {}
- ) {
- return MockApiClient.addMockResponse({
- url: `/organizations/org-slug/events/project_slug:${id}/?averageColumn=span.self_time&averageColumn=span.duration`,
- method: 'GET',
- asyncDelay: 1,
- body,
- ...(resp ?? {}),
- });
- }
- let sid = -1;
- let tid = -1;
- const span_id = () => `${++sid}`;
- const txn_id = () => `${++tid}`;
- function makeTransaction(overrides: Partial<TraceFullDetailed> = {}): TraceFullDetailed {
- const t = txn_id();
- const s = span_id();
- return {
- children: [],
- event_id: t,
- parent_event_id: 'parent_event_id',
- parent_span_id: 'parent_span_id',
- start_timestamp: 0,
- timestamp: 1,
- generation: 0,
- span_id: s,
- 'transaction.duration': 1,
- transaction: 'transaction-name' + t,
- 'transaction.op': 'transaction-op-' + t,
- 'transaction.status': '',
- project_id: 0,
- project_slug: 'project_slug',
- errors: [],
- performance_issues: [],
- ...overrides,
- };
- }
- function makeEvent(overrides: Partial<Event> = {}, spans: RawSpanType[] = []): Event {
- return {
- entries: [{type: EntryType.SPANS, data: spans}],
- ...overrides,
- } as Event;
- }
- function makeSpan(overrides: Partial<RawSpanType> = {}): TraceTree.Span {
- return {
- span_id: '',
- op: '',
- description: '',
- start_timestamp: 0,
- timestamp: 10,
- data: {},
- trace_id: '',
- childTransactions: [],
- event: makeEvent() as EventTransaction,
- ...overrides,
- };
- }
- async function keyboardNavigationTestSetup() {
- const keyboard_navigation_transactions: TraceFullDetailed[] = [];
- for (let i = 0; i < 1e4; i++) {
- keyboard_navigation_transactions.push(
- makeTransaction({
- span_id: i + '',
- event_id: i + '',
- transaction: 'transaction-name' + i,
- 'transaction.op': 'transaction-op-' + i,
- })
- );
- mockTransactionDetailsResponse(i.toString());
- }
- mockTraceResponse({
- body: {
- transactions: keyboard_navigation_transactions,
- orphan_errors: [],
- },
- });
- mockTraceMetaResponse();
- mockTraceRootFacets();
- mockTraceRootEvent('0', {body: DetailedEventsFixture()[0]});
- mockTraceEventDetails();
- const value = render(<TraceViewWithProviders traceSlug="trace-id" />);
- const virtualizedContainer = screen.queryByTestId('trace-virtualized-list');
- const virtualizedScrollContainer = screen.queryByTestId(
- 'trace-virtualized-list-scroll-container'
- );
- if (!virtualizedContainer) {
- throw new Error('Virtualized container not found');
- }
- if (!virtualizedScrollContainer) {
- throw new Error('Virtualized scroll container not found');
- }
- // Awaits for the placeholder rendering rows to be removed
- expect(await findByText(value.container, /transaction-op-0/i)).toBeInTheDocument();
- return {...value, virtualizedContainer, virtualizedScrollContainer};
- }
- async function pageloadTestSetup() {
- const keyboard_navigation_transactions: TraceFullDetailed[] = [];
- for (let i = 0; i < 1e4; i++) {
- keyboard_navigation_transactions.push(
- makeTransaction({
- span_id: i + '',
- event_id: i + '',
- transaction: 'transaction-name' + i,
- 'transaction.op': 'transaction-op-' + i,
- })
- );
- mockTransactionDetailsResponse(i.toString());
- }
- mockTraceResponse({
- body: {
- transactions: keyboard_navigation_transactions,
- orphan_errors: [],
- },
- });
- mockTraceMetaResponse();
- mockTraceRootFacets();
- mockTraceRootEvent('0', {body: DetailedEventsFixture()[0]});
- mockTraceEventDetails();
- const value = render(<TraceViewWithProviders traceSlug="trace-id" />);
- const virtualizedContainer = screen.queryByTestId('trace-virtualized-list');
- const virtualizedScrollContainer = screen.queryByTestId(
- 'trace-virtualized-list-scroll-container'
- );
- if (!virtualizedContainer) {
- throw new Error('Virtualized container not found');
- }
- if (!virtualizedScrollContainer) {
- throw new Error('Virtualized scroll container not found');
- }
- // Awaits for the placeholder rendering rows to be removed
- expect((await screen.findAllByText(/transaction-op-/i)).length).toBeGreaterThan(0);
- return {...value, virtualizedContainer, virtualizedScrollContainer};
- }
- async function searchTestSetup() {
- const transactions: TraceFullDetailed[] = [];
- for (let i = 0; i < 11; i++) {
- transactions.push(
- makeTransaction({
- span_id: i + '',
- event_id: i + '',
- transaction: 'transaction-name' + i,
- 'transaction.op': 'transaction-op-' + i,
- })
- );
- mockTransactionDetailsResponse(i.toString());
- }
- mockTraceResponse({
- body: {
- transactions: transactions,
- orphan_errors: [],
- },
- });
- mockTraceMetaResponse();
- mockTraceRootFacets();
- mockTraceRootEvent('0', {body: DetailedEventsFixture()[0]});
- mockTraceEventDetails({body: DetailedEventsFixture()[0]});
- const value = render(<TraceViewWithProviders traceSlug="trace-id" />);
- const virtualizedContainer = screen.queryByTestId('trace-virtualized-list');
- const virtualizedScrollContainer = screen.queryByTestId(
- 'trace-virtualized-list-scroll-container'
- );
- if (!virtualizedContainer) {
- throw new Error('Virtualized container not found');
- }
- if (!virtualizedScrollContainer) {
- throw new Error('Virtualized scroll container not found');
- }
- // Awaits for the placeholder rendering rows to be removed
- expect(await findByText(value.container, /transaction-op-0/i)).toBeInTheDocument();
- return {...value, virtualizedContainer, virtualizedScrollContainer};
- }
- async function simpleTestSetup() {
- const transactions: TraceFullDetailed[] = [];
- let parent: any;
- for (let i = 0; i < 1e3; i++) {
- const next = makeTransaction({
- span_id: i + '',
- event_id: i + '',
- transaction: 'transaction-name' + i,
- 'transaction.op': 'transaction-op-' + i,
- });
- if (parent) {
- parent.children.push(next);
- } else {
- transactions.push(next);
- }
- parent = next;
- mockTransactionDetailsResponse(i.toString());
- }
- mockTraceResponse({
- body: {
- transactions: transactions,
- orphan_errors: [],
- },
- });
- mockTraceMetaResponse();
- mockTraceRootFacets();
- mockTraceRootEvent('0', {body: DetailedEventsFixture()[0]});
- mockTraceEventDetails();
- const value = render(<TraceViewWithProviders traceSlug="trace-id" />);
- const virtualizedContainer = screen.queryByTestId('trace-virtualized-list');
- const virtualizedScrollContainer = screen.queryByTestId(
- 'trace-virtualized-list-scroll-container'
- );
- if (!virtualizedContainer) {
- throw new Error('Virtualized container not found');
- }
- if (!virtualizedScrollContainer) {
- throw new Error('Virtualized scroll container not found');
- }
- // Awaits for the placeholder rendering rows to be removed
- expect(await findByText(value.container, /transaction-op-0/i)).toBeInTheDocument();
- return {...value, virtualizedContainer, virtualizedScrollContainer};
- }
- const DRAWER_TABS_TEST_ID = 'trace-drawer-tab';
- const DRAWER_TABS_PIN_BUTTON_TEST_ID = 'trace-drawer-tab-pin-button';
- // @ts-expect-error ignore this line
- // eslint-disable-next-line
- const DRAWER_TABS_CONTAINER_TEST_ID = 'trace-drawer-tabs';
- const VISIBLE_TRACE_ROW_SELECTOR = '.TraceRow:not(.Hidden)';
- const ACTIVE_SEARCH_HIGHLIGHT_ROW = '.TraceRow.SearchResult.Highlight:not(.Hidden)';
- const wait = (ms: number) => new Promise(resolve => setTimeout(resolve, ms));
- const searchToUpdate = (): Promise<void> => {
- return act(async () => {
- await wait(500);
- });
- };
- const scrollToEnd = (): Promise<void> => {
- return act(async () => {
- await wait(1000);
- });
- };
- // @ts-expect-error ignore this line
- // eslint-disable-next-line
- function printVirtualizedList(container: HTMLElement) {
- const stdout: string[] = [];
- const scrollContainer = screen.queryByTestId(
- 'trace-virtualized-list-scroll-container'
- )!;
- stdout.push(
- 'top:' + scrollContainer.scrollTop + ' ' + 'left:' + scrollContainer.scrollLeft
- );
- stdout.push('///////////////////');
- const rows = Array.from(container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR));
- for (const r of [...rows]) {
- let t = r.textContent ?? '';
- if (r.classList.contains('SearchResult')) {
- t = 'search ' + t;
- }
- if (r.classList.contains('Highlight')) {
- t = 'highlight ' + t;
- }
- stdout.push(t);
- }
- // This is a debug fn, we need it to log
- // eslint-disable-next-line
- console.log(stdout.join('\n'));
- }
- // @ts-expect-error ignore this line
- // eslint-disable-next-line
- function printTabs() {
- const tabs = screen.queryAllByTestId(DRAWER_TABS_TEST_ID);
- const stdout: string[] = [];
- for (const tab of tabs) {
- let text = tab.textContent ?? 'empty tab??';
- if (tab.hasAttribute('aria-selected')) {
- text = 'active' + text;
- }
- stdout.push(text);
- }
- // This is a debug fn, we need it to log
- // eslint-disable-next-line
- console.log(stdout.join(' | '));
- }
- function assertHighlightedRowAtIndex(virtualizedContainer: HTMLElement, index: number) {
- expect(virtualizedContainer.querySelectorAll('.TraceRow.Highlight')).toHaveLength(1);
- const highlighted_row = virtualizedContainer.querySelector(ACTIVE_SEARCH_HIGHLIGHT_ROW);
- const r = Array.from(virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR));
- expect(r.indexOf(highlighted_row!)).toBe(index);
- }
- describe('trace view', () => {
- beforeEach(() => {
- globalThis.ResizeObserver = MockResizeObserver as any;
- // We are having replay errors about invalid stylesheets, though the CSS seems valid
- jest.spyOn(console, 'error').mockImplementation(() => {});
- Object.defineProperty(window, 'location', {
- value: {
- search: '',
- },
- });
- MockDate.reset();
- });
- afterEach(() => {
- // @ts-expect-error clear mock
- globalThis.ResizeObserver = undefined;
- // @ts-expect-error override it
- window.location = new URL('http://localhost/');
- });
- it('renders loading state', async () => {
- mockTraceResponse();
- mockTraceMetaResponse();
- mockTraceTagsResponse();
- render(<TraceViewWithProviders traceSlug="trace-id" />);
- expect(await screen.findByText(/assembling the trace/i)).toBeInTheDocument();
- });
- it('renders error state', async () => {
- mockTraceResponse({statusCode: 404});
- mockTraceMetaResponse({statusCode: 404});
- mockTraceTagsResponse({statusCode: 404});
- render(<TraceViewWithProviders traceSlug="trace-id" />);
- expect(await screen.findByText(/we failed to load your trace/i)).toBeInTheDocument();
- });
- it('renders empty state', async () => {
- mockTraceResponse({
- body: {
- transactions: [],
- orphan_errors: [],
- },
- });
- mockTraceMetaResponse();
- mockTraceTagsResponse();
- render(<TraceViewWithProviders traceSlug="trace-id" />);
- expect(
- await screen.findByText(/trace does not contain any data/i)
- ).toBeInTheDocument();
- });
- describe('pageload', () => {
- it('highlights row at load and sets it as focused', async () => {
- Object.defineProperty(window, 'location', {
- value: {
- search: '?node=txn-5',
- },
- });
- const {virtualizedContainer} = await pageloadTestSetup();
- expect(await screen.findByTestId('trace-drawer-title')).toHaveTextContent(
- 'transaction-op-5'
- );
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[6]).toHaveFocus();
- });
- it('scrolls at transaction span', async () => {
- Object.defineProperty(window, 'location', {
- value: {
- search: '?node=span-5&node=txn-5',
- },
- });
- mockSpansResponse(
- '5',
- {},
- {
- entries: [
- {
- type: EntryType.SPANS,
- data: [makeSpan({span_id: '5', op: 'special-span'})],
- },
- ],
- }
- );
- const {virtualizedContainer} = await pageloadTestSetup();
- expect(await screen.findByTestId('trace-drawer-title')).toHaveTextContent(
- 'special-span'
- );
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[7]).toHaveFocus();
- });
- it('scrolls far down the list of transactions', async () => {
- Object.defineProperty(window, 'location', {
- value: {
- search: '?node=txn-500',
- },
- });
- await pageloadTestSetup();
- expect(await screen.findByTestId('trace-drawer-title')).toHaveTextContent(
- 'transaction-op-500'
- );
- await act(async () => {
- await wait(1000);
- });
- await waitFor(() => {
- expect(document.activeElement).toHaveClass('TraceRow');
- expect(
- document.activeElement?.textContent?.includes('transaction-op-500')
- ).toBeTruthy();
- });
- });
- it('scrolls to event id query param and fetches its spans', async () => {
- Object.defineProperty(window, 'location', {
- value: {
- search: '?eventId=500',
- },
- });
- const spanRequest = mockSpansResponse(
- '500',
- {},
- {
- entries: [
- {
- type: EntryType.SPANS,
- data: [makeSpan({span_id: '1', op: 'special-span'})],
- },
- ],
- }
- );
- await pageloadTestSetup();
- expect(await screen.findByTestId('trace-drawer-title')).toHaveTextContent(
- 'transaction-op-500'
- );
- await waitFor(() => {
- expect(document.activeElement).toHaveClass('TraceRow');
- expect(
- document.activeElement?.textContent?.includes('transaction-op-500')
- ).toBeTruthy();
- });
- expect(spanRequest).toHaveBeenCalledTimes(1);
- expect(await screen.findByText('special-span')).toBeInTheDocument();
- });
- it('logs if path is not found', async () => {
- Object.defineProperty(window, 'location', {
- value: {
- search: '?eventId=bad_value',
- },
- });
- const sentrySpy = jest.spyOn(Sentry, 'captureMessage');
- await pageloadTestSetup();
- await waitFor(() => {
- expect(sentrySpy).toHaveBeenCalledWith(
- 'Failed to find and scroll to node in tree'
- );
- });
- });
- it('triggers search on load', async () => {
- Object.defineProperty(window, 'location', {
- value: {
- search: '?search=transaction-op-5',
- },
- });
- await pageloadTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- expect(searchInput).toHaveValue('transaction-op-5');
- await waitFor(() => {
- expect(screen.getByTestId('trace-search-result-iterator')).toHaveTextContent(
- '1/1'
- );
- });
- });
- it('triggers search on load but does not steal focus from node param', async () => {
- Object.defineProperty(window, 'location', {
- value: {
- search: '?search=transaction-op-9999&node=txn-0',
- },
- });
- const {container} = await pageloadTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- expect(searchInput).toHaveValue('transaction-op-9999');
- await waitFor(() => {
- expect(screen.getByTestId('trace-search-result-iterator')).toHaveTextContent(
- '-/1'
- );
- });
- const rows = container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[1]).toHaveFocus();
- });
- it('if search on load does not match anything, it does not steal focus or highlight first result', async () => {
- Object.defineProperty(window, 'location', {
- value: {
- search: '?search=dead&node=txn-5',
- },
- });
- const {container} = await pageloadTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- expect(searchInput).toHaveValue('dead');
- await waitFor(() => {
- expect(screen.getByTestId('trace-search-result-iterator')).toHaveTextContent(
- 'no results'
- );
- });
- const rows = container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[6]).toHaveFocus();
- });
- });
- describe('keyboard navigation', () => {
- it('arrow down', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- await userEvent.click(rows[0]);
- await waitFor(() => expect(rows[0]).toHaveFocus());
- await userEvent.keyboard('{arrowdown}');
- await waitFor(() => expect(rows[1]).toHaveFocus());
- });
- it('arrow up', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- await userEvent.click(rows[1]);
- await waitFor(() => expect(rows[1]).toHaveFocus());
- await userEvent.keyboard('{arrowup}');
- await waitFor(() => expect(rows[0]).toHaveFocus());
- });
- // biome-ignore lint/suspicious/noSkippedTests: Flaky test
- it.skip('arrow right expands row and fetches data', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- mockSpansResponse(
- '0',
- {},
- {
- entries: [
- {type: EntryType.SPANS, data: [makeSpan({span_id: '0', op: 'special-span'})]},
- ],
- }
- );
- await userEvent.click(rows[1]);
- await waitFor(() => expect(rows[1]).toHaveFocus());
- await userEvent.keyboard('{arrowright}');
- expect(await screen.findByText('special-span')).toBeInTheDocument();
- });
- it('arrow left collapses row', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- mockSpansResponse(
- '0',
- {},
- {
- entries: [
- {type: EntryType.SPANS, data: [makeSpan({span_id: '0', op: 'special-span'})]},
- ],
- }
- );
- await userEvent.click(rows[1]);
- await waitFor(() => expect(rows[1]).toHaveFocus());
- await userEvent.keyboard('{arrowright}');
- expect(await screen.findByText('special-span')).toBeInTheDocument();
- await userEvent.keyboard('{arrowleft}');
- expect(screen.queryByText('special-span')).not.toBeInTheDocument();
- });
- it('roving updates the element in the drawer', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- mockSpansResponse(
- '0',
- {},
- {
- entries: [
- {type: EntryType.SPANS, data: [makeSpan({span_id: '0', op: 'special-span'})]},
- ],
- }
- );
- await userEvent.click(rows[1]);
- await waitFor(() => expect(rows[1]).toHaveFocus());
- expect(await screen.findByTestId('trace-drawer-title')).toHaveTextContent(
- 'transaction-op-0'
- );
- await userEvent.keyboard('{arrowright}');
- expect(await screen.findByText('special-span')).toBeInTheDocument();
- await userEvent.keyboard('{arrowdown}');
- await waitFor(() => expect(rows[2]).toHaveFocus());
- expect(await screen.findByTestId('trace-drawer-title')).toHaveTextContent(
- 'special-span'
- );
- });
- it('arrowup on first node jumps to start', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- let rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- await userEvent.click(rows[0]);
- await waitFor(() => expect(rows[0]).toHaveFocus());
- await userEvent.keyboard('{arrowup}');
- expect(
- await findByText(virtualizedContainer, /transaction-op-9999/i)
- ).toBeInTheDocument();
- await waitFor(() => {
- rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[rows.length - 1]).toHaveFocus();
- });
- });
- it('arrowdown on last node jumps to start', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- let rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- await userEvent.click(rows[0]);
- await waitFor(() => expect(rows[0]).toHaveFocus());
- await userEvent.keyboard('{arrowup}');
- expect(
- await findByText(virtualizedContainer, /transaction-op-9999/i)
- ).toBeInTheDocument();
- await waitFor(() => {
- rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[rows.length - 1]).toHaveFocus();
- });
- await userEvent.keyboard('{arrowdown}');
- expect(
- await findByText(virtualizedContainer, /transaction-op-0/i)
- ).toBeInTheDocument();
- await waitFor(() => {
- rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[0]).toHaveFocus();
- });
- });
- it('tab scrolls to next node', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- let rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- await userEvent.click(rows[0]);
- await waitFor(() => expect(rows[0]).toHaveFocus());
- await userEvent.keyboard('{tab}');
- await waitFor(() => {
- rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[1]).toHaveFocus();
- });
- });
- it('shift+tab scrolls to previous node', async () => {
- const {virtualizedContainer} = await keyboardNavigationTestSetup();
- let rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- await userEvent.click(rows[1]);
- await waitFor(() => expect(rows[1]).toHaveFocus());
- await userEvent.keyboard('{Shift>}{tab}{/Shift}');
- await waitFor(() => {
- rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[0]).toHaveFocus();
- });
- });
- it('arrowdown+shift scrolls to the end of the list', async () => {
- const {container, virtualizedContainer} = await keyboardNavigationTestSetup();
- let rows = container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- await userEvent.click(rows[0]);
- await waitFor(() => expect(rows[0]).toHaveFocus());
- await userEvent.keyboard('{Shift>}{arrowdown}{/Shift}');
- expect(
- await findByText(virtualizedContainer, /transaction-op-9999/i)
- ).toBeInTheDocument();
- await waitFor(() => {
- rows = container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[rows.length - 1]).toHaveFocus();
- });
- });
- it('arrowup+shift scrolls to the start of the list', async () => {
- const {container, virtualizedContainer} = await keyboardNavigationTestSetup();
- let rows = container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- await userEvent.click(rows[0]);
- await waitFor(() => expect(rows[0]).toHaveFocus());
- await userEvent.keyboard('{Shift>}{arrowdown}{/Shift}');
- expect(
- await findByText(virtualizedContainer, /transaction-op-9999/i)
- ).toBeInTheDocument();
- await waitFor(() => {
- rows = container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[rows.length - 1]).toHaveFocus();
- });
- await userEvent.keyboard('{Shift>}{arrowup}{/Shift}');
- expect(
- await findByText(virtualizedContainer, /transaction-op-0/i)
- ).toBeInTheDocument();
- await scrollToEnd();
- await waitFor(() => {
- rows = container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(rows[0]).toHaveFocus();
- });
- });
- });
- describe('search', () => {
- it('searches in transaction', async () => {
- const {container} = await searchTestSetup();
- let rows = Array.from(container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR));
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- fireEvent.change(searchInput, {target: {value: 'transaction-op'}});
- await waitFor(() => {
- const highlighted_row = container.querySelector(
- '.TraceRow:not(.Hidden).SearchResult.Highlight'
- );
- rows = Array.from(container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR));
- expect(rows.indexOf(highlighted_row!)).toBe(1);
- });
- });
- it('supports roving with arrowup and arrowdown', async () => {
- const {container} = await searchTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- // Fire change because userEvent triggers this letter by letter
- fireEvent.change(searchInput, {target: {value: 'transaction-op'}});
- // Wait for the search results to resolve
- await searchToUpdate();
- for (const action of [
- // starting at the top, jumpt bottom with shift+arrowdown
- ['{Shift>}{arrowdown}{/Shift}', 9],
- // // move to row above with arrowup
- ['{arrowup}', 8],
- // // and jump back to top with shift+arrowup
- ['{Shift>}{arrowup}{/Shift}', 1],
- // // // and jump to next row with arrowdown
- ['{arrowdown}', 2],
- ] as const) {
- await userEvent.keyboard(action[0] as string);
- // assert that focus on search input is never lost
- expect(searchInput).toHaveFocus();
- await waitFor(() => {
- // Only a single row is highlighted, the rest are search results
- assertHighlightedRowAtIndex(container, action[1]);
- });
- }
- });
- // @TODO I am torn on this because left-right
- // should probably also move the input cursor...
- // it.todo("supports expanding with arrowright")
- // it.todo("supports collapsing with arrowleft")
- it('search roving updates the element in the drawer', async () => {
- await searchTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- // Fire change because userEvent triggers this letter by letter
- fireEvent.change(searchInput, {target: {value: 'transaction-op'}});
- // Wait for the search results to resolve
- await searchToUpdate();
- expect(await screen.findByTestId('trace-drawer-title')).toHaveTextContent(
- 'transaction-op-0'
- );
- // assert that focus on search input is never lost
- expect(searchInput).toHaveFocus();
- await userEvent.keyboard('{arrowdown}');
- await waitFor(() => {
- expect(screen.getByTestId('trace-drawer-title')).toHaveTextContent(
- 'transaction-op-1'
- );
- });
- });
- it('highlighted node narrows down on the first result', async () => {
- const {container} = await searchTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- // Fire change because userEvent triggers this letter by letter
- fireEvent.change(searchInput, {target: {value: 'transaction-op-1'}});
- // Wait for the search results to resolve
- await searchToUpdate();
- assertHighlightedRowAtIndex(container, 2);
- fireEvent.change(searchInput, {target: {value: 'transaction-op-10'}});
- await searchToUpdate();
- await waitFor(() => {
- assertHighlightedRowAtIndex(container, 9);
- });
- });
- it('highlighted is persisted on node while it is part of the search results', async () => {
- const {container} = await searchTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- // Fire change because userEvent triggers this letter by letter
- fireEvent.change(searchInput, {target: {value: 'trans'}});
- // Wait for the search results to resolve
- await searchToUpdate();
- await userEvent.keyboard('{arrowdown}');
- await searchToUpdate();
- assertHighlightedRowAtIndex(container, 2);
- fireEvent.change(searchInput, {target: {value: 'transa'}});
- await searchToUpdate();
- // Highlighting is persisted on the row
- assertHighlightedRowAtIndex(container, 2);
- fireEvent.change(searchInput, {target: {value: 'this wont match anything'}});
- await searchToUpdate();
- // When there is no match, the highlighting is removed
- expect(container.querySelectorAll('.TraceRow.Highlight')).toHaveLength(0);
- });
- it('auto highlights the first result when search begins', async () => {
- const {container} = await searchTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- // Nothing is highlighted
- expect(container.querySelectorAll('.TraceRow.Highlight')).toHaveLength(0);
- // Fire change because userEvent triggers this letter by letter
- fireEvent.change(searchInput, {target: {value: 't'}});
- // Wait for the search results to resolve
- await searchToUpdate();
- assertHighlightedRowAtIndex(container, 1);
- });
- it('clicking a row that is also a search result updates the result index', async () => {
- const {container} = await searchTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- // Fire change because userEvent triggers this letter by letter
- fireEvent.change(searchInput, {target: {value: 'transaction-op-1'}});
- await searchToUpdate();
- assertHighlightedRowAtIndex(container, 2);
- const rows = container.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- // By default, we highlight the first result
- expect(await screen.findByTestId('trace-search-result-iterator')).toHaveTextContent(
- '1/2'
- );
- await scrollToEnd();
- // Click on a random row in the list that is not a search result
- await userEvent.click(rows[5]);
- await waitFor(() => {
- expect(screen.queryByTestId('trace-search-result-iterator')).toHaveTextContent(
- '-/2'
- );
- });
- await scrollToEnd();
- // Click on a the row in the list that is a search result
- await userEvent.click(rows[2]);
- await waitFor(() => {
- expect(screen.queryByTestId('trace-search-result-iterator')).toHaveTextContent(
- '1/2'
- );
- });
- });
- it('during search, expanding a row retriggers search', async () => {
- mockTraceMetaResponse();
- mockTraceRootFacets();
- mockTraceRootEvent('0', {body: DetailedEventsFixture()[0]});
- mockTraceEventDetails();
- mockTraceResponse({
- body: {
- transactions: [
- makeTransaction({
- span_id: '0',
- event_id: '0',
- transaction: 'transaction-name-0',
- 'transaction.op': 'transaction-op-0',
- }),
- ],
- orphan_errors: [],
- },
- });
- mockSpansResponse(
- '0',
- {},
- {
- entries: [
- {
- type: EntryType.SPANS,
- data: [
- makeSpan({span_id: '0', description: 'span-description', op: 'op-0'}),
- ],
- },
- ],
- }
- );
- const value = render(<TraceViewWithProviders traceSlug="trace-id" />);
- // Awaits for the placeholder rendering rows to be removed
- expect(await findByText(value.container, /transaction-op-0/i)).toBeInTheDocument();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- // Fire change because userEvent triggers this letter by letter
- fireEvent.change(searchInput, {target: {value: 'op-0'}});
- await searchToUpdate();
- expect(await screen.findByTestId('trace-search-result-iterator')).toHaveTextContent(
- '1/1'
- );
- const highlighted_row = value.container.querySelector(ACTIVE_SEARCH_HIGHLIGHT_ROW);
- await userEvent.click(await screen.findByRole('button', {name: '+'}));
- expect(await screen.findByText('span-description')).toBeInTheDocument();
- await searchToUpdate();
- // The search is retriggered, but highlighting of current row is preserved
- expect(value.container.querySelector(ACTIVE_SEARCH_HIGHLIGHT_ROW)).toBe(
- highlighted_row
- );
- expect(await screen.findByTestId('trace-search-result-iterator')).toHaveTextContent(
- '1/2'
- );
- });
- it('during search, highlighting is persisted on the row', async () => {
- const {container} = await searchTestSetup();
- const searchInput = await screen.findByPlaceholderText('Search in trace');
- await userEvent.click(searchInput);
- // Fire change because userEvent triggers this letter by letter
- fireEvent.change(searchInput, {target: {value: 'transaction-op'}});
- await searchToUpdate();
- assertHighlightedRowAtIndex(container, 1);
- await searchToUpdate();
- // User moves down the list using keyboard navigation
- for (const _ of [1, 2, 3, 4, 5]) {
- const initial = screen.getByTestId('trace-search-result-iterator').textContent;
- await userEvent.keyboard('{arrowDown}');
- await waitFor(() => {
- expect(screen.getByTestId('trace-search-result-iterator')).not.toBe(initial);
- });
- }
- // User clicks on an entry in the list, then proceeds to search
- await waitFor(() => {
- expect(screen.getByTestId('trace-search-result-iterator')).toHaveTextContent(
- '6/11'
- );
- });
- // And then continues the query - the highlighting is preserved as long as the
- // rwo is part of the search results
- assertHighlightedRowAtIndex(container, 6);
- fireEvent.change(searchInput, {target: {value: 'transaction-op-'}});
- await searchToUpdate();
- assertHighlightedRowAtIndex(container, 6);
- fireEvent.change(searchInput, {target: {value: 'transaction-op-5'}});
- await searchToUpdate();
- assertHighlightedRowAtIndex(container, 6);
- fireEvent.change(searchInput, {target: {value: 'transaction-op-none'}});
- await searchToUpdate();
- expect(container.querySelectorAll('.TraceRow.Highlight')).toHaveLength(0);
- });
- });
- describe('tabbing', () => {
- beforeEach(() => {
- jest.spyOn(console, 'error').mockImplementation();
- });
- afterEach(() => {
- jest.restoreAllMocks();
- });
- it('clicking on a node spawns a new tab when none is selected', async () => {
- const {virtualizedContainer} = await simpleTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(1);
- await userEvent.click(rows[5]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(2);
- });
- });
- it('clicking on a node replaces the previously selected tab', async () => {
- const {virtualizedContainer} = await simpleTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(1);
- await userEvent.click(rows[5]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(2);
- expect(
- screen
- .queryAllByTestId(DRAWER_TABS_TEST_ID)[1]
- .textContent?.includes('transaction-op-4')
- ).toBeTruthy();
- });
- await userEvent.click(rows[7]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(2);
- expect(
- screen
- .queryAllByTestId(DRAWER_TABS_TEST_ID)[1]
- .textContent?.includes('transaction-op-6')
- ).toBeTruthy();
- });
- });
- it('pinning a tab and clicking on a new node spawns a new tab', async () => {
- const {virtualizedContainer} = await simpleTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(1);
- await userEvent.click(rows[5]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(2);
- });
- await userEvent.click(await screen.findByTestId(DRAWER_TABS_PIN_BUTTON_TEST_ID));
- await userEvent.click(rows[7]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(3);
- expect(
- screen
- .queryAllByTestId(DRAWER_TABS_TEST_ID)[1]
- .textContent?.includes('transaction-op-4')
- ).toBeTruthy();
- expect(
- screen
- .queryAllByTestId(DRAWER_TABS_TEST_ID)[2]
- .textContent?.includes('transaction-op-6')
- ).toBeTruthy();
- });
- });
- it('unpinning a tab removes it', async () => {
- const {virtualizedContainer} = await simpleTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(1);
- await userEvent.click(rows[5]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(2);
- });
- await userEvent.click(await screen.findByTestId(DRAWER_TABS_PIN_BUTTON_TEST_ID));
- await userEvent.click(rows[7]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(3);
- });
- const tabButtons = screen.queryAllByTestId(DRAWER_TABS_PIN_BUTTON_TEST_ID);
- expect(tabButtons).toHaveLength(2);
- await userEvent.click(tabButtons[0]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(2);
- });
- });
- it('clicking a node that is already open in a tab switches to that tab and persists the previous node', async () => {
- const {virtualizedContainer} = await simpleTestSetup();
- const rows = virtualizedContainer.querySelectorAll(VISIBLE_TRACE_ROW_SELECTOR);
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(1);
- await userEvent.click(rows[5]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(2);
- });
- await userEvent.click(await screen.findByTestId(DRAWER_TABS_PIN_BUTTON_TEST_ID));
- await userEvent.click(rows[7]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(3);
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)[2]).toHaveAttribute(
- 'aria-selected',
- 'true'
- );
- });
- await userEvent.click(rows[5]);
- await waitFor(() => {
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)[1]).toHaveAttribute(
- 'aria-selected',
- 'true'
- );
- expect(screen.queryAllByTestId(DRAWER_TABS_TEST_ID)).toHaveLength(3);
- });
- });
- });
- });
|