cssmagic / action

Easy and lazy solution for click-event-binding.
95 stars 32 forks source link

增加测试能够测试出混合使用data和attr接口产生bug的情况 #31

Closed lyt9304 closed 8 years ago

lyt9304 commented 8 years ago

测试用例写的逻辑有点问题,谢谢鹏哥指出! 总结一下:

  1. 首先整个测试文件的结构分为Util(测试组件内部的一些函数的功能),API(测试组件暴露给外部的接口),Dom Binding(给Dom元素绑定action事件,测试触发后功能是否正常)。结构是根据功能来分类的,所加的用例其实是在功能上属于Dom Binding的,不应该重新开一个分类。
  2. 其次两个测试其实是需要先后执行,并且之前是互相有关系的,所以应该写到一个测试当中,而不是分成两个,这样就变成了两个独立的,其实和想要的效果是不一样的。
  3. 另外测试的描述应该是基于行为的,而不是基于原理的,这样更加清晰也更符合逻辑,一个test case实际上是重现一个用户使用的场景,而不是根据原理来写的,所以该用例的名字不应该写的是内部实现的原理,而是应该将这个使用场景描述出来。
cssmagic commented 8 years ago

为节省时间,这个 PR 我先合。稍后我会用一个 commit 来完善这个 PR。