reactuse icon indicating copy to clipboard operation
reactuse copied to clipboard

[test]: useHover

Open wmoooid opened this issue 1 year ago • 5 comments

сделал тесты для useHover closes #37

wmoooid avatar May 28 '24 15:05 wmoooid

@debabin когда писал тест пришлось перекрывать тип рефа, иначе тс ругался. Кажется, с этим надо что-то сделать.

const MockComponent = () => {
      const [ref, isHovering] = useHover({});

      return (
        <div data-testid='target' ref={ref as React.RefObject<HTMLDivElement>}>
          {isHovering ? 'true' : 'false'}
        </div>
      );
    };

Текст ошибки:

Type 'RefObject<UseHoverTarget>' is not assignable to type 'LegacyRef<HTMLDivElement> | undefined'. Type 'RefObject<UseHoverTarget>' is not assignable to type 'RefObject<HTMLDivElement>'. Type 'UseHoverTarget' is not assignable to type 'HTMLDivElement'. Type 'Element' is missing the following properties from type 'HTMLDivElement': align, accessKey, accessKeyLabel, autocapitalize, and 124 more.ts(2322) index.d.ts(303, 9): The expected type comes from property 'ref' which is declared here on type 'DetailedHTMLProps<HTMLAttributes<HTMLDivElement>, HTMLDivElement>'

wmoooid avatar May 28 '24 15:05 wmoooid

@wmoooid в useHover надо просто дженерик прокинуть useHover<HTMLDivElement> Надо бы обновить пример в описание хука @debabin

str0yka avatar Jun 03 '24 16:06 str0yka

@wmoooid в useHover надо просто дженерик прокинуть useHover<HTMLDivElement> Надо бы обновить пример в описание хука @debabin

Пример для js кода, а generic это дефалт для useRef и тут тоже самое

debabin avatar Jun 03 '24 18:06 debabin

@wmoooid в useHover надо просто дженерик прокинуть useHover<HTMLDivElement>

@str0yka Да, спасибо, не заметил, что тип можно прокинуть

wmoooid avatar Jun 04 '24 11:06 wmoooid

@debabin переделал тесты, посмотри пожалуйста

wmoooid avatar Jun 04 '24 11:06 wmoooid

outdated

debabin avatar Apr 29 '25 05:04 debabin