152

I would like to run a unit-tests before every git push and if tests fails, cancel the push, but I can't even find pre-push hook, there is pre-commit and pre-rebase only.

1

7 Answers 7

238

Git got the pre-push hook in the 1.8.2 release.

Sample pre-push script: https://github.com/git/git/blob/87c86dd14abe8db7d00b0df5661ef8cf147a72a3/templates/hooks--pre-push.sample

1.8.2 release notes talking about the new pre-push hook: https://github.com/git/git/blob/master/Documentation/RelNotes/1.8.2.txt

3
  • 1
    @manojlds do you know what is this hook designed to be used for? I would like to use it to push my binary to my customers when pushing to a specific branch(i.e. build the nightly version and upload it with curl, before pushing). The problem is that it takes a while to build and upload, and remote closes connection. So i end up with my binary built and uploaded to customers but not pushed to a repo, because remote repo closes connection. Any idea how to work around this? Or maybe it is a bad idea in it's root.
    – igrek
    Jun 19, 2014 at 13:51
  • @igrek did you find a solution to the connection closing issue? Apr 27, 2015 at 3:54
  • 1
    @MarioEstrada, yes, i don't remember exactly how, but i made it push twice: first git command runs unit tests and then if it doesn't disconnect it pushes and starts another push in another thread, if the first one push times out, the second one from another thread works for me. If either first and second succeeds, then the first pushes changes, and the second pushes nothing. The trick is that i had some argument added, which bypasses unit tests(which was used for the second git push, so it didn't start unit tests again)
    – igrek
    Apr 28, 2015 at 7:12
40

Git got the pre-push hook in the 1.8.2 release.

Pre-push hooks are what I needed along with pre-commit hooks. Apart from protecting a branch, they can also provide extra security combined with pre-commit hooks.

And for an example on how to use (taken and adopted and enhanced from this nice entry)

Simple example to login to vagrant, run tests and then push

#!/bin/bash
# Run the following command in the root of your project to install this pre-push hook:
# cp git-hooks/pre-push .git/hooks/pre-push; chmod 700 .git/hooks/pre-push

CMD="ssh [email protected] -i ~/.vagrant.d/insecure_private_key 'cd /vagrant/tests; /vagrant/vendor/bin/phpunit'"
protected_branch='master'

# Check if we actually have commits to push
commits=`git log @{u}..`
if [ -z "$commits" ]; then
    exit 0
fi

current_branch=$(git symbolic-ref HEAD | sed -e 's,.*/\(.*\),\1,')

if [[ $current_branch = $protected_branch ]]; then
    eval $CMD
    RESULT=$?
    if [ $RESULT -ne 0 ]; then
        echo "failed $CMD"
        exit 1
    fi
fi
exit 0

As you can see the example uses a protected branch, subject of the pre-push hook.

4
  • What is RESULT=$? in bash do?
    – Jwan622
    Sep 10, 2021 at 18:05
  • 1
    @Jwan622 it offers the result of the last operation. #false #echo $? 1 #true #echo $? 0 #
    – INS
    Nov 22, 2021 at 13:45
  • don't post here ssh accesses with usernames and IP addresses, it's a security risk Oct 12, 2022 at 21:34
  • @JoãoPimentelFerreira where are those posted ?
    – Jimmy Kane
    Oct 17, 2022 at 9:22
14

If you are using the command line, the easiest way to do this is to write a push script that runs your unit tests and, if they succeed, completes the push.

Edit

As of git 1.8.2 this answer is outdated. See manojlds's answer above.

3
  • do you mean not using hooks at all? just replace "git pull" with, for example, "git uinttestspull"? that 's not exactly what I need Nov 16, 2010 at 16:22
  • 1
    @sheepwalker: s/pull/push/, and use an alias to make it nice and short.
    – Cascabel
    Nov 16, 2010 at 18:02
  • @sheepwalker Yeah, that's not exactly what you asked for, but like @calmh said, there's no pre-push hooks.
    – kubi
    Nov 16, 2010 at 20:08
6

There isn't a hook for it, because a push isn't an operation that modifies your repository.

You can do the checks on the receiving side though, in the post-receive hook. That is where you would usually reject an incoming push. Running unit tests might be a little intensive to do in a hook, but that's up to you.

6

For the record, there is a patch to Git 1.6 that adds a pre-push hook. I don't know whether it works against 1.7.

Rather than mess with that, you could run push script like @kubi recommended. You could also make it a Rake task instead so it's in your repo. ruby-git could help with this. If you check the target repo, you could run tests only when pushing to the production repo.

Finally, you could run your tests in your pre-commit hook but check for what branch is being committed to. Then you could have a, say, a production branch that requires all tests pass before accepting a commit but your master doesn't care. limerick_rake may be useful in that scenario.

1
  • thanks, actually I've already chosen the last variant (Finally, you could run your tests in your pre-commit hook..) Mar 1, 2011 at 15:56
4

The script linked by the highly-voted answer shows the parameters etc to the pre-push hook ($1 is remote name, $2 URL) and how to access the commits (lines read from stdin have structure <local ref> <local sha1> <remote ref> <remote sha1>)

#!/bin/sh

# An example hook script to verify what is about to be pushed.  Called by "git
# push" after it has checked the remote status, but before anything has been
# pushed.  If this script exits with a non-zero status nothing will be pushed.
#
# This hook is called with the following parameters:
#
# $1 -- Name of the remote to which the push is being done
# $2 -- URL to which the push is being done
#
# If pushing without using a named remote those arguments will be equal.
#
# Information about the commits which are being pushed is supplied as lines to
# the standard input in the form:
#
#   <local ref> <local sha1> <remote ref> <remote sha1>
#
# This sample shows how to prevent push of commits where the log message starts
# with "WIP" (work in progress).

remote="$1"
url="$2"

z40=0000000000000000000000000000000000000000

while read local_ref local_sha remote_ref remote_sha
do
    if [ "$local_sha" = $z40 ]
    then
        # Handle delete
        :
    else
        if [ "$remote_sha" = $z40 ]
        then
            # New branch, examine all commits
            range="$local_sha"
        else
            # Update to existing branch, examine new commits
            range="$remote_sha..$local_sha"
        fi

        # Check for WIP commit
        commit=`git rev-list -n 1 --grep '^WIP' "$range"`
        if [ -n "$commit" ]
        then
            echo >&2 "Found WIP commit in $local_ref, not pushing"
            exit 1
        fi
    fi
done

exit 0
3

I would rather run the test in a pre-commit-hook. Because the change is already recorded when committing. Push and pull only exchange information about already recorded changed. If a test fails you would already have a "broken" revision in your repository. Whether you're pushing it or not.

11
  • 238
    I generally agree, though if you're in the habit of making a lot of incremental commits to squash later, and the test suite is large, this could be impractical.
    – Cascabel
    Nov 16, 2010 at 18:03
  • 42
    Down voted because - while informative - it completely ignores the OP's question. Jan 16, 2017 at 18:24
  • 12
    @calder.ty - Nah. manojlds better addresses what matters. In fact, pre-commit hooks that run tests are generally a bad idea imo. It assumes that all things that get committed must pass tests. Which is bad for common work flows that focus on collaboration. So yea...I disagree; its not a better way to do "it" nor does it address the question. May 8, 2017 at 16:16
  • 10
    This is not an answer to the question. It is just a personal opinion and as such does not belong as an answer
    – gman
    Mar 14, 2018 at 2:03
  • 3
    If the responder wished to simply share their opinion, the appropriate response would be to either 1) respond with a comment, or 2) answer the question as expected and then provide their personal opinion below it for better visibility. Apr 29, 2019 at 18:59

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.