12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325 |
- import {act} from 'react-dom/test-utils';
- 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 {
- 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) {
- throw new Error('not implemented');
- }
- 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 ?? {}),
- });
- }
- function mockTraceRootEvent(id: string, resp?: Partial<ResponseType>) {
- MockApiClient.addMockResponse({
- url: `/organizations/org-slug/events/project_slug:${id}/`,
- method: 'GET',
- asyncDelay: 1,
- ...(resp ?? {}),
- });
- }
- 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();
- 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;
- 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());
- });
- it('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);
- });
- await userEvent.click(
- await screen.findAllByTestId(DRAWER_TABS_PIN_BUTTON_TEST_ID)[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);
- });
- });
- });
- });
|