From 169381bbce998807c119f291d69b8ae1d8fb5785 Mon Sep 17 00:00:00 2001 From: scott Chacon Date: Wed, 7 Nov 2007 17:19:09 -0800 Subject: added some testing files and got some real tests to run --- README | 2 + Rakefile | 3 +- lib/git.rb | 3 - tests/files/index | Bin 0 -> 256 bytes tests/files/working | 1 + tests/files/working.git/HEAD | 1 + tests/files/working.git/config | 4 + tests/files/working.git/description | 1 + tests/files/working.git/hooks/applypatch-msg | 15 +++ tests/files/working.git/hooks/commit-msg | 21 +++ tests/files/working.git/hooks/post-commit | 8 ++ tests/files/working.git/hooks/post-receive | 16 +++ tests/files/working.git/hooks/post-update | 8 ++ tests/files/working.git/hooks/pre-applypatch | 14 ++ tests/files/working.git/hooks/pre-commit | 70 ++++++++++ tests/files/working.git/hooks/pre-rebase | 150 +++++++++++++++++++++ tests/files/working.git/hooks/update | 78 +++++++++++ tests/files/working.git/info/exclude | 6 + .../0b/c0d846cf80b079e763e35c3af273171bf01fca | Bin 0 -> 88 bytes .../54/5ffc79786f268524c35e1e05b1770c7c74faf1 | 3 + .../6b/790ddc5eab30f18cabdd0513e8f8dac0d2d3ed | Bin 0 -> 51 bytes .../e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 | Bin 0 -> 15 bytes tests/files/working.git/refs/heads/master | 1 + tests/test_helper.rb | 21 +++ tests/test_init.rb | 12 -- tests/units/test_git_path.rb | 29 ++++ tests/units/test_init.rb | 35 +++++ 27 files changed, 486 insertions(+), 16 deletions(-) create mode 100644 tests/files/index create mode 160000 tests/files/working create mode 100644 tests/files/working.git/HEAD create mode 100644 tests/files/working.git/config create mode 100644 tests/files/working.git/description create mode 100644 tests/files/working.git/hooks/applypatch-msg create mode 100644 tests/files/working.git/hooks/commit-msg create mode 100644 tests/files/working.git/hooks/post-commit create mode 100644 tests/files/working.git/hooks/post-receive create mode 100644 tests/files/working.git/hooks/post-update create mode 100644 tests/files/working.git/hooks/pre-applypatch create mode 100644 tests/files/working.git/hooks/pre-commit create mode 100644 tests/files/working.git/hooks/pre-rebase create mode 100644 tests/files/working.git/hooks/update create mode 100644 tests/files/working.git/info/exclude create mode 100644 tests/files/working.git/objects/0b/c0d846cf80b079e763e35c3af273171bf01fca create mode 100644 tests/files/working.git/objects/54/5ffc79786f268524c35e1e05b1770c7c74faf1 create mode 100644 tests/files/working.git/objects/6b/790ddc5eab30f18cabdd0513e8f8dac0d2d3ed create mode 100644 tests/files/working.git/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 create mode 100644 tests/files/working.git/refs/heads/master delete mode 100644 tests/test_init.rb create mode 100644 tests/units/test_git_path.rb create mode 100644 tests/units/test_init.rb diff --git a/README b/README index e881eb2..a5f93f8 100644 --- a/README +++ b/README @@ -37,6 +37,8 @@ g.index.writable? g.repo g.dir +********** + g.log # returns array of Git::Commit objects g.log.stat g.log.stat.summary diff --git a/Rakefile b/Rakefile index bbc6c6f..20aa494 100644 --- a/Rakefile +++ b/Rakefile @@ -25,7 +25,8 @@ task :default => "pkg/#{spec.name}-#{spec.version}.gem" do puts "generated latest version" end -task :test do +desc "Run Unit Tests" +task :test do |t| require File.dirname(__FILE__) + '/tests/all_tests.rb' end diff --git a/lib/git.rb b/lib/git.rb index 25dc2b3..2e372d0 100644 --- a/lib/git.rb +++ b/lib/git.rb @@ -51,6 +51,3 @@ module Git end end - -g = Git.open('/Users/schacon/Sites/glue') -g = Git.repo('/Users/schacon/Sites/glue/.git') diff --git a/tests/files/index b/tests/files/index new file mode 100644 index 0000000..3f1679d Binary files /dev/null and b/tests/files/index differ diff --git a/tests/files/working b/tests/files/working new file mode 160000 index 0000000..545ffc7 --- /dev/null +++ b/tests/files/working @@ -0,0 +1 @@ +Subproject commit 545ffc79786f268524c35e1e05b1770c7c74faf1 diff --git a/tests/files/working.git/HEAD b/tests/files/working.git/HEAD new file mode 100644 index 0000000..cb089cd --- /dev/null +++ b/tests/files/working.git/HEAD @@ -0,0 +1 @@ +ref: refs/heads/master diff --git a/tests/files/working.git/config b/tests/files/working.git/config new file mode 100644 index 0000000..07d359d --- /dev/null +++ b/tests/files/working.git/config @@ -0,0 +1,4 @@ +[core] + repositoryformatversion = 0 + filemode = true + bare = true diff --git a/tests/files/working.git/description b/tests/files/working.git/description new file mode 100644 index 0000000..c6f25e8 --- /dev/null +++ b/tests/files/working.git/description @@ -0,0 +1 @@ +Unnamed repository; edit this file to name it for gitweb. diff --git a/tests/files/working.git/hooks/applypatch-msg b/tests/files/working.git/hooks/applypatch-msg new file mode 100644 index 0000000..02de1ef --- /dev/null +++ b/tests/files/working.git/hooks/applypatch-msg @@ -0,0 +1,15 @@ +#!/bin/sh +# +# An example hook script to check the commit log message taken by +# applypatch from an e-mail message. +# +# The hook should exit with non-zero status after issuing an +# appropriate message if it wants to stop the commit. The hook is +# allowed to edit the commit message file. +# +# To enable this hook, make this file executable. + +. git-sh-setup +test -x "$GIT_DIR/hooks/commit-msg" && + exec "$GIT_DIR/hooks/commit-msg" ${1+"$@"} +: diff --git a/tests/files/working.git/hooks/commit-msg b/tests/files/working.git/hooks/commit-msg new file mode 100644 index 0000000..c5cdb9d --- /dev/null +++ b/tests/files/working.git/hooks/commit-msg @@ -0,0 +1,21 @@ +#!/bin/sh +# +# An example hook script to check the commit log message. +# Called by git-commit with one argument, the name of the file +# that has the commit message. The hook should exit with non-zero +# status after issuing an appropriate message if it wants to stop the +# commit. The hook is allowed to edit the commit message file. +# +# To enable this hook, make this file executable. + +# Uncomment the below to add a Signed-off-by line to the message. +# SOB=$(git var GIT_AUTHOR_IDENT | sed -n 's/^\(.*>\).*$/Signed-off-by: \1/p') +# grep -qs "^$SOB" "$1" || echo "$SOB" >> "$1" + +# This example catches duplicate Signed-off-by lines. + +test "" = "$(grep '^Signed-off-by: ' "$1" | + sort | uniq -c | sed -e '/^[ ]*1[ ]/d')" || { + echo >&2 Duplicate Signed-off-by lines. + exit 1 +} diff --git a/tests/files/working.git/hooks/post-commit b/tests/files/working.git/hooks/post-commit new file mode 100644 index 0000000..8be6f34 --- /dev/null +++ b/tests/files/working.git/hooks/post-commit @@ -0,0 +1,8 @@ +#!/bin/sh +# +# An example hook script that is called after a successful +# commit is made. +# +# To enable this hook, make this file executable. + +: Nothing diff --git a/tests/files/working.git/hooks/post-receive b/tests/files/working.git/hooks/post-receive new file mode 100644 index 0000000..b70c8fd --- /dev/null +++ b/tests/files/working.git/hooks/post-receive @@ -0,0 +1,16 @@ +#!/bin/sh +# +# An example hook script for the post-receive event +# +# This script is run after receive-pack has accepted a pack and the +# repository has been updated. It is passed arguments in through stdin +# in the form +# +# For example: +# aa453216d1b3e49e7f6f98441fa56946ddcd6a20 68f7abf4e6f922807889f52bc043ecd31b79f814 refs/heads/master +# +# see contrib/hooks/ for an sample, or uncomment the next line (on debian) +# + + +#. /usr/share/doc/git-core/contrib/hooks/post-receive-email diff --git a/tests/files/working.git/hooks/post-update b/tests/files/working.git/hooks/post-update new file mode 100644 index 0000000..bcba893 --- /dev/null +++ b/tests/files/working.git/hooks/post-update @@ -0,0 +1,8 @@ +#!/bin/sh +# +# An example hook script to prepare a packed repository for use over +# dumb transports. +# +# To enable this hook, make this file executable by "chmod +x post-update". + +exec git-update-server-info diff --git a/tests/files/working.git/hooks/pre-applypatch b/tests/files/working.git/hooks/pre-applypatch new file mode 100644 index 0000000..eeccc93 --- /dev/null +++ b/tests/files/working.git/hooks/pre-applypatch @@ -0,0 +1,14 @@ +#!/bin/sh +# +# An example hook script to verify what is about to be committed +# by applypatch from an e-mail message. +# +# The hook should exit with non-zero status after issuing an +# appropriate message if it wants to stop the commit. +# +# To enable this hook, make this file executable. + +. git-sh-setup +test -x "$GIT_DIR/hooks/pre-commit" && + exec "$GIT_DIR/hooks/pre-commit" ${1+"$@"} +: diff --git a/tests/files/working.git/hooks/pre-commit b/tests/files/working.git/hooks/pre-commit new file mode 100644 index 0000000..18b8730 --- /dev/null +++ b/tests/files/working.git/hooks/pre-commit @@ -0,0 +1,70 @@ +#!/bin/sh +# +# An example hook script to verify what is about to be committed. +# Called by git-commit with no arguments. The hook should +# exit with non-zero status after issuing an appropriate message if +# it wants to stop the commit. +# +# To enable this hook, make this file executable. + +# This is slightly modified from Andrew Morton's Perfect Patch. +# Lines you introduce should not have trailing whitespace. +# Also check for an indentation that has SP before a TAB. + +if git-rev-parse --verify HEAD 2>/dev/null +then + git-diff-index -p -M --cached HEAD +else + # NEEDSWORK: we should produce a diff with an empty tree here + # if we want to do the same verification for the initial import. + : +fi | +perl -e ' + my $found_bad = 0; + my $filename; + my $reported_filename = ""; + my $lineno; + sub bad_line { + my ($why, $line) = @_; + if (!$found_bad) { + print STDERR "*\n"; + print STDERR "* You have some suspicious patch lines:\n"; + print STDERR "*\n"; + $found_bad = 1; + } + if ($reported_filename ne $filename) { + print STDERR "* In $filename\n"; + $reported_filename = $filename; + } + print STDERR "* $why (line $lineno)\n"; + print STDERR "$filename:$lineno:$line\n"; + } + while (<>) { + if (m|^diff --git a/(.*) b/\1$|) { + $filename = $1; + next; + } + if (/^@@ -\S+ \+(\d+)/) { + $lineno = $1 - 1; + next; + } + if (/^ /) { + $lineno++; + next; + } + if (s/^\+//) { + $lineno++; + chomp; + if (/\s$/) { + bad_line("trailing whitespace", $_); + } + if (/^\s* /) { + bad_line("indent SP followed by a TAB", $_); + } + if (/^(?:[<>=]){7}/) { + bad_line("unresolved merge conflict", $_); + } + } + } + exit($found_bad); +' diff --git a/tests/files/working.git/hooks/pre-rebase b/tests/files/working.git/hooks/pre-rebase new file mode 100644 index 0000000..981c454 --- /dev/null +++ b/tests/files/working.git/hooks/pre-rebase @@ -0,0 +1,150 @@ +#!/bin/sh +# +# Copyright (c) 2006 Junio C Hamano +# + +publish=next +basebranch="$1" +if test "$#" = 2 +then + topic="refs/heads/$2" +else + topic=`git symbolic-ref HEAD` +fi + +case "$basebranch,$topic" in +master,refs/heads/??/*) + ;; +*) + exit 0 ;# we do not interrupt others. + ;; +esac + +# Now we are dealing with a topic branch being rebased +# on top of master. Is it OK to rebase it? + +# Is topic fully merged to master? +not_in_master=`git-rev-list --pretty=oneline ^master "$topic"` +if test -z "$not_in_master" +then + echo >&2 "$topic is fully merged to master; better remove it." + exit 1 ;# we could allow it, but there is no point. +fi + +# Is topic ever merged to next? If so you should not be rebasing it. +only_next_1=`git-rev-list ^master "^$topic" ${publish} | sort` +only_next_2=`git-rev-list ^master ${publish} | sort` +if test "$only_next_1" = "$only_next_2" +then + not_in_topic=`git-rev-list "^$topic" master` + if test -z "$not_in_topic" + then + echo >&2 "$topic is already up-to-date with master" + exit 1 ;# we could allow it, but there is no point. + else + exit 0 + fi +else + not_in_next=`git-rev-list --pretty=oneline ^${publish} "$topic"` + perl -e ' + my $topic = $ARGV[0]; + my $msg = "* $topic has commits already merged to public branch:\n"; + my (%not_in_next) = map { + /^([0-9a-f]+) /; + ($1 => 1); + } split(/\n/, $ARGV[1]); + for my $elem (map { + /^([0-9a-f]+) (.*)$/; + [$1 => $2]; + } split(/\n/, $ARGV[2])) { + if (!exists $not_in_next{$elem->[0]}) { + if ($msg) { + print STDERR $msg; + undef $msg; + } + print STDERR " $elem->[1]\n"; + } + } + ' "$topic" "$not_in_next" "$not_in_master" + exit 1 +fi + +exit 0 + +################################################################ + +This sample hook safeguards topic branches that have been +published from being rewound. + +The workflow assumed here is: + + * Once a topic branch forks from "master", "master" is never + merged into it again (either directly or indirectly). + + * Once a topic branch is fully cooked and merged into "master", + it is deleted. If you need to build on top of it to correct + earlier mistakes, a new topic branch is created by forking at + the tip of the "master". This is not strictly necessary, but + it makes it easier to keep your history simple. + + * Whenever you need to test or publish your changes to topic + branches, merge them into "next" branch. + +The script, being an example, hardcodes the publish branch name +to be "next", but it is trivial to make it configurable via +$GIT_DIR/config mechanism. + +With this workflow, you would want to know: + +(1) ... if a topic branch has ever been merged to "next". Young + topic branches can have stupid mistakes you would rather + clean up before publishing, and things that have not been + merged into other branches can be easily rebased without + affecting other people. But once it is published, you would + not want to rewind it. + +(2) ... if a topic branch has been fully merged to "master". + Then you can delete it. More importantly, you should not + build on top of it -- other people may already want to + change things related to the topic as patches against your + "master", so if you need further changes, it is better to + fork the topic (perhaps with the same name) afresh from the + tip of "master". + +Let's look at this example: + + o---o---o---o---o---o---o---o---o---o "next" + / / / / + / a---a---b A / / + / / / / + / / c---c---c---c B / + / / / \ / + / / / b---b C \ / + / / / / \ / + ---o---o---o---o---o---o---o---o---o---o---o "master" + + +A, B and C are topic branches. + + * A has one fix since it was merged up to "next". + + * B has finished. It has been fully merged up to "master" and "next", + and is ready to be deleted. + + * C has not merged to "next" at all. + +We would want to allow C to be rebased, refuse A, and encourage +B to be deleted. + +To compute (1): + + git-rev-list ^master ^topic next + git-rev-list ^master next + + if these match, topic has not merged in next at all. + +To compute (2): + + git-rev-list master..topic + + if this is empty, it is fully merged to "master". diff --git a/tests/files/working.git/hooks/update b/tests/files/working.git/hooks/update new file mode 100644 index 0000000..d8c7626 --- /dev/null +++ b/tests/files/working.git/hooks/update @@ -0,0 +1,78 @@ +#!/bin/sh +# +# An example hook script to blocks unannotated tags from entering. +# Called by git-receive-pack with arguments: refname sha1-old sha1-new +# +# To enable this hook, make this file executable by "chmod +x update". +# +# Config +# ------ +# hooks.allowunannotated +# This boolean sets whether unannotated tags will be allowed into the +# repository. By default they won't be. +# + +# --- Command line +refname="$1" +oldrev="$2" +newrev="$3" + +# --- Safety check +if [ -z "$GIT_DIR" ]; then + echo "Don't run this script from the command line." >&2 + echo " (if you want, you could supply GIT_DIR then run" >&2 + echo " $0 )" >&2 + exit 1 +fi + +if [ -z "$refname" -o -z "$oldrev" -o -z "$newrev" ]; then + echo "Usage: $0 " >&2 + exit 1 +fi + +# --- Config +allowunannotated=$(git-repo-config --bool hooks.allowunannotated) + +# check for no description +projectdesc=$(sed -e '1p' "$GIT_DIR/description") +if [ -z "$projectdesc" -o "$projectdesc" = "Unnamed repository; edit this file to name it for gitweb" ]; then + echo "*** Project description file hasn't been set" >&2 + exit 1 +fi + +# --- Check types +# if $newrev is 0000...0000, it's a commit to delete a branch +if [ "$newrev" = "0000000000000000000000000000000000000000" ]; then + newrev_type=commit +else + newrev_type=$(git-cat-file -t $newrev) +fi + +case "$refname","$newrev_type" in + refs/tags/*,commit) + # un-annotated tag + short_refname=${refname##refs/tags/} + if [ "$allowunannotated" != "true" ]; then + echo "*** The un-annotated tag, $short_refname, is not allowed in this repository" >&2 + echo "*** Use 'git tag [ -a | -s ]' for tags you want to propagate." >&2 + exit 1 + fi + ;; + refs/tags/*,tag) + # annotated tag + ;; + refs/heads/*,commit) + # branch + ;; + refs/remotes/*,commit) + # tracking branch + ;; + *) + # Anything else (is there anything else?) + echo "*** Update hook: unknown type of update to ref $refname of type $newrev_type" >&2 + exit 1 + ;; +esac + +# --- Finished +exit 0 diff --git a/tests/files/working.git/info/exclude b/tests/files/working.git/info/exclude new file mode 100644 index 0000000..2c87b72 --- /dev/null +++ b/tests/files/working.git/info/exclude @@ -0,0 +1,6 @@ +# git-ls-files --others --exclude-from=.git/info/exclude +# Lines that start with '#' are comments. +# For a project mostly in C, the following would be a good set of +# exclude patterns (uncomment them if you want to use them): +# *.[oa] +# *~ diff --git a/tests/files/working.git/objects/0b/c0d846cf80b079e763e35c3af273171bf01fca b/tests/files/working.git/objects/0b/c0d846cf80b079e763e35c3af273171bf01fca new file mode 100644 index 0000000..d22d1d5 Binary files /dev/null and b/tests/files/working.git/objects/0b/c0d846cf80b079e763e35c3af273171bf01fca differ diff --git a/tests/files/working.git/objects/54/5ffc79786f268524c35e1e05b1770c7c74faf1 b/tests/files/working.git/objects/54/5ffc79786f268524c35e1e05b1770c7c74faf1 new file mode 100644 index 0000000..0d0d2d2 --- /dev/null +++ b/tests/files/working.git/objects/54/5ffc79786f268524c35e1e05b1770c7c74faf1 @@ -0,0 +1,3 @@ +xQ + D)@hBO٬iVY 3cxC)ƽ꾟/U5,& +p3ɲb5,L TxW](ժ/ѷB $%ԝQ #U]30:}m&Ę߬Q8'N \ No newline at end of file diff --git a/tests/files/working.git/objects/6b/790ddc5eab30f18cabdd0513e8f8dac0d2d3ed b/tests/files/working.git/objects/6b/790ddc5eab30f18cabdd0513e8f8dac0d2d3ed new file mode 100644 index 0000000..6d20206 Binary files /dev/null and b/tests/files/working.git/objects/6b/790ddc5eab30f18cabdd0513e8f8dac0d2d3ed differ diff --git a/tests/files/working.git/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 b/tests/files/working.git/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 new file mode 100644 index 0000000..7112238 Binary files /dev/null and b/tests/files/working.git/objects/e6/9de29bb2d1d6434b8b29ae775ad8c2e48c5391 differ diff --git a/tests/files/working.git/refs/heads/master b/tests/files/working.git/refs/heads/master new file mode 100644 index 0000000..4b03b19 --- /dev/null +++ b/tests/files/working.git/refs/heads/master @@ -0,0 +1 @@ +545ffc79786f268524c35e1e05b1770c7c74faf1 diff --git a/tests/test_helper.rb b/tests/test_helper.rb index 2216f60..4a1cac9 100644 --- a/tests/test_helper.rb +++ b/tests/test_helper.rb @@ -1,2 +1,23 @@ require 'test/unit' +require 'fileutils' require File.dirname(__FILE__) + '/../lib/git' + +class Test::Unit::TestCase + + def set_file_paths + @wdir = File.join(File.dirname(__FILE__), 'files', 'working') + @wbare = File.join(File.dirname(__FILE__), 'files', 'working.git') + @index = File.join(File.dirname(__FILE__), 'files', 'index') + end + + def in_temp_dir + filename = 'git_test' + Time.now.to_i.to_s + rand(300).to_s + tmp_path = File.join("/tmp/", filename) + FileUtils.mkdir(tmp_path) + Dir.chdir tmp_path do + yield tmp_path + end + FileUtils.rm_r(tmp_path) + end + +end \ No newline at end of file diff --git a/tests/test_init.rb b/tests/test_init.rb deleted file mode 100644 index a8e93e9..0000000 --- a/tests/test_init.rb +++ /dev/null @@ -1,12 +0,0 @@ -#!/usr/bin/env ruby - -require File.dirname(__FILE__) + '/test_helper' - -class TestInit < Test::Unit::TestCase - def setup - end - - def test_open - Git.open - end -end diff --git a/tests/units/test_git_path.rb b/tests/units/test_git_path.rb new file mode 100644 index 0000000..c136abb --- /dev/null +++ b/tests/units/test_git_path.rb @@ -0,0 +1,29 @@ +#!/usr/bin/env ruby + +require File.dirname(__FILE__) + '/../test_helper' + +class TestGitPath < Test::Unit::TestCase + + def setup + set_file_paths + @git = Git.open(@wdir) + end + + def test_readables + assert(@git.dir.readable?) + assert(@git.index.readable?) + assert(@git.repo.readable?) + end + + def test_readables + in_temp_dir do |dir| + FileUtils.cp_r(@wdir, 'test') + g = Git.open(File.join(dir, 'test')) + + assert(g.dir.writable?) + assert(g.index.writable?) + assert(g.repo.writable?) + end + end + +end \ No newline at end of file diff --git a/tests/units/test_init.rb b/tests/units/test_init.rb new file mode 100644 index 0000000..56d2a18 --- /dev/null +++ b/tests/units/test_init.rb @@ -0,0 +1,35 @@ +#!/usr/bin/env ruby + +require File.dirname(__FILE__) + '/../test_helper' + +class TestInit < Test::Unit::TestCase + def setup + set_file_paths + end + + def test_open_simple + g = Git.open(@wdir) + assert_equal(g.dir.path, @wdir) + assert_equal(g.repo.path, File.join(@wdir, '.git')) + assert_equal(g.index.path, File.join(@wdir, '.git', 'index')) + end + + def test_open_opts + g = Git.open @wdir, :repository => @wbare, :index => @index + assert_equal(g.repo.path, @wbare) + assert_equal(g.index.path, @index) + end + + def test_git_bare + g = Git.repo @wbare + assert_equal(g.repo.path, @wbare) + end + + # trying to open a git project using a bare repo - rather than using Git.repo + def test_git_open_error + assert_raise ArgumentError do + g = Git.open @wbare + end + end + +end -- cgit