istanbul 增量代码覆盖率1——格式化 git diff 内容

在需求测试阶段,全量代码覆盖率其实是比较鸡肋的一个东西,它的数据太多了,并没有针对需求,提供一个明确的信息给到使用者。所以更多时候,我们需要的是增量代码覆盖率,这个数据可以让所有人更直观的看到,某个需求的新增代码中,未测试到的部分。

在获取增量代码覆盖率之前,需要先通过 git diff 将当前版本和基线版本的内容的不同整理出来。

直接上代码。

# git_diff.sh
cd ${1}
git diff ${2} ${3} > a.txt

${1} 目标代码的根路径。
${2} 基线 commit id。
${3} 目标 commit id,不填写的情况下,默认是当前已 pull 的 commit。

然后将获取到的 diff 信息存入当前路径的 a.txt 中。

# analysis_git_diff.py
import json

def get_commit_diff_line(project_root_path, base_commit, cur_commit=''):

    cmd = 'sh git_diff.sh %s %s %s' % (project_root_path, base_commit, cur_commit)

    Popen(cmd, stdout=PIPE, stdin=PIPE, shell=True).stdout.read()

    with open(project_root_path + '/a.txt', 'r') as f:
        content = f.read()
        content_list = content.split('\n')

    pattern = r'\d+'
    diff_dict = dict()
    cur_file_path = ''
    diff_line = []
    for i in range(len(content_list)):
        cur_content = content_list[i]

        if cur_content.startswith('diff --git'):
            if cur_file_path != '' and cur_file_path.endswith('.js'):
                diff_dict[cur_file_path] = diff_line
                diff_line = []
            cur_file_path = cur_content.split(' ')[2][2:]
            cur_file_path = project_root_path + '/a_' + cur_file_path
        elif cur_content.startswith('index') or cur_content.startswith('---') or cur_content.startswith('+++'):
            pass
        elif cur_content.startswith('@@') and cur_file_path.endswith('.js'):
            change_line = re.findall(pattern, cur_content)
            cur_line = int(change_line[2])
        else:
            if cur_file_path.endswith('.js'):
                if cur_content.startswith('+'):
                    diff_line.append(cur_line)
                    cur_line += 1
                elif cur_content.startswith('-'):
                    pass
                else:
                    cur_line += 1
        if i+1 == len(content_list):
            diff_dict[cur_file_path] = diff_line
            diff_line = []
    print(diff_dict)
    return diff_dict

project_root_path 目标工程的根目录。
base_commit 目标工程的基线 commit 号。
cur_commit 目标工程的当前 commit 号。

这个方法还没有经过大量的测试,或许还有很多没有覆盖到的 diff 情况,每个人都可以根据自己的实际情况作出相应的修改。
方法返回的内容是一个 {"file_abspath": [diff_line_nums_list]} 类型的数据,大概如下

{
    "/app/js_demo/src/api/api.js": [121, 123, 124, 125, 126, 127, 128, 129, 130, 133, 134, 135, 136, 137, 138, 139, 140, 141, 142],
    "/app/js_demo/src/api/defined.js": [45, 46, 47, 48, 89, 90, 196, 197, 198, 199, 200, 201, 202, 203, 204],
    "/app/js_demo/src/api/index.js": [76, 77, 78, 79, 182, 183, 184, 185, 186, 187, 188, 189, 190, 191, 192, 193, 203, 204, 205, 206, 358, 359, 360, 361, 362, 363, 624, 625, 626, 627, 628, 629, 630, 631, 632, 633, 634, 635, 636, 637, 638, 639]
}

这样,我们就可以知道对应文件中,哪些行是新增的内容了。
为什么要这么做呢?
一个是要适配 istanbul 已经有的数据类型,绝对路径可以很好的和已有的内容配合,第二个是只需要知道行数,就可以很好的融入已有的数据,会比较方便。
拥有这些内容后,我们还需要将生成的数据上传到服务器,目前选择了使用上传文件的方式,因为这个是 im 已经有的接口,无需额外的开发工作。

# analysis_git_diff.py
def save_commit_diff(diff_dict):
    with open('./commit_diff.txt', 'wb') as f:
        f.write(json.dumps(diff_dict, ensure_ascii=False).encode())

diff_dictget_commit_diff_line 方法返回的数据。

# upload_to_server.py
from subprocess import Popen, PIPE
import requests
from requests_toolbelt import MultipartEncoder

def save_file_to_server(commit_id, file_path, file_dir):
    file_name = file_path.split('/')[-1]
    local_url = 'http://localhost:8889/upload'
    m = MultipartEncoder(fields={
        'commit': commit_id,
        'fieldname': 'logo',
        'filedir': file_dir,
        'originalname': file_name,
        'mimetype': 'application/octet-stream',
        'destination': 'upload/',
        'filename': file_name,
        'path': 'upload/' + file_name,
        'file': open(file_path, 'rb')
    })
    headers = {'Content-Type': m.content_type}
    res = requests.post(local_url, data=m, headers=headers)

commit_id 基线版本的 commit 号。
file_path 文件的绝对路径。
file_dir 存储到服务器时的相对路径。

这样,获取到 diff 文件之后,通过 save_file_to_server 方法就可以将文件上传到服务器。
接下来,将服务代码修改一下,就可以同步 diff 内容了。

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 202,723评论 5 476
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 85,080评论 2 379
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 149,604评论 0 335
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,440评论 1 273
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,431评论 5 364
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,499评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 37,893评论 3 395
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,541评论 0 256
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 40,751评论 1 296
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,547评论 2 319
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,619评论 1 329
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,320评论 4 318
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 38,890评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,896评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,137评论 1 259
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 42,796评论 2 349
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,335评论 2 342