Skip to content

Commit a472c05

Browse files
ref: use additionalWrapper as suggested
1 parent bc25c6d commit a472c05

File tree

1 file changed

+72
-76
lines changed

1 file changed

+72
-76
lines changed

static/app/views/explore/logs/tables/logsTableRow.spec.tsx

Lines changed: 72 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -213,17 +213,15 @@ describe('logsTableRow', () => {
213213
jest.useFakeTimers();
214214
expect(rowDetailsMock).toHaveBeenCalledTimes(0);
215215
render(
216-
<ProviderWrapper>
217-
<LogRowContent
218-
dataRow={rowData}
219-
highlightTerms={[]}
220-
meta={LogFixtureMeta(rowData)}
221-
sharedHoverTimeoutRef={
222-
{current: null} as React.MutableRefObject<NodeJS.Timeout | null>
223-
}
224-
/>
225-
</ProviderWrapper>,
226-
{organization, initialRouterConfig}
216+
<LogRowContent
217+
dataRow={rowData}
218+
highlightTerms={[]}
219+
meta={LogFixtureMeta(rowData)}
220+
sharedHoverTimeoutRef={
221+
{current: null} as React.MutableRefObject<NodeJS.Timeout | null>
222+
}
223+
/>,
224+
{organization, initialRouterConfig, additionalWrapper: ProviderWrapper}
227225
);
228226

229227
expect(screen.queryByLabelText('Toggle trace details')).not.toBeInTheDocument(); // Fake button
@@ -247,19 +245,17 @@ describe('logsTableRow', () => {
247245

248246
it('renders row details', async () => {
249247
render(
250-
<ProviderWrapper>
251-
<LogRowContent
252-
dataRow={rowData}
253-
highlightTerms={[]}
254-
meta={LogFixtureMeta(rowData)}
255-
sharedHoverTimeoutRef={
256-
{
257-
current: null,
258-
} as React.MutableRefObject<NodeJS.Timeout | null>
259-
}
260-
/>
261-
</ProviderWrapper>,
262-
{organization, initialRouterConfig}
248+
<LogRowContent
249+
dataRow={rowData}
250+
highlightTerms={[]}
251+
meta={LogFixtureMeta(rowData)}
252+
sharedHoverTimeoutRef={
253+
{
254+
current: null,
255+
} as React.MutableRefObject<NodeJS.Timeout | null>
256+
}
257+
/>,
258+
{organization, initialRouterConfig, additionalWrapper: ProviderWrapper}
263259
);
264260

265261
// Check that the log body and timestamp are rendered
@@ -349,19 +345,21 @@ describe('logsTableRow', () => {
349345

350346
it('shows a link when hovering over code file path in the table', async () => {
351347
render(
352-
<ProviderWrapper>
353-
<LogRowContent
354-
dataRow={rowDataWithCodeFilePath}
355-
highlightTerms={[]}
356-
meta={LogFixtureMeta(rowDataWithCodeFilePath)}
357-
sharedHoverTimeoutRef={
358-
{
359-
current: null,
360-
} as React.MutableRefObject<NodeJS.Timeout | null>
361-
}
362-
/>
363-
</ProviderWrapper>,
364-
{organization, initialRouterConfig: initialRouterConfigWithCodeFilePath}
348+
<LogRowContent
349+
dataRow={rowDataWithCodeFilePath}
350+
highlightTerms={[]}
351+
meta={LogFixtureMeta(rowDataWithCodeFilePath)}
352+
sharedHoverTimeoutRef={
353+
{
354+
current: null,
355+
} as React.MutableRefObject<NodeJS.Timeout | null>
356+
}
357+
/>,
358+
{
359+
organization,
360+
initialRouterConfig: initialRouterConfigWithCodeFilePath,
361+
additionalWrapper: ProviderWrapper,
362+
}
365363
);
366364

367365
// Expand the row to show the attributes
@@ -430,19 +428,17 @@ describe('logsTableRow', () => {
430428
});
431429

432430
render(
433-
<ProviderWrapper>
434-
<LogRowContent
435-
dataRow={rowData}
436-
highlightTerms={[]}
437-
meta={LogFixtureMeta(rowData)}
438-
sharedHoverTimeoutRef={
439-
{
440-
current: null,
441-
} as React.MutableRefObject<NodeJS.Timeout | null>
442-
}
443-
/>
444-
</ProviderWrapper>,
445-
{organization, initialRouterConfig}
431+
<LogRowContent
432+
dataRow={rowData}
433+
highlightTerms={[]}
434+
meta={LogFixtureMeta(rowData)}
435+
sharedHoverTimeoutRef={
436+
{
437+
current: null,
438+
} as React.MutableRefObject<NodeJS.Timeout | null>
439+
}
440+
/>,
441+
{organization, initialRouterConfig, additionalWrapper: ProviderWrapper}
446442
);
447443

448444
// Expand the row to show the action buttons
@@ -499,19 +495,17 @@ describe('logsTableRow', () => {
499495
});
500496

501497
render(
502-
<ProviderWrapper>
503-
<LogRowContent
504-
dataRow={rowData}
505-
highlightTerms={[]}
506-
meta={LogFixtureMeta(rowData)}
507-
sharedHoverTimeoutRef={
508-
{
509-
current: null,
510-
} as React.MutableRefObject<NodeJS.Timeout | null>
511-
}
512-
/>
513-
</ProviderWrapper>,
514-
{organization, initialRouterConfig}
498+
<LogRowContent
499+
dataRow={rowData}
500+
highlightTerms={[]}
501+
meta={LogFixtureMeta(rowData)}
502+
sharedHoverTimeoutRef={
503+
{
504+
current: null,
505+
} as React.MutableRefObject<NodeJS.Timeout | null>
506+
}
507+
/>,
508+
{organization, initialRouterConfig, additionalWrapper: ProviderWrapper}
515509
);
516510

517511
const logTableRow = await screen.findByTestId('log-table-row');
@@ -580,19 +574,21 @@ describe('logsTableRow', () => {
580574
});
581575

582576
render(
583-
<ProviderWrapper>
584-
<LogRowContent
585-
dataRow={rowDataWithScrubbedFields}
586-
highlightTerms={[]}
587-
meta={LogFixtureMeta(rowDataWithScrubbedFields)}
588-
sharedHoverTimeoutRef={
589-
{
590-
current: null,
591-
} as React.MutableRefObject<NodeJS.Timeout | null>
592-
}
593-
/>
594-
</ProviderWrapper>,
595-
{organization, initialRouterConfig: initialRouterConfigWithScrubbedFields}
577+
<LogRowContent
578+
dataRow={rowDataWithScrubbedFields}
579+
highlightTerms={[]}
580+
meta={LogFixtureMeta(rowDataWithScrubbedFields)}
581+
sharedHoverTimeoutRef={
582+
{
583+
current: null,
584+
} as React.MutableRefObject<NodeJS.Timeout | null>
585+
}
586+
/>,
587+
{
588+
organization,
589+
initialRouterConfig: initialRouterConfigWithScrubbedFields,
590+
additionalWrapper: ProviderWrapper,
591+
}
596592
);
597593

598594
const logTableRow = await screen.findByTestId('log-table-row');

0 commit comments

Comments
 (0)