Skip to content

test support#82

Merged
nighca merged 5 commits intoqiniu:masterfrom
nighca:test
Jun 29, 2018
Merged

test support#82
nighca merged 5 commits intoqiniu:masterfrom
nighca:test

Conversation

@nighca
Copy link
Copy Markdown
Collaborator

@nighca nighca commented May 10, 2018

  • 支持运行单元测试(使用 webpack 先将测试文件作为 entry 进行转换,然后将转换结果作为测试用例执行,从而避免重复配置 jest 等测试工具),使用示例

    添加 build config 配置项 test,包括 setupFiles & moduleNameMapper 字段

    image

  • build config 添加 transformIncludes,指定 node_modules 中需要被(transform)处理的 package:原有行为是简单地对 js 内容忽略 node_modules,现在改为,对于所有内容,忽略 node_modules,可以通过 transformIncludes 指定不被忽略的内容

  • 优化 task 执行 log,打印运行时间:

    image

@nighca nighca requested a review from doxiaodong May 10, 2018 14:45
@doxiaodong
Copy link
Copy Markdown
Contributor

提供 https://github.com/airbnb/enzyme 么,这个用起来还挺方便的

@nighca
Copy link
Copy Markdown
Collaborator Author

nighca commented May 13, 2018

@doxiaodong 这个不用内置啊,是运行时的,项目自己安装 import 进来应该就可以直接用的 https://github.com/airbnb/enzyme/blob/master/docs/guides/jest.md

@doxiaodong
Copy link
Copy Markdown
Contributor

moduleDirectories 有包含 src 目录么,我们之前默认把 src 当做module 的,test 也需要吧

@doxiaodong
Copy link
Copy Markdown
Contributor

@doxiaodong
Copy link
Copy Markdown
Contributor

还有一点啊, jest 应该可以不用 webpack 参与就能工作的吧

@nighca
Copy link
Copy Markdown
Collaborator Author

nighca commented May 14, 2018

@doxiaodong

moduleDirectories 有包含 src 目录么,我们之前默认把 src 当做module 的,test 也需要吧

单测经常直接写在 src 里,所以这次没有特别处理 test 目录

transformIgnorePatterns 也挺有用的吧

没有使用 jest 的 transform,所以不用这个啦,这次加的 transformIncludes 就是个类似作用的东西,不过它不仅影响测试时的行为,也影响构建时

还有一点啊, jest 应该可以不用 webpack 参与就能工作的吧

不依赖 webpack 的话就需要手动给 jest 配很多 transform 跟 moduleNameMapper(处理 ts/tsx/svg/css/less/vue 等...),一方面是重复逻辑,另外一方面 jest 的 transform 跟 webpack 的 loader 行为不一定能完全一一对应上,也就会导致测试环境跟浏览器环境逻辑的不一致;这也是把测试集成进 builder 的主要原因

@doxiaodong
Copy link
Copy Markdown
Contributor

example 加个test 看一下吧,我空了也试一试。

@nighca
Copy link
Copy Markdown
Collaborator Author

nighca commented May 14, 2018

@doxiaodong 已经加了俩,看这个 commit

@doxiaodong
Copy link
Copy Markdown
Contributor

加进 fec-test.sh 然后更新下 submodule 吧,我最近忙爆了,先合代码试用吧。

@nighca
Copy link
Copy Markdown
Collaborator Author

nighca commented May 19, 2018

@doxiaodong 哈哈我也是,这个不着急合,已经发了个 beta 版本可用;在大阿里要开始 996 模式了嘛?

@doxiaodong
Copy link
Copy Markdown
Contributor

倒没有6

@nighca nighca merged commit 59c6746 into qiniu:master Jun 29, 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.

2 participants