Skip to content

Feature/issue#19 on mouse up on mouse down tests#24

Open
reginafernandezpinar wants to merge 9 commits intodevelopfrom
feature/issue#19-onMouseUp-onMouseDown-tests
Open

Feature/issue#19 on mouse up on mouse down tests#24
reginafernandezpinar wants to merge 9 commits intodevelopfrom
feature/issue#19-onMouseUp-onMouseDown-tests

Conversation

@reginafernandezpinar
Copy link
Copy Markdown
Collaborator

@reginafernandezpinar reginafernandezpinar commented Apr 27, 2020

  • onMouseDown and onMouseUp events listeners and tests
  • oncontexMenu event listener and tests

Copy link
Copy Markdown
Owner

@dailymp dailymp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor details to fix, good job :)

import * as React from 'react';
import { shallow } from 'enzyme';
import { configure } from 'enzyme';
import Adapter from 'enzyme-adapter-react-16';
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please try to do this generic to only acces as config file and not need to call and configure this in every test file

import { MessageList } from './MessageList';

describe('Message List component test', () => {
configure({ adapter: new Adapter() });
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The same as before

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants