From aa9b1b9d49ec37d28b6f576656170d47618e25b5 Mon Sep 17 00:00:00 2001 From: Yann Dirson Date: Sun, 16 Apr 2006 12:52:28 +0200 Subject: [PATCH] Add a testsuite framework copied from git-core See git's t/README for details on how to use this framework. There is no integration yet in the toplevel Makefile, I'll let python masters take care of this. Use "make -C t" to run the tests for now. A patch-naming policy should be defined for stgit, since the git one does not apply. Signed-off-by: Yann Dirson --- TODO | 2 +- t/Makefile | 25 +++++++ t/README | 210 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ t/t0000-dummy.sh | 17 +++++ t/test-lib.sh | 203 +++++++++++++++++++++++++++++++++++++++++++++++++++++ 5 files changed, 456 insertions(+), 1 deletion(-) create mode 100644 t/Makefile create mode 100644 t/README create mode 100755 t/t0000-dummy.sh create mode 100755 t/test-lib.sh diff --git a/TODO b/TODO index e5affe0..d97ffd1 100644 --- a/TODO +++ b/TODO @@ -6,7 +6,7 @@ The TODO list until 1.0: - debian package support - man page - code execution allowed from templates -- regression tests +- more regression tests - release 1.0 diff --git a/t/Makefile b/t/Makefile new file mode 100644 index 0000000..d5d7b6f --- /dev/null +++ b/t/Makefile @@ -0,0 +1,25 @@ +# Run tests +# +# Copyright (c) 2005 Junio C Hamano +# + +#GIT_TEST_OPTS=--verbose --debug +SHELL_PATH ?= $(SHELL) +TAR ?= $(TAR) + +# Shell quote; +SHELL_PATH_SQ = $(subst ','\'',$(SHELL_PATH)) + +T = $(wildcard t[0-9][0-9][0-9][0-9]-*.sh) + +all: $(T) clean + +$(T): + @echo "*** $@ ***"; '$(SHELL_PATH_SQ)' $@ $(GIT_TEST_OPTS) + +clean: + rm -fr trash + +.PHONY: $(T) clean +.NOPARALLEL: + diff --git a/t/README b/t/README new file mode 100644 index 0000000..d88bad2 --- /dev/null +++ b/t/README @@ -0,0 +1,210 @@ +Core GIT Tests +============== + +This directory holds many test scripts for core GIT tools. The +first part of this short document describes how to run the tests +and read their output. + +When fixing the tools or adding enhancements, you are strongly +encouraged to add tests in this directory to cover what you are +trying to fix or enhance. The later part of this short document +describes how your test scripts should be organized. + +The mechanism that powers this testsuite is directly imported from the +Core GIT Tests, in directory t/ of the git repository. Files are base +on Core GIT version 1.3.0.rc4.g5069. + + +Running Tests +------------- + +The easiest way to run tests is to say "make -C t". This runs all +the tests. + + *** t0000-basic.sh *** + * ok 1: .git/objects should be empty after git-init-db in an empty repo. + * ok 2: .git/objects should have 256 subdirectories. + * ok 3: git-update-index without --add should fail adding. + ... + * ok 23: no diff after checkout and git-update-index --refresh. + * passed all 23 test(s) + *** t0100-environment-names.sh *** + * ok 1: using old names should issue warnings. + * ok 2: using old names but having new names should not issue warnings. + ... + +Or you can run each test individually from command line, like +this: + + $ sh ./t3001-ls-files-killed.sh + * ok 1: git-update-index --add to add various paths. + * ok 2: git-ls-files -k to show killed files. + * ok 3: validate git-ls-files -k output. + * passed all 3 test(s) + +You can pass --verbose (or -v), --debug (or -d), and --immediate +(or -i) command line argument to the test. + +--verbose:: + This makes the test more verbose. Specifically, the + command being run and their output if any are also + output. + +--debug:: + This may help the person who is developing a new test. + It causes the command defined with test_debug to run. + +--immediate:: + This causes the test to immediately exit upon the first + failed test. + + +Naming Tests +------------ + +The test files are named as: + + tNNNN-commandname-details.sh + +where N is a decimal digit. + +Here is a proposal for numbering, loosely based on the Core GIT +numbering conventions. + +First two digit tells the particular command we are testing: + + 00 - stgit itself + 10 - branch + 11 - clone + 12 - push + +Third and fourth digit (optionally) tells the particular switch or +group of switches we are testing. + +If you create files under t/ directory (i.e. here) that is not +the top-level test script, never name the file to match the above +pattern. The Makefile here considers all such files as the +top-level test script and tries to run all of them. A care is +especially needed if you are creating a common test library +file, similar to test-lib.sh, because such a library file may +not be suitable for standalone execution. + + +Writing Tests +------------- + +The test script is written as a shell script. It should start +with the standard "#!/bin/sh" with copyright notices, and an +assignment to variable 'test_description', like this: + + #!/bin/sh + # + # Copyright (c) 2005 Junio C Hamano + # + + test_description='xxx test (option --frotz) + + This test registers the following structure in the cache + and tries to run git-ls-files with option --frotz.' + + +Source 'test-lib.sh' +-------------------- + +After assigning test_description, the test script should source +test-lib.sh like this: + + . ./test-lib.sh + +This test harness library does the following things: + + - If the script is invoked with command line argument --help + (or -h), it shows the test_description and exits. + + - Creates an empty test directory with an empty .git/objects + database and chdir(2) into it. This directory is 't/trash' + if you must know, but I do not think you care. + + - Defines standard test helper functions for your scripts to + use. These functions are designed to make all scripts behave + consistently when command line arguments --verbose (or -v), + --debug (or -d), and --immediate (or -i) is given. + + +End with test_done +------------------ + +Your script will be a sequence of tests, using helper functions +from the test harness library. At the end of the script, call +'test_done'. + + +Test harness library +-------------------- + +There are a handful helper functions defined in the test harness +library for your script to use. + + - test_expect_success