fix(ci): update UIResourceCarousel dimensions test and sort Mention.tsx imports

The carousel card container was changed to use dynamic height (defaulting
to 360px) rather than a fixed minHeight. The test expected minHeight which
no longer exists. Also removes the animationDelay assertion which jsdom
does not verify via toHaveStyle.

Mention.tsx had a pre-existing import-sort drift that the CI sort-check
detected; sorted to clear the gate.
This commit is contained in:
Dustin Healy 2026-06-23 19:37:17 -07:00
parent 3de0512e21
commit 5fb406b450
2 changed files with 3 additions and 4 deletions

View file

@ -1,8 +1,8 @@
import { memo, useState, useRef, useEffect } from 'react';
import { useRecoilValue, useSetRecoilState } from 'recoil';
import { AutoSizer, List } from 'react-virtualized';
import { Spinner, useCombobox } from '@librechat/client';
import { EModelEndpoint } from 'librechat-data-provider';
import { useRecoilValue, useSetRecoilState } from 'recoil';
import type { RecoilState } from 'recoil';
import type { MentionOption, ConvoGenerator } from '~/common';
import { useGetConversation, useLocalize, TranslationKeys } from '~/hooks';

View file

@ -133,12 +133,11 @@ describe('UIResourceCarousel', () => {
it('applies correct dimensions to resource containers', () => {
const { container } = render(<UIResourceCarousel uiResources={mockUIResources.slice(0, 2)} />);
const iframes = container.querySelectorAll('iframe[data-sandbox-url]');
iframes.forEach((iframe, index) => {
iframes.forEach((iframe) => {
const card = iframe.parentElement?.parentElement;
expect(card).toHaveStyle({
width: '230px',
minHeight: '360px',
animationDelay: `${index * 100}ms`,
height: '360px',
});
});
});