The Git Pre-Commit Hook to Avoid Pushing Only Unit Tests In Node

  • 时间:2020-09-20 13:49:13
  • 分类:网络文摘
  • 阅读:104 次

In NodeJS unit testing frameworks such as mocha/chai, you could add .only after describe or it, so that the specific set of tests are run, which excludes/disables all other tests. For example,

1
2
3
4
5
describe.only("I just want to run these tests", function() {
   it("should work", function() {
      expect(1).to.equal(1);
   });
});
describe.only("I just want to run these tests", function() {
   it("should work", function() {
      expect(1).to.equal(1);
   });
});

Chances are, you might forget to remove the .only before you commit and push. And even worse, the code review may miss this – which in fact will disable all other tests in the Continous Integration process.

So, you can easily add a pre-commit hook check that will give you a nice warning if you try to commit some code that has .only.

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
# Redirect output to stderr.
exec 1>&2
 
# prevent it.only or describe.only commited
if [ "$allowonlytests" != "true" ] &&
    test $(git diff --cached | grep -E "\b(it|describe).only\("  | wc -l) != 0
then
    cat <<\EOF
Error: Attempt to add it.only or describe.only - which may disable all other tests
 
If you know what you are doing you can disable this check using:
 
    git config hooks.allowonlytests true
EOF
    exit 1
fi
 
exit 0
# Redirect output to stderr.
exec 1>&2
 
# prevent it.only or describe.only commited
if [ "$allowonlytests" != "true" ] &&
    test $(git diff --cached | grep -E "\b(it|describe).only\("  | wc -l) != 0
then
    cat <<\EOF
Error: Attempt to add it.only or describe.only - which may disable all other tests
 
If you know what you are doing you can disable this check using:
 
    git config hooks.allowonlytests true
EOF
    exit 1
fi
 
exit 0

Save above as filename “pre-commit” (without the quotes) in the folder .git/hooks at your project repository. The script will exit non-zero ONE which indicates a No if there is it.only or describe.only in the diff changes. However, the command/script is smart enough (to use grep with regular expression) that will not trigger the warning if it is followed by // line comment.

You can also disable the warning by using the following:

1
git config hooks.allowonlytests true
git config hooks.allowonlytests true
git-pre-commit-hook The Git Pre-Commit Hook to Avoid Pushing Only Unit Tests In NodeJs git

git-pre-commit-hook

Here’s an eslint plugin that has the same effect.
https://www.npmjs.com/package/eslint-plugin-no-only-tests

Here is another post on the pre-push hooks to the master branch: How to Prevent Commiting to master/develop branch by Accidents using Pre-push Hooks in Git?

–EOF (The Ultimate Computing & Technology Blog) —

推荐阅读:
Google SEO优化,4个“另类”的排名方法  商城系统建设心得,轻松搞定选择困难  什么是PPC?为什么你会用到它?  SEO网站优化能为你带来那些好处?  高级SEO思维详细解读  PageAdmin CMS站群系统教程:网站站群的添加和管理  友情链接交换,对于谷歌SEO是好是坏  商城系统建设心得,轻松搞定选择困难  建站行业乱象:建站合同鲜有提及源码版权  集思广益造就成功 
评论列表
添加评论