Add @ice/store Unit Test - Githubissues

ice-lab / icestore

🌲 Simple and friendly state for React

MIT License

397 stars 35 forks source link

Add @ice/store Unit Test #69

Closed luhc228 closed 4 years ago

luhc228 commented 4 years ago

增加 @ice/store单元测试

0. add some examples for test

todos
users
counter

1. createStore Module

[x] Provider
[x] useModel
[x] useModelActions
[x] useModelEffectsState
[x] withModel
[x] withModelActions
[x] withModelEffectsState

2. createModel Module

[x] Provider
[x] useState
[x] useActions
[x] useEffectsState

3. createContainer

[x] createUseContainer
[x] createContainer

补充：

withModelActionsState & useModelActionsState 没有做相应的测试，看代码上注释着@deprecated，只分别对应测试 withModelEffectsState & useModelEffectsState
createContainer.tsx 里面的方法createContainer需要传参useValue。而 useValue是在模块createModel 中的方法，useValue没有从模块中导出。解决方法：在src目录下新建 __mocks__文件夹，并新建createModel.tsx，这个文件是/src/createModel.tsx的mock方法，返回的值只有 useValue，便于测试createContainer功能。参考：https://jestjs.io/docs/zh-Hans/manual-mocks

alvinhui commented 4 years ago

createContainer.tsx 里面的方法createContainer需要传参useValue.

这个 useValue 应该可以是任意的 hooks ，不一定需要 createModel 里的 useValue？

luhc228 commented 4 years ago

createContainer.tsx 里面的方法createContainer需要传参useValue.

这个 useValue 应该可以是任意的 hooks ，不一定需要 createModel 里的 useValue？

useValue是任意的hooks 我的理解是如果是任意的hooks的话，没办法拿到他的值呢？后面就没法测试到Provider？比如下面的 const value = useValue(props); 应该要拿到value吧

alvinhui commented 4 years ago

@luhc228 这个 PR 非常的出色！但我们在在 1.3.0 中重构了 icestore https://github.com/ice-lab/icestore/pull/82 ，如果可以的话，是不是可以基于这个 PR 为 1.3.0 版本补充单测？可以将 PR 的目标分支指向 master。