接口测试原则之测试代码中的判断

作者：网络转载发布时间：[ 2012/7/9 11:18:07 ] 推荐标签：

　　前几天把我之前整理的接口测试原则发了上来，不过没有人评论，想想估计写的太简单了，没有几个人明白我说什么，不过前几天有个同学关注到我的这个blog，还问了我问题，我真是很激动，边把他提的问题与关于这条原则的一些理解，发出来与大家分享。

　　接口测试原则

　　1、用例代码简单，只覆盖单个场景，不能写判断，如if怎样，则这判断什么，否则判断什么。尽量不catch异常，使用时要配置fail断言一起使用。

　　2、接口测试代码做到只依赖被测试接口，不要准备数据或者验证数据时调用你还不能完全信任的系统接口(这点可能会有很多争议)。

　　3、接口要操作数据不能依赖数据库原有的数据，接口产生的数据必须清理。

　　4、对于接口返回要通过断言进行验证，必须做到用例设计要求。

　　5、Mock的原则是为了把测试的重点与精力放到我们代码逻辑上来，mock的目标是成功模拟场景。

　　他的问题和我提的第一个原则有关，是关于测试代码中的含有if的问题，他觉得一些数据校验的方法，实在是太乱了，不好维护。

　　我研究了一下他发给的代码，有两种情况，第一种

public void doAssertItemConfirmStatus(long itemId， int sign) {
String resultSql = "select * from wlb_order_item where id =" + itemId;
Map sqlMap = wlbJdbcTemplate.queryForMap(resultSql);
if (sign == 1) {
assertThat(sqlMap.get("confirm_status").toString()， equalTo("1"));
} else if (sign == 2) {
assertThat(sqlMap.get("confirm_status").toString()， equalTo("2"));
}
。。。
}

　　对于这类来讲，由于判断的值是我们调用传递的，是明确知道值的，所以问题不大，一般这种我一般会这样写

assertThat(getConfirmStatus()， equalTo("1"));

　　下面代码是封装了返回状态的方法

String getConfirmStatus() {
String resultSql = "select * from wlb_order_item where id =" + itemId;
Map sqlMap = wlbJdbcTemplate.queryForMap(resultSql);
return sqlMap.get("confirm_status").toString()
}

　　还有一类：

public void doAssertWeightAndVolume(FillBackOrderDTO orderDetail， int sign) {
String resultSql = "select * from wlb_order where order_code=‘"
+ orderDetail.getOrderCode() + "‘";
Map sqlMap = wlbJdbcTemplate.queryForMap(resultSql);
if (sign == 1) {
String attributes = sqlMap.get("attributes").toString();
int flag = 0;
Map optionalPropertiesMap = KeyValueParser.parseString2Map(attributes);
if (orderDetail.getPackageWeight() != null) {
assertThat(optionalPropertiesMap.get(OrderConstant.ATT_PACKAGE_WEIGHT).toString()，
equalTo(orderDetail.getPackageWeight().toString()));
}
if (orderDetail.getPackageHeight() != null) {
flag++;
assertThat(optionalPropertiesMap.get(OrderConstant.ATT_PACKAGE_HEIGHT).toString()，
equalTo(orderDetail.getPackageHeight().toString()));
}
。。。
if (flag == 3) {
Double volume = orderDetail.getPackageHeight() * orderDetail.getPackageWidth()
* orderDetail.getPackageLength();
assertThat(optionalPropertiesMap.get(OrderConstant.ATT_PACKAGE_VOLUME).toString()，
equalTo(volume.toString()));
}
} else {
if (sqlMap.get("attributes") == null) {
assertThat(sqlMap.get("attributes")， equalTo(null));
} else {
assertThat(sqlMap.get("attributes").toString()， equalTo(""));
}
}
}
　　这段中含有 if (orderDetail.getPackageWeight() != null)