Skip to content

Conversation

@chosak
Copy link
Member

@chosak chosak commented Sep 20, 2018

When running against versions of enzyme > 3.3.0, one of the frontend tests fails. The test actually had an error: it was erroneously trying to select a <button class=".o-expandable_cue"> using an ".o-expandable_cue button" selector. This worked in enzyme <= 3.3.0 but was fixed in 3.4.0, in enzymejs/enzyme#1680.

See this comment for someone else encountering the same thing:

enzymejs/enzyme#1680 (comment)

To test, disable the use of package-lock with "npm config set package-lock false", then run "npm install", then "npm test". This will fail without the test to the change and pass with this change.

Thanks to @wpears for assistance with this.

When running against versions of enzyme > 3.3.0, one of the frontend
tests fails. The test actually had an error: it was erroneously trying
to select a <button class=".o-expandable_cue"> using a
".o-expandable_cue button" selector. This worked in enzyme <= 3.3.0 but
was fixed in 3.4.0, in enzymejs/enzyme#1680.

See this comment for someone else encountering the same thing:

enzymejs/enzyme#1680 (comment)

To test, disable the use of package-lock with "npm config set
package-lock false", then run "npm install", then "npm test". This will
fail without the test to the change and pass with this change.

Thanks to @wpears for assistance with this.
@chosak chosak requested a review from wpears September 20, 2018 20:00
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 964acab on chosak:fix-broken-enzyme-test into df43150 on cfpb:master.

Copy link
Member

@wpears wpears left a comment

Choose a reason for hiding this comment

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

🎊

@chosak chosak merged commit a852083 into cfpb:master Sep 20, 2018
@chosak chosak deleted the fix-broken-enzyme-test branch September 20, 2018 20:17
Scotchester pushed a commit that referenced this pull request Jul 20, 2020
Fix test broken on recent versions of enzyme
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