如何写一个好的测试?

发表于:2020-3-03 10:40

字体: | 上一篇 | 下一篇 | 我要投稿

 作者:boydfd    来源:博客园

  背景
  在上一个项目上,由于项目成员大部分是新入职的同事,所以对于测试不是很熟悉,
  这就导致了在项目前期,项目上的很多测试都不太make sense,虽然没有什么定量的东西来描述,
  但是总结起来就2个点:
  1.测试的名字比较模糊。
  2.测试代码不易读。
  深入剖析
  测试名字比较模糊
  对于这一个问题,是因为很多刚开始写测试的开发脑子里不会快就想到given、when、then这三个词,
  一般我们写测试写得比较多的同事,都会使用should_return_xxxx_when(if)_xxxx。其实就是在脑子中
  想到了given、when、then。而新同事照着模仿的时候,很可能就单纯地写了should_xxx。他们只考虑了结果,
  但是没有考虑条件,这就导致了读名字还是get不到测试想要干什么。
  对于这个问题,我想起了多年前在stackoverflow上面看到的一种写测试名字的模板,他是这样推荐的:
  GivenA_WhenB_C,我觉得这中写法挺好的,所以在项目中用了起来,结合实际使用,我又对此提出了两个改进:
  given when这两个词可以省略,当然前提是我们约定了最前面就是given,中间就是when,最后就是then。
  通过施加规则限制来缩短测试名。
  测试名字常常很长,一大堆驼峰其实比较不容易阅读。所以可以使用蛇形命名法,但是这样就需要想一个符号来
  分隔given、when、then了,我选择使用___(也就是3个_),因为经过试验,发现2个_太短了不容易一眼看出分隔
  4个则又没有必要。
  最终的效果是这样的(这是一个例子,来自最近我参加的DDD进阶培训中的训练题):
   parking_order_is_natural_order___park_cars_by_parking_assistant___car_parked_to_correct_parking_lot_in_turn
  car_is_already_took___take_back_car_with_used_receipt___exception_is_thrown
  parking_lot_is_available___park_a_car_by_parking_assistant___receipt_returned
  car_is_parked___take_back_car_with_invalid_receipt___exception_is_thrown
  这里还有一个小技巧,如果一个测试真的没有given的时候,或given不重要的时候,可以省略,但是___不能省略:
 ___xxx_xxx___xxx_xxx
  可以总结一下这种命名方式的优点:
  1.能制定一个规则的话,项目上的测试标题能更容易统一(可以说是统一语言了)。还可以加上静态检查,使得一些名字不规范的测试提前被发现
  名字不规范,说明是新进项目的同事写的,确实要重点检查一下。
  2.强制规定出given、when、then,那么,我们在写测试的时候,就会被强迫想清楚我们的测试要做什么。
  3.结构化的东西更适合大脑阅读,读测试的时候更容易,我们不需要先读一遍测试名才能提取到given、when、then,可以一眼就定位出三个部分。
  当然,也是有缺点的:
  我用这种方式写出来的测试名字一般都比较长,这个有可能是我用词还不够精炼,在given、when的部分可能有时候是有一部分重复的
  所以也需要刻意练习,学会精简用词。
  最后,给这种命名方式命个名吧,就叫GWT测试命名法好了。
  测试代码不易读
  我在项目上发现,很多人习惯在构建fake数据的时候直接将所有信息屏蔽,就提供一个createX()的方法,
  在createX的方法里面可能还要构建X的构成部分:
   X createX() {
  Y yyy = new Y;
  X xxx = new X(YYY, field1, field12);
  return xxx;
  }
  除了createX外还有createAX、createBX,然后在不同的测试里面混合调用。
  作为看测试的我,我就很难看到到底需要一个怎样的场景,field1到底是怎么设置的,field2到底是怎么设置
  的,而且在想改一下createX的时候,还会牵扯到其他的测试莫名其妙就挂了。
  我目前使用的解决方式是这样的:
  1.先要确定好测试要涉及到的重要信息,比如上面如果field1,field2都会对测试的逻辑起到作用,
  那么,即使冗余,也一定要写在测试方法内。比如:
   @Test
  void ___xxx___xxx() {
  Y y = createY();
  X x = createX(field1, field12, y)
  // do some thing
  // assert some thing
  }
  这么做的好处是,当我要看某一个测试的时候,它的前置数据我一眼就能看出来,不需要不停地command+b跟进去才知道需要什么。
  2.在应用第一条原则的时候,一定要记得,只罗列出这个测试所关心的数据。假如field3完全不参与这次
  逻辑的处理,又必须要有值,那么,在createX内部给个默认值即可,不需要放在createX参数列表中。
  为什么不适用builder?
  其实之前也试过用builder,但是看起来太多了,适用create的方式能缩短要写得代码行数,更容易一眼看完测试。
 
      本文内容不用于商业目的,如涉及知识产权问题,请权利人联系博为峰小编(021-64471599-8017),我们将立即处理
《2023软件测试行业现状调查报告》独家发布~

关注51Testing

联系我们

快捷面板 站点地图 联系我们 广告服务 关于我们 站长统计 发展历程

法律顾问:上海兰迪律师事务所 项棋律师
版权所有 上海博为峰软件技术股份有限公司 Copyright©51testing.com 2003-2024
投诉及意见反馈:webmaster@51testing.com; 业务联系:service@51testing.com 021-64471599-8017

沪ICP备05003035号

沪公网安备 31010102002173号