httpSamplesPanel.spec.tsx 14 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440
  1. import {OrganizationFixture} from 'sentry-fixture/organization';
  2. import {
  3. render,
  4. screen,
  5. userEvent,
  6. waitForElementToBeRemoved,
  7. } from 'sentry-test/reactTestingLibrary';
  8. import {useLocation} from 'sentry/utils/useLocation';
  9. import useOrganization from 'sentry/utils/useOrganization';
  10. import usePageFilters from 'sentry/utils/usePageFilters';
  11. import {HTTPSamplesPanel} from 'sentry/views/performance/http/httpSamplesPanel';
  12. jest.mock('sentry/utils/useLocation');
  13. jest.mock('sentry/utils/usePageFilters');
  14. jest.mock('sentry/utils/useOrganization');
  15. describe('HTTPSamplesPanel', () => {
  16. const organization = OrganizationFixture();
  17. let eventsRequestMock;
  18. jest.mocked(usePageFilters).mockReturnValue({
  19. isReady: true,
  20. desyncedFilters: new Set(),
  21. pinnedFilters: new Set(),
  22. shouldPersist: true,
  23. selection: {
  24. datetime: {
  25. period: '10d',
  26. start: null,
  27. end: null,
  28. utc: false,
  29. },
  30. environments: [],
  31. projects: [],
  32. },
  33. });
  34. jest.mocked(useLocation).mockReturnValue({
  35. pathname: '',
  36. search: '',
  37. query: {
  38. domain: '*.sentry.dev',
  39. statsPeriod: '10d',
  40. transaction: '/api/0/users',
  41. transactionMethod: 'GET',
  42. panel: 'status',
  43. },
  44. hash: '',
  45. state: undefined,
  46. action: 'PUSH',
  47. key: '',
  48. });
  49. jest.mocked(useOrganization).mockReturnValue(organization);
  50. beforeEach(() => {
  51. jest.clearAllMocks();
  52. eventsRequestMock = MockApiClient.addMockResponse({
  53. url: `/organizations/${organization.slug}/events/`,
  54. method: 'GET',
  55. match: [
  56. MockApiClient.matchQuery({
  57. referrer: 'api.starfish.http-module-samples-panel-metrics-ribbon',
  58. }),
  59. ],
  60. body: {
  61. data: [
  62. {
  63. 'project.id': 1,
  64. 'transaction.id': '',
  65. 'spm()': 22.18,
  66. 'http_response_rate(3)': 0.01,
  67. 'http_response_rate(4)': 0.025,
  68. 'http_response_rate(5)': 0.015,
  69. 'avg(span.self_time)': 140.2,
  70. 'sum(span.self_time)': 2709238,
  71. },
  72. ],
  73. meta: {
  74. fields: {
  75. 'spm()': 'rate',
  76. 'avg(span.self_time)': 'duration',
  77. 'http_response_rate(3)': 'percentage',
  78. 'http_response_rate(4)': 'percentage',
  79. 'http_response_rate(5)': 'percentage',
  80. 'sum(span.self_time)': 'duration',
  81. },
  82. },
  83. },
  84. });
  85. });
  86. afterAll(() => {
  87. jest.resetAllMocks();
  88. });
  89. describe('Status panel', () => {
  90. let eventsStatsRequestMock, samplesRequestMock;
  91. beforeEach(() => {
  92. jest.mocked(useLocation).mockReturnValue({
  93. pathname: '',
  94. search: '',
  95. query: {
  96. domain: '*.sentry.dev',
  97. statsPeriod: '10d',
  98. transaction: '/api/0/users',
  99. transactionMethod: 'GET',
  100. panel: 'status',
  101. responseCodeClass: '3',
  102. },
  103. hash: '',
  104. state: undefined,
  105. action: 'PUSH',
  106. key: '',
  107. });
  108. eventsStatsRequestMock = MockApiClient.addMockResponse({
  109. url: `/organizations/${organization.slug}/events-stats/`,
  110. method: 'GET',
  111. match: [
  112. MockApiClient.matchQuery({
  113. referrer: 'api.starfish.http-module-samples-panel-response-code-chart',
  114. }),
  115. ],
  116. body: {
  117. '301': {
  118. data: [
  119. [1699907700, [{count: 7810.2}]],
  120. [1699908000, [{count: 1216.8}]],
  121. ],
  122. },
  123. '304': {
  124. data: [
  125. [1699907700, [{count: 2701.5}]],
  126. [1699908000, [{count: 78.12}]],
  127. ],
  128. },
  129. },
  130. });
  131. samplesRequestMock = MockApiClient.addMockResponse({
  132. url: `/organizations/${organization.slug}/events/`,
  133. method: 'GET',
  134. match: [
  135. MockApiClient.matchQuery({
  136. referrer: 'api.starfish.http-module-samples-panel-response-code-samples',
  137. }),
  138. ],
  139. body: {
  140. data: [
  141. {
  142. span_id: 'b1bf1acde131623a',
  143. trace: '2b60b2eb415c4bfba3efeaf65c21c605',
  144. 'span.description':
  145. 'GET https://sentry.io/api/0/organizations/sentry/info/?projectId=1',
  146. project: 'javascript',
  147. timestamp: '2024-03-25T20:31:36+00:00',
  148. 'span.status_code': '200',
  149. 'transaction.id': '11c910c9c10b3ec4ecf8f209b8c6ce48',
  150. 'span.self_time': 320.300102,
  151. },
  152. ],
  153. meta: {},
  154. },
  155. });
  156. });
  157. it('fetches panel data', async () => {
  158. render(<HTTPSamplesPanel />);
  159. expect(eventsRequestMock).toHaveBeenNthCalledWith(
  160. 1,
  161. `/organizations/${organization.slug}/events/`,
  162. expect.objectContaining({
  163. method: 'GET',
  164. query: {
  165. dataset: 'spansMetrics',
  166. environment: [],
  167. field: [
  168. 'spm()',
  169. 'avg(span.self_time)',
  170. 'sum(span.self_time)',
  171. 'http_response_rate(3)',
  172. 'http_response_rate(4)',
  173. 'http_response_rate(5)',
  174. 'time_spent_percentage()',
  175. ],
  176. per_page: 50,
  177. project: [],
  178. query:
  179. 'span.module:http span.domain:"\\*.sentry.dev" transaction:/api/0/users',
  180. referrer: 'api.starfish.http-module-samples-panel-metrics-ribbon',
  181. statsPeriod: '10d',
  182. },
  183. })
  184. );
  185. expect(eventsStatsRequestMock).toHaveBeenNthCalledWith(
  186. 1,
  187. `/organizations/${organization.slug}/events-stats/`,
  188. expect.objectContaining({
  189. method: 'GET',
  190. query: {
  191. cursor: undefined,
  192. dataset: 'spansMetrics',
  193. environment: [],
  194. excludeOther: 0,
  195. field: ['span.status_code', 'count()'],
  196. interval: '30m',
  197. orderby: undefined,
  198. partial: 1,
  199. per_page: 50,
  200. project: [],
  201. query:
  202. 'span.module:http span.domain:"\\*.sentry.dev" transaction:/api/0/users span.status_code:[300,301,302,303,304,305,307,308]',
  203. referrer: 'api.starfish.http-module-samples-panel-response-code-chart',
  204. statsPeriod: '10d',
  205. topEvents: '5',
  206. yAxis: 'count()',
  207. },
  208. })
  209. );
  210. expect(samplesRequestMock).toHaveBeenNthCalledWith(
  211. 1,
  212. `/organizations/${organization.slug}/events/`,
  213. expect.objectContaining({
  214. method: 'GET',
  215. query: expect.objectContaining({
  216. dataset: 'spansIndexed',
  217. query:
  218. 'span.module:http span.domain:"\\*.sentry.dev" transaction:/api/0/users span.status_code:[300,301,302,303,304,305,307,308]',
  219. project: [],
  220. field: [
  221. 'project',
  222. 'trace',
  223. 'transaction.id',
  224. 'span_id',
  225. 'timestamp',
  226. 'span.description',
  227. 'span.status_code',
  228. ],
  229. sort: '-span_id',
  230. referrer: 'api.starfish.http-module-samples-panel-response-code-samples',
  231. statsPeriod: '10d',
  232. }),
  233. })
  234. );
  235. await waitForElementToBeRemoved(() => screen.queryAllByTestId('loading-indicator'));
  236. });
  237. it('shows basic transaction info', async () => {
  238. render(<HTTPSamplesPanel />);
  239. // Panel heading
  240. expect(screen.getByRole('heading', {name: 'GET /api/0/users'})).toBeInTheDocument();
  241. // Metrics ribbon
  242. await waitForElementToBeRemoved(() => screen.queryAllByTestId('loading-indicator'));
  243. expect(
  244. screen.getByRole('heading', {name: 'Requests Per Minute'})
  245. ).toBeInTheDocument();
  246. expect(screen.getByRole('heading', {name: 'Avg Duration'})).toBeInTheDocument();
  247. expect(screen.getByRole('heading', {name: '3XXs'})).toBeInTheDocument();
  248. expect(screen.getByRole('heading', {name: '4XXs'})).toBeInTheDocument();
  249. expect(screen.getByRole('heading', {name: '5XXs'})).toBeInTheDocument();
  250. expect(screen.getByRole('heading', {name: 'Time Spent'})).toBeInTheDocument();
  251. expect(screen.getByText('22.2/min')).toBeInTheDocument();
  252. expect(screen.getByText('140.20ms')).toBeInTheDocument();
  253. expect(screen.getByText('1%')).toBeInTheDocument();
  254. expect(screen.getByText('2.5%')).toBeInTheDocument();
  255. expect(screen.getByText('1.5%')).toBeInTheDocument();
  256. expect(screen.getByText('45.15min')).toBeInTheDocument();
  257. });
  258. });
  259. describe('Duration panel', () => {
  260. let chartRequestMock, samplesRequestMock;
  261. beforeEach(() => {
  262. jest.mocked(useLocation).mockReturnValue({
  263. pathname: '',
  264. search: '',
  265. query: {
  266. domain: '*.sentry.dev',
  267. statsPeriod: '10d',
  268. transaction: '/api/0/users',
  269. transactionMethod: 'GET',
  270. panel: 'duration',
  271. },
  272. hash: '',
  273. state: undefined,
  274. action: 'PUSH',
  275. key: '',
  276. });
  277. chartRequestMock = MockApiClient.addMockResponse({
  278. url: `/organizations/${organization.slug}/events-stats/`,
  279. method: 'GET',
  280. match: [
  281. MockApiClient.matchQuery({
  282. referrer: 'api.starfish.http-module-samples-panel-duration-chart',
  283. }),
  284. ],
  285. body: {data: [[1711393200, [{count: 900}]]]},
  286. });
  287. samplesRequestMock = MockApiClient.addMockResponse({
  288. url: `/api/0/organizations/${organization.slug}/spans-samples/`,
  289. method: 'GET',
  290. body: {
  291. data: [
  292. {
  293. span_id: 'b1bf1acde131623a',
  294. trace: '2b60b2eb415c4bfba3efeaf65c21c605',
  295. 'span.description':
  296. 'GET https://sentry.io/api/0/organizations/sentry/info/?projectId=1',
  297. project: 'javascript',
  298. timestamp: '2024-03-25T20:31:36+00:00',
  299. 'span.status_code': '200',
  300. 'transaction.id': '11c910c9c10b3ec4ecf8f209b8c6ce48',
  301. 'span.self_time': 320.300102,
  302. },
  303. ],
  304. },
  305. });
  306. });
  307. it('fetches panel data', async () => {
  308. render(<HTTPSamplesPanel />);
  309. await waitForElementToBeRemoved(() => screen.queryAllByTestId('loading-indicator'));
  310. expect(chartRequestMock).toHaveBeenNthCalledWith(
  311. 1,
  312. `/organizations/${organization.slug}/events-stats/`,
  313. expect.objectContaining({
  314. method: 'GET',
  315. query: expect.objectContaining({
  316. dataset: 'spansMetrics',
  317. environment: [],
  318. interval: '30m',
  319. per_page: 50,
  320. project: [],
  321. query:
  322. 'span.module:http span.domain:"\\*.sentry.dev" transaction:/api/0/users',
  323. referrer: 'api.starfish.http-module-samples-panel-duration-chart',
  324. statsPeriod: '10d',
  325. yAxis: 'avg(span.self_time)',
  326. }),
  327. })
  328. );
  329. expect(samplesRequestMock).toHaveBeenNthCalledWith(
  330. 1,
  331. `/api/0/organizations/${organization.slug}/spans-samples/`,
  332. expect.objectContaining({
  333. method: 'GET',
  334. query: expect.objectContaining({
  335. query:
  336. 'span.module:http span.domain:"\\*.sentry.dev" transaction:/api/0/users',
  337. project: [],
  338. additionalFields: [
  339. 'trace',
  340. 'transaction.id',
  341. 'span.description',
  342. 'span.status_code',
  343. ],
  344. lowerBound: 0,
  345. firstBound: expect.closeTo(333.3333),
  346. secondBound: expect.closeTo(666.6666),
  347. upperBound: 1000,
  348. referrer: 'api.starfish.http-module-samples-panel-duration-samples',
  349. statsPeriod: '10d',
  350. }),
  351. })
  352. );
  353. });
  354. it('show basic transaction info', async () => {
  355. render(<HTTPSamplesPanel />);
  356. // Panel heading
  357. expect(screen.getByRole('heading', {name: 'GET /api/0/users'})).toBeInTheDocument();
  358. // Metrics ribbon
  359. await waitForElementToBeRemoved(() => screen.queryAllByTestId('loading-indicator'));
  360. expect(
  361. screen.getByRole('heading', {name: 'Requests Per Minute'})
  362. ).toBeInTheDocument();
  363. expect(screen.getByRole('heading', {name: 'Avg Duration'})).toBeInTheDocument();
  364. expect(screen.getByRole('heading', {name: '3XXs'})).toBeInTheDocument();
  365. expect(screen.getByRole('heading', {name: '4XXs'})).toBeInTheDocument();
  366. expect(screen.getByRole('heading', {name: '5XXs'})).toBeInTheDocument();
  367. expect(screen.getByRole('heading', {name: 'Time Spent'})).toBeInTheDocument();
  368. expect(screen.getByText('22.2/min')).toBeInTheDocument();
  369. expect(screen.getByText('140.20ms')).toBeInTheDocument();
  370. expect(screen.getByText('1%')).toBeInTheDocument();
  371. expect(screen.getByText('2.5%')).toBeInTheDocument();
  372. expect(screen.getByText('1.5%')).toBeInTheDocument();
  373. expect(screen.getByText('45.15min')).toBeInTheDocument();
  374. // Samples table
  375. expect(screen.getByRole('table', {name: 'Span Samples'})).toBeInTheDocument();
  376. expect(screen.getByRole('columnheader', {name: 'Span ID'})).toBeInTheDocument();
  377. expect(screen.getByRole('columnheader', {name: 'Status'})).toBeInTheDocument();
  378. expect(screen.getByRole('columnheader', {name: 'URL'})).toBeInTheDocument();
  379. expect(screen.getByRole('cell', {name: 'b1bf1acde131623a'})).toBeInTheDocument();
  380. expect(screen.getByRole('link', {name: 'b1bf1acde131623a'})).toHaveAttribute(
  381. 'href',
  382. '/organizations/org-slug/performance/javascript:11c910c9c10b3ec4ecf8f209b8c6ce48/?domain=%2A.sentry.dev&panel=duration&statsPeriod=10d&transactionMethod=GET'
  383. );
  384. expect(screen.getByRole('cell', {name: '200'})).toBeInTheDocument();
  385. });
  386. it('re-fetches samples', async () => {
  387. render(<HTTPSamplesPanel />);
  388. await waitForElementToBeRemoved(() => screen.queryAllByTestId('loading-indicator'));
  389. expect(samplesRequestMock).toHaveBeenCalledTimes(1);
  390. await userEvent.click(screen.getByRole('button', {name: 'Try Different Samples'}));
  391. expect(samplesRequestMock).toHaveBeenCalledTimes(2);
  392. });
  393. });
  394. });