electron覆盖率在jenkins打包,注入的环境变量的值不正确

最近一段时间一直在预研electron的覆盖率的事宜。大致的流程已经基本ok,剩余最后的数据验证。由于electron的插桩分为了渲染进程的插桩以及主进程的插桩。然后在验证的时候发现。主进程的覆盖率数据基本都没有上报上来。

从日志入手

由于主进程的定时任务数据上报是有日志打印的,所以我们先看下日志的信息是如何的。

image

找到了如下的日志:

image

提示说Commit提交号没有匹配到对应的测试计划。

这里先需要说明下我们electron覆盖率的逻辑,我们做插桩以后会将对应当前的代码的commit号以及覆盖率数据进行上报到覆盖率后台。由覆盖率后台来判断上报的commit所对应的测试计划是哪个。

但是我们通过fiddler抓包发现实际上渲染进程上报就不存在这样子的问题

image

这里就很奇怪了。因为不管是主进程还是渲染进程获取git的commit都是用的同样的逻辑进行获取的,为啥唯独主进程的数据上报存在问题呢? 那首先我们需要确认一个问题主进程中获取到的commit到底是多少

通过查看electron打包后的asar包的内容,我们发现了真正的commit号:

image

确实与渲染进程上报的commit是不一致的。 所以我们需要确认主进程与渲染进程在commit号或者说数据提交上有什么区别了。

大胆推测

以下就是主进程打包的变量赋值的过程

const istanbul = require("./webpack/istanbul");
option.plugins.unshift(new webpack.EnvironmentPlugin({
    JENKINS_PATH: process.cwd(),
    GIT_COMMIT: istanbul.getGitCommit(),
}))

istanbul.js

/**
 * Helper file
 * 增加了一些方法,供覆盖率测试使用
 */
var fs = require('fs');

var helper = {
    getGitCommit: function () {
        try {
            var gitHEAD = fs.readFileSync('.git/HEAD', 'utf-8').trim(); // 在jenkins上面,这里拿到的就已经是commit号了
            //var ref = gitHEAD.split(': ')[1];
            //var gitVersion = fs.readFileSync('.git/' + ref, 'utf-8').trim();
            return gitHEAD;
        } catch (e) {
            console.log('get git commit fail!');
        }
    },
};

module.exports = helper;

看上去问题不大呀。那我们就打印下 gitHEAD 试试看吧。

image

从这个打印信息看 主进程以及渲染进程的commit号都是一样的才对。 那到底是谁又从新将这个commit号做了修改呢?

所以我们需要关注到一个变量名称 GIT_COMMIT 有没有可能是这个变量名称跟哪个冲突了呢? 所以我们在jenkins的页面全局搜索了下。结果还真的有所发现

image

本身就有一个变量名称就叫做这个。并且就是主进程上报的commit号。 到这里真相就大白了。我们用的变量名称与本身的全局的变量名称冲突了。所以我们只要做一下修改就好了。

image
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念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

推荐阅读更多精彩内容