测试覆盖率 请教大家一个问题:对于老接口或者说不几乎不用的老接口了,是直接让后端开发删掉代码还是说继续保存?

一只咸鱼 · 2019年09月23日 · 最后由 一只咸鱼 回复于 2019年10月08日 · 3441 次阅读

如果继续保存的话,那么在代码覆盖率的统计上,就会将弃用的接口统计到,导致覆盖率不准确。大家遇到过类似的情况吗、

共收到 31 条回复 时间 点赞

几乎不用? 那假如删了之后发现还有在用怎么办?
覆盖率会比功能稳定更重要吗?

Jerry li 回复

比如 5 年前的接口,最初的版本 iv=1.0,根据接口统计率显示这个接口的访问量为 0 ,那么是不是这个接口就可以干掉了、

一只咸鱼 回复

除非能找到接口正式下线的批准,否则删掉是很大风险的;
另一个层面,接口即使不用了,对应的代码应该还是有参考价值的,一般也没有删掉的必要

Jerry li 回复

可以注销吗?而不是删掉代码

请了解一下代码覆盖可以 exclude。另外,不要认为调用量为 0 就可以删除,有些老系统真的很少调用,但一旦调用就会出事故。我曾经遇到一个接口,半年内调用量为 0,但这个接口确实有人在用,幸好没删除,不然死翘翘。要下接口一定要有官方的流程,这样锅才能不上身。

  • 很难说一个茅草屋是古建筑需要保护,还是碍事的玩意应该被拆掉。等后人考证出这是名人故居,然后被哪个人下令做掉了,就有一帮人站出来政治正确。要是被拆掉盖了这块地方弄了大坝,好像大家也不说什么了。然而大坝影响了下游气候,大伙又该说还不如不建,还把那么有价值的遗址给弄没了。
  • 看着办,没法预估被你端掉的接口,在 20 天后会遇到什么样的场景。现在怎么说你都有道理,20 天后遇到特殊情况出事了,不懂事的老板才不和你讲道理。但是一直没有事儿那,那就成了你的绩效

按接口和接口流量来看,梳理出来每条长链路。如果的确没有流量,也没有流量分支进来的话,可以删除。不过这种手术非常容易出故障

一般不再使用的代码,开发都会注释掉,但不会删除,预防下次需要类似的功能可以直接使用或套用。

0x88 回复

嗯 。一定要有接口正式下线的接口对吧

hellohell 回复

多谢指教,你的比喻很形象

恒温 回复

嗯,已经排除删除接口这个想法了,多谢

Pxc 回复

嗯,多谢

注销,并做好备注。已备不时之需。如果后期出现问题,也可以快速处理。

几乎不用怎么判断出来的?我们现在在做冗余代码剔除,不过要结合线上流量回放的方式来做,非常谨慎

simple 回复

我们有接口的统计量,有些版本很老的接口的访问量为 0 ,因为是移动端 APP 接口,客户端版本号很小使用的接口访问量为 0

keiskei 回复

嗯 不删除

可以分享下怎么做的代码覆盖率吗

肯定不删呀,到时删爽了,碰到一些功能产品叫开回来,那时只能你来背锅了,删代码一秒钟的事,弄回来短则一两天,长则一个星期。以前公司的同事碰到过,隔了几个月的功能叫上线。代码呢?删光了

你为毛有这么奇葩的想法,不要说删,没有人提的话,最好不要动好不好,自己找锅了啊

一般情况下不会删除,但是会在方法上说明已经不用。具体情况具体分析吧,需要根据自身的业务发展情况分析好删与不删的利弊即可。

代码管理由开发去做,具体是注释掉还是删掉开发应该比测试更有判断力,别去抢这个锅

前端先下掉调用,业务再监控一段时间,然后后端再下掉,也只是注释掉代码,把风险降到最低

匿名 #23 · 2019年09月30日

用版本的增量覆盖率来保证当前版本的改动,全量覆盖率去除这些废弃的接口,看回归测试的覆盖率。删除这些代码的主要原因并不是为了覆盖率,而是保证代码的整洁。

Elsie 回复

可以了解下 jacoco

玮琪 回复

嗯,绝对没删的念头了额

Lee 回复

不动了不动了。。。确实危险,我可不背锅

小学徒V 回复

嗯多谢

退之 回复

好的 多谢

xinxjxjxj 回复

嗯 好办法

多谢指教

需要 登录 后方可回复, 如果你还没有账号请点击这里 注册