Skip to content

Improved Matcher and Matcher Test#47

Merged
selvagsz merged 6 commits intoselvagsz:masterfrom
SCasarotto:master
Apr 11, 2018
Merged

Improved Matcher and Matcher Test#47
selvagsz merged 6 commits intoselvagsz:masterfrom
SCasarotto:master

Conversation

@SCasarotto
Copy link
Copy Markdown
Contributor

Improved the matcher and added tests.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.8%) to 81.368% when pulling 74f151e on SCasarotto:master into 8da70da on selvagsz:master.

import PageObjectBase from '../../__tests__/utils/create-page-object';
import { frameworks, countries, KEY_CODES } from '../../__tests__/utils/constants';
import PageObjectBase from '../../__tests__/test-utils//create-page-object';
import { frameworks, countries, KEY_CODES } from '../../__tests__/test-utils//constants';
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.

@SCasarotto double slash ^^

Repository owner deleted a comment from coveralls Apr 11, 2018
Copy link
Copy Markdown
Contributor Author

@SCasarotto SCasarotto left a comment

Choose a reason for hiding this comment

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

removed "//".

@selvagsz selvagsz merged commit 80a957a into selvagsz:master Apr 11, 2018
@selvagsz
Copy link
Copy Markdown
Owner

Thanks @SCasarotto ❤️

@SCasarotto
Copy link
Copy Markdown
Contributor Author

SCasarotto commented Apr 11, 2018

Appreciate your patience and guidance.

@selvagsz selvagsz mentioned this pull request Apr 11, 2018
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.

3 participants