test(Anchor): optimize dataNowMockFn, avoid being consumed in advance

[shaodahong]

🤔 This is a ...

• New feature
• Bug fix
• Site / document update
• Component style update
• TypeScript definition update
• Refactoring
• Code style optimization
• Test Case
• Branch merge
• Other (about what?)

🔗 Related issue link

#17827

💡 Background and solution

📝 Changelog

Language	Changelog
🇺🇸 English	optimize dataNowMockFn, Avoid being consumed in advance
🇨🇳 Chinese	优化 dataNowMockFn, 避免被提前消费

☑️ Self Check before Merge

• Doc is updated/provided or not needed
• Demo is updated/provided or not needed
• TypeScript definition is updated/provided or not needed
• Changelog is provided or not needed

[tests-checker]

Could you please add tests to make sure this change works as expected?

[netlify]

Deploy preview for ant-design ready!

Built with commit da5c01b

https://deploy-preview-18663--ant-design.netlify.com

[shaodahong]

@afc163 @zombieJ 我换了下 circleci 的 docker image，然后跑的测试，看起来像是没问题的

    ✓ Anchor targetOffset prop (11ms)

circleci/node:latest 这个版本是不是 latest lts 比较好

[afc163]

circleci/node:latest 这个版本是不是 latest lts 比较好

对的，改之。

[shaodahong]

看了下 node 版本是 10.16.3 但是现在报的其它一些错

[shaodahong]

这个测试错误看起来像是 facebook/react#12214 (comment) 命名导出的问题，没发现 node 新的版本对导出有什么限制更新

[zombieJ]

合一下 master， 下午 fix 了 React.createElement: type is invalid 相关的错误，估计是同一个。

[codecov]

Codecov Report

Merging #18663 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #18663   +/-   ##
=======================================
  Coverage   96.74%   96.74%           
=======================================
  Files         281      281           
  Lines        7551     7551           
  Branches     2106     2106           
=======================================
  Hits         7305     7305           
  Misses        244      244           
  Partials        2        2

---

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 bf374da...da5c01b. Read the comment docs.

[shaodahong]

合一下 master， 下午 fix 了 React.createElement: type is invalid 相关的错误，估计是同一个。

还是不行

[shaodahong]

我本地试了下是有问题

 FAIL  components/list/__tests__/demo.test.js (11.789s)
  ● renders ./components/list/demo/basic.md correctly

    ReferenceError: window is not defined

      at node_modules/ismobilejs/dist/isMobile.min.js:1:1292
      at Object.<anonymous> (node_modules/ismobilejs/dist/isMobile.min.js:1:1314)
      at Object.<anonymous> (node_modules/rc-menu/lib/util.js:17:16)

[daqi]

@shaodahong 正好赶上我们发布上线，服务端渲染也报这个错误

[shaodahong]

嗯，isMobile 库用了 window，其实我奇怪的是为什么之前 node:8 的测试是没有问题的，emm……

[shaodahong]

@afc163 我建议学下 jest 的 CI 把主流的 node 版本都加进去

[shaodahong]

• 支持主流 Node 版本 CI
• 修复 React.createElement: type is invalid 相关报错
• 修复部分组件 Demo Test 中 undefined 错误（暂时没找到原因）

[zombieJ]

重装一下，rc-menu 的 is mobile 降级发了个 patch

[afc163]

@afc163 我建议学下 jest 的 CI 把主流的 node 版本都加进去

没啥必要，jest 是 node 端的工具，兼容不同 node 版本很重要。我们是 UI 库，node 环境保证自用就行，测试的是浏览器环境。

[shaodahong]

好的，那看来是没问题了