enzymejs / enzyme

JavaScript Testing utilities for React
https://enzymejs.github.io/enzyme/
MIT License
19.95k stars 2.01k forks source link

support lazy components #2521

Open artola opened 3 years ago

artola commented 3 years ago

Fix #2205

codecov[bot] commented 3 years ago

Codecov Report

Merging #2521 (795045d) into master (41bf606) will decrease coverage by 0.53%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2521      +/-   ##
==========================================
- Coverage   96.31%   95.77%   -0.54%     
==========================================
  Files          49       49              
  Lines        4207     4215       +8     
  Branches     1130     1134       +4     
==========================================
- Hits         4052     4037      -15     
- Misses        155      178      +23     
Impacted Files Coverage Δ
...enzyme-adapter-react-16/src/ReactSixteenAdapter.js 92.25% <100.00%> (-3.21%) :arrow_down:
...ges/enzyme-adapter-react-16/src/detectFiberTags.js 94.02% <100.00%> (+0.58%) :arrow_up:
...pter-react-16/src/findCurrentFiberUsingSlowPath.js 56.14% <0.00%> (-12.29%) :arrow_down:
packages/enzyme/src/ReactWrapper.js 99.27% <0.00%> (-0.25%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 41bf606...795045d. Read the comment docs.

artola commented 3 years ago

@ljharb I will appreciate your help about these test failing. I changed only enzyme-adapter-react-16 to target React v16.14.

ljharb commented 3 years ago

@artola when i rebase this and run it locally, the tests hang - do you see the same?

artola commented 3 years ago

@artola when i rebase this and run it locally, the tests hang - do you see the same?

@ljharb I also rebase, clean and install ... and tests are running with the following results:

  1061 passing (6s)
  87 pending
  57 failing