Почему getComputedStyle () в тесте JEST возвращает разные результаты вычисленным стилям в Chrome / Firefox DevTools

16

Я написал пользовательскую кнопку ( MyStyledButton) на основе материала-интерфейса Button .

import React from "react";
import { Button } from "@material-ui/core";
import { makeStyles } from "@material-ui/styles";

const useStyles = makeStyles({
  root: {
    minWidth: 100
  }
});

function MyStyledButton(props) {
  const buttonStyle = useStyles(props);
  const { children, width, ...others } = props;

  return (

      <Button classes={{ root: buttonStyle.root }} {...others}>
        {children}
      </Button>
     );
}

export default MyStyledButton;

Это стилизовано с использованием темы, и это указывает, backgroundColorчтобы быть оттенком желтого (Специально #fbb900)

import { createMuiTheme } from "@material-ui/core/styles";

export const myYellow = "#FBB900";

export const theme = createMuiTheme({
  overrides: {
    MuiButton: {
      containedPrimary: {
        color: "black",
        backgroundColor: myYellow
      }
    }
  }
});

Компонент создается в моем основном index.jsи упакован в theme.

  <MuiThemeProvider theme={theme}>
     <MyStyledButton variant="contained" color="primary">
       Primary Click Me
     </MyStyledButton>
  </MuiThemeProvider>

Если я исследую кнопку в Chrome DevTools, то background-colorона «вычисляется», как и ожидалось. Это также относится и к Firefox DevTools.

Скриншот из Chrome

Однако, когда я пишу JEST-тест для проверки background-colorи запрашиваю стиль узла DOM кнопки, getComputedStyles()я transparentвозвращаюсь, и тест завершается неудачей.

const wrapper = mount(
    <MyStyledButton variant="contained" color="primary">
      Primary
    </MyStyledButton>
  );
  const foundButton = wrapper.find("button");
  expect(foundButton).toHaveLength(1);
  //I want to check the background colour of the button here
  //I've tried getComputedStyle() but it returns 'transparent' instead of #FBB900
  expect(
    window
      .getComputedStyle(foundButton.getDOMNode())
      .getPropertyValue("background-color")
  ).toEqual(myYellow);

Я включил CodeSandbox с точной проблемой, минимальным кодом для воспроизведения и провальным JEST-тестом.

Редактировать безголовый снег снег

Саймон Лонг
источник
.MuiButtonBase-root-33 background-color является прозрачным, в то время как .MuiButton -agedPrimary-13 - нет, так что проблема в том, что классы в CSS одинаково важны, поэтому их различают только в порядке загрузки -> в тестовых стилях загружаются в неправильном порядке.
Зиднар
1
@Andreas - Обновлено по запросу
Саймон Лонг,
@ Зиндар - Да, я знаю это. Есть ли способ пройти этот тест?
Саймон Лонг
Не themeнужно ли использовать в тесте? Как и в, оберните <MyStyledButton>в <MuiThemeProvider theme={theme}>? Или использовать какую-нибудь функцию-обертку, чтобы добавить тему ко всем компонентам?
Бретт ДеВуди
Нет, это не имеет никакого значения.
Саймон Лонг

Ответы:

1

Я подошел ближе, но пока не совсем в решении.

Основная проблема заключается в том, что MUIButton внедряет тег в элемент для включения стилей. Это не происходит в вашем модульном тесте. Я смог заставить это работать, используя createMount, который используют тесты материала.

После этого исправления стиль отображается правильно. Тем не менее, вычисленный стиль все еще не работает. Похоже, у других возникли проблемы с правильной обработкой ферментами, поэтому я не уверен, возможно ли это.

Чтобы добраться туда, где я был, возьмите свой тестовый фрагмент, скопируйте его в начало и затем измените свой тестовый код на:

const myMount = createMount({ strict: true });
  const wrapper = myMount(
    <MuiThemeProvider theme={theme}>
      <MyStyledButton variant="contained" color="primary">
        Primary
      </MyStyledButton>
    </MuiThemeProvider>
  );
class Mode extends React.Component {
  static propTypes = {
    /**
     * this is essentially children. However we can't use children because then
     * using `wrapper.setProps({ children })` would work differently if this component
     * would be the root.
     */
    __element: PropTypes.element.isRequired,
    __strict: PropTypes.bool.isRequired,
  };

  render() {
    // Excess props will come from e.g. enzyme setProps
    const { __element, __strict, ...other } = this.props;
    const Component = __strict ? React.StrictMode : React.Fragment;

    return <Component>{React.cloneElement(__element, other)}</Component>;
  }
}

// Generate an enhanced mount function.
function createMount(options = {}) {

  const attachTo = document.createElement('div');
  attachTo.className = 'app';
  attachTo.setAttribute('id', 'app');
  document.body.insertBefore(attachTo, document.body.firstChild);

  const mountWithContext = function mountWithContext(node, localOptions = {}) {
    const strict = true;
    const disableUnnmount = false;
    const localEnzymeOptions = {};
    const globalEnzymeOptions = {};

    if (!disableUnnmount) {
      ReactDOM.unmountComponentAtNode(attachTo);
    }

    // some tests require that no other components are in the tree
    // e.g. when doing .instance(), .state() etc.
    return mount(strict == null ? node : <Mode __element={node} __strict={Boolean(strict)} />, {
      attachTo,
      ...globalEnzymeOptions,
      ...localEnzymeOptions,
    });
  };

  mountWithContext.attachTo = attachTo;
  mountWithContext.cleanUp = () => {
    ReactDOM.unmountComponentAtNode(attachTo);
    attachTo.parentElement.removeChild(attachTo);
  };

  return mountWithContext;
}
AnilRedshift
источник