e1-bsd/omni-common-ui

Similar blocks of code found in 2 locations. Consider refactoring.

Open

describe('#isSuccess()', () => {
  test('returns true if action.type ends with _SUCCESS', () => {
    const callAction = ApiAction.create(buildAction({ type: 'CALL_SUCCESS' }));
    expect(ApiAction.isStarted(callAction)).toBe(false);
    expect(ApiAction.isSuccess(callAction)).toBe(true);

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

    test('renders the error page if a failed ApiCall.State is received as erroredApi and its code is not 500', () => {
      props = buildProps({ status: 400 });
      const wrapper = mount(<ErrorPageHandler {...props}><div id="inner" /></ErrorPageHandler>);
      expect(wrapper.find(ErrorPage)).toHaveLength(1);
      expect(wrapper.find('#inner')).toHaveLength(0);

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

    test('renders the error page if a failed ApiCall.State is received as erroredApi and its code is 500', () => {
      props = buildProps({ status: 500 });
      const wrapper = mount(<ErrorPageHandler {...props}><div id="inner" /></ErrorPageHandler>);
      expect(wrapper.find(ErrorPage)).toHaveLength(1);
      expect(wrapper.find('#inner')).toHaveLength(0);

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

    test('renders the error page if a failed ApiCall.State is received as erroredApi and its code is 500', () => {
      props = buildProps({ status: 500 });
      const wrapper = mount(<ErrorPageHandler {...props}><div id="inner" /></ErrorPageHandler>);
      expect(wrapper.find(ErrorPage)).toHaveLength(1);
      expect(wrapper.find('#inner')).toHaveLength(0);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  test('gets rid of "." items', () => {
    expect(normalizeUrl('/test/./path')).toBe('/test/path');
    expect(normalizeUrl('/test/path/.')).toBe('/test/path');
    expect(normalizeUrl('/./test/path')).toBe('/test/path');
    expect(normalizeUrl('./test/path')).toBe('test/path');

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  test('removes duplicated slashes', () => {
    expect(normalizeUrl('/test//path')).toBe('/test/path');
    expect(normalizeUrl('/test///path')).toBe('/test/path');
    expect(normalizeUrl('/test_///path')).toBe('/test_/path');
    expect(normalizeUrl('/test9///path')).toBe('/test9/path');

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  test('the progress is 100 if fraction is greater than 1', () => {
    const wrapper = shallow(<ProgressBar value={200} max={100} />);
    const progress = wrapper.find(`.${styles.ProgressBar_progress}`);
    expect(progress.props().style.width).toBe('100%');
  });

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  test('calculates the progress percentage out of "progress" and "total"', () => {
    const wrapper = shallow(<ProgressBar value={45} max={45} />);
    const progress = wrapper.find(`.${styles.ProgressBar_progress}`);
    expect(progress.props().style.width).toBe('100%');
  });

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  test('the progress is 0 if no "progress" is provided', () => {
    const wrapper = shallow(<ProgressBar max={45} />);
    const progress = wrapper.find(`.${styles.ProgressBar_progress}`);
    expect(progress.props().style.width).toBe('0%');
  });

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  test('the progress is 100 if "progress" is more than 100', () => {
    const wrapper = shallow(<ProgressBar value={145} />);
    const progress = wrapper.find(`.${styles.ProgressBar_progress}`);
    expect(progress.props().style.width).toBe('100%');
  });

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  test('uses the "progress" property directly as a percentage', () => {
    const wrapper = shallow(<ProgressBar value={45} />);
    const progress = wrapper.find(`.${styles.ProgressBar_progress}`);
    expect(progress.props().style.width).toBe('45%');
  });

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

      test('returns a default ApiResponse if data set to undefined', () => {
        const result = ApiResponseHelper.create({ data: undefined });
        expect(result.loading).toBe(false);
        expect(result.error).toBeUndefined();
        expect(result.data).toBeUndefined();

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

      test('returns a default ApiResponse if loading set to undefined', () => {
        const result = ApiResponseHelper.create({ loading: undefined });
        expect(result.loading).toBe(false);
        expect(result.error).toBeUndefined();
        expect(result.data).toBeUndefined();

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

      test('returns a default ApiResponse if error set to undefined', () => {
        const result = ApiResponseHelper.create({ error: undefined });
        expect(result.loading).toBe(false);
        expect(result.error).toBeUndefined();
        expect(result.data).toBeUndefined();

• Read up
  Read up

Function testButtonClick has 67 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function testButtonClick(props = {}) {
  describe('when clicked', () => {
    test('calls onClick', () => {
      const onClick = jest.fn();
      const wrapper = shallow(<Button {...props} onClick={onClick} />);

File index.jsx has 278 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import styles from './style.postcss';

import React, { PureComponent } from 'react';
import PropTypes from 'prop-types';
import { Set } from 'immutable';

Similar blocks of code found in 2 locations. Consider refactoring.

Open

test('renders profile info with the given surname', () => {
  const wrapper = mount(<Profile name="John" surname="Doe" localName="王呆呆" />);
  expect(wrapper.find(`.${styles.StudentCard_profile_localName}`).text()).toBe('王呆呆');
});

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

test('renders profile info with the given name', () => {
  const wrapper = mount(<Profile name="John" surname="Doe" localName="王呆呆" />);
  expect(wrapper.find(`.${styles.StudentCard_profile_name}`).text()).toBe('John Doe');
});

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  test('applies Field.__stackedHorizontally when enabled', () => {
    const wrapper = mount(<Field {...requiredPropNoops} neighborStackMode="horizontal" />);
    expect(wrapper.find(`.${styles.Field}.${styles.__stackedHorizontally}`)).toHaveLength(1);
  });

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  test('applies Field_wrap.__stackedVertically when enabled', () => {
    const wrapper = mount(<Field {...requiredPropNoops} innerStackMode="vertical" />);
    expect(wrapper.find(`.${styles.Field_wrap}.${styles.__stackedVertically}`)).toHaveLength(1);
  });

• Read up
  Read up