Skip to content
This repository has been archived by the owner on Mar 5, 2022. It is now read-only.

fix: use display name of the component #92

Merged
merged 3 commits into from
Jun 5, 2019

Conversation

atomicpages
Copy link
Contributor

@atomicpages atomicpages commented May 22, 2019

atomicpages added 2 commits May 22, 2019 15:34
* Adding tests for React.memo
* Adding tests for React.forwardRef
* Adding basic hook tests
* Fixing bad .type property in get where the selector is a function
* Ensuring all specs pass
* Adding stub for hooks spec for the time being...
@atomicpages
Copy link
Contributor Author

Looks like travis has some missing dependencies for cypress. Uploading the test report
Screen Shot 2019-05-22 at 8 34 40 PM


/* eslint-env mocha */
describe('CounterWithHooks component', function () {
it.skip('works', function () {
Copy link
Contributor

Choose a reason for hiding this comment

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

can I ask why this test is skipped - is it not working yet?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Related to #51 I wasn't able to get it working. Leaving it there as a stub for now

Copy link
Contributor

Choose a reason for hiding this comment

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

gotcha

@bahmutov bahmutov changed the title Fixing #91 fix: use display name of the component Jun 5, 2019
@bahmutov
Copy link
Contributor

bahmutov commented Jun 5, 2019

Super, merging this now, thank you @atomicpages - if you plan any more contributions, let me know, I could just add you as a collaborator to this repo

@bahmutov bahmutov merged commit 721f55c into cypress-io:master Jun 5, 2019
@atomicpages
Copy link
Contributor Author

atomicpages commented Jun 6, 2019

Sure! I can tackle a few more issues 👍

@OneHatRepo OneHatRepo mentioned this pull request Jun 12, 2019
dmtrKovalenko pushed a commit to cypress-io/cypress that referenced this pull request Oct 1, 2020
…-test#92)

* Fixing cypress-io/cypress-react-unit-test#91

* Adding tests for React.memo
* Adding tests for React.forwardRef
* Adding basic hook tests

* Fixing some issues

* Fixing bad .type property in get where the selector is a function
* Ensuring all specs pass
* Adding stub for hooks spec for the time being...
dmtrKovalenko pushed a commit to cypress-io/cypress that referenced this pull request Oct 1, 2020
cypress-io/cypress-react-unit-test#92 broke the alias functionality. If a jsx.type exists, the alias will be ignored. This restores the alias functionality such that a supplied alias will override the jsx.type.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Components with forwardRef fail when trying to get displayName
2 participants