今天真的被恶心到了，给 API 新加了一个 field，所以改了单元测试。结果发现改测试的时间比开发用的时间都长。仔细想了想原因，大致有以下几点 1) 代码没有复用。感觉以前完全是新加一个功能，便去复制前面人写的测试，然后稍加修改，导致代码冗余度相当高。某一段代码，至少改了四五次
摘取一点代码结构
▼ NoContactTest : class +_create_contact : function +_create_virtual_product : function +setUp : function +tearDown : function +test_create_order : function +test_create_order_from_cart : function +test_create_order_from_cart_contact : function ▼ OrderAPITest : class +setUp : function +tearDown : function +test_cancel_order : function +test_complete_order : function +test_create_comment : function +test_create_order : function +test_create_order_from_cart : function +test_list_order_comments : function +test_list_orders : function +test_list_reasons : function +test_retrieve_order : function |
test_create_order 方法的代码相似度起码有 60%
你能体会改代码时一直重复，然后迷失，分不清自己具体在哪个测试用例的那种感觉么？将 OrderAPITest 作为基类应该是一个不错的选择
2) 硬编码实在太过严重
stock = models.Stock.objects.create( product=product, choice1=self.choice1, shipping_type=0, origin_price=100.00, price=100.00, amount=1000) stock_1 = models.Stock.objects.create( product=product, choice1=self.choice1, shipping_type=1, origin_price=50.00, price=40.00, amount=1000) models.StockObject.objects.create( stock=stock, business_code=self.business_code, object_id=1) items = [] item = models.CartItem.objects.create( user_id=self.normal_user_id, product=product, stock=stock, quantity=2) items.append(item) item = models.CartItem.objects.create( user_id=self.normal_user_id, product=product, stock=stock_1, quantity=1) items.append(item) payload = { "item_ids": [self.pk2string(i.id) for i in items], "user_coins": 0 } resp = self.client.post(url, payload) self.assertEqual(resp['status_code'], 0) self.assertEqual(resp['data']['amount'], '245.00') self.assertEqual(resp['data']['shipping_fee'], '8') |
你能知道 amount 为什么是 245 么？反正我是去又翻了一遍代码逻辑。可这是一个原则性的错误，测试代码不应当去迎合你的业务代码逻辑，而应该去检验预期逻辑。否则要什么测试？我都不知道之前的人是不是将代码运行结果直接粘到测试中了。测试中比较理想的是填充一定程度上随机的数据，不过这是比较麻烦的，因为有些复杂的逻辑难以实现，无法准确模拟。覆盖边界条件一般就差不多了，毕竟想要完全覆盖边界也是很麻烦的
3) 一个测试应当仅做一件事情
这测试代码处处违反了这个规则，比如之前测试过创建订单了。但是有些业务依赖于订单的创建这个条件，比如取消订单。那么需要在取消订单这里需要再测试一遍创建订单么。之前已经测试过了，除非你对自己先前的测试不满意，你应当将其视为数据填充才是正确的
所以说测试代码和生产代码一样重要，生产代码需要可扩展、可维护、可复用，测试代码也同样需要。希望不要抱着反正下次不是我写的这种消极态度去敷衍测试环节。
