[pacman-dev] [PATCH] Add simple vercmp test script

Dan McGee dan at archlinux.org
Fri May 30 20:02:11 EDT 2008


Commit 84283672853350a84d2a71b72dc06e180cad1587 updated the versioncmp code
in libalpm. Unfortunately for us, it also introduced the regression that
becomes apparant with the following upgrade:

warning: sonata: local (1.5-2) is newer than extra (1.5.1-2)

Add a vercmptest.sh test script that is run during the make check phase
which now points out three regressions in the version comparison function
that will need fixing. All current tests in this script pass with the old
versioncmp code.

Signed-off-by: Dan McGee <dan at archlinux.org>
---
 Makefile.am           |    6 ++-
 pactest/vercmptest.sh |   77 +++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 81 insertions(+), 2 deletions(-)
 create mode 100755 pactest/vercmptest.sh

diff --git a/Makefile.am b/Makefile.am
index dc08542..ea3b0e0 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -11,12 +11,14 @@ EXTRA_DIST = HACKING
 pkgdatadir = ${datadir}/${PACKAGE}
 dist_pkgdata_DATA = PKGBUILD.proto proto.install ChangeLog.proto
 
-# run the pactest test suite
-check-local:  src/pacman
+# run the pactest test suite and vercmp tests
+check-local: pactest src/pacman src/util
 	$(PYTHON) $(top_srcdir)/pactest/pactest.py --debug=1 \
 		--test $(top_srcdir)/pactest/tests/*.py \
 		-p $(top_builddir)/src/pacman/pacman
 	rm -rf $(top_builddir)/root
+	$(SH) $(top_srcdir)/pactest/vercmptest.sh \
+		$(top_builddir)/src/util/vercmp
 
 # create the pacman DB and cache directories upon install
 install-data-local:
diff --git a/pactest/vercmptest.sh b/pactest/vercmptest.sh
new file mode 100755
index 0000000..62a3e32
--- /dev/null
+++ b/pactest/vercmptest.sh
@@ -0,0 +1,77 @@
+#!/bin/sh
+#
+# vercmptest - a test suite for the vercmp/libalpm program
+#
+#   Copyright (c) 2008 by Dan McGee <dan at archlinux.org>
+#
+#   This program is free software; you can redistribute it and/or modify
+#   it under the terms of the GNU General Public License as published by
+#   the Free Software Foundation; either version 2 of the License, or
+#   (at your option) any later version.
+#
+#   This program is distributed in the hope that it will be useful,
+#   but WITHOUT ANY WARRANTY; without even the implied warranty of
+#   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#   GNU General Public License for more details.
+#
+#   You should have received a copy of the GNU General Public License
+#   along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+# default binary if one was not specified as $1
+bin='vercmp'
+# holds count of failed tests
+failure=0
+
+# args:
+# fail ver1 ver2 ret expected
+fail() {
+	echo "unexpected failure:"
+	echo "  ver1: $1 ver2: $2 ret: $3 expected: $4"
+	failure=$(expr $failure + 1)
+}
+
+# args:
+# runtest ver1 ver2 expected
+runtest() {
+	ret=$($bin $1 $2)
+	[ $ret -eq $3 ] || fail $1 $2 $ret $3
+}
+
+# use first arg as our binary if specified
+[ -n "$1" ] && bin="$1"
+
+# BEGIN TESTS
+
+# all similar length, no pkgrel
+runtest 1.5.0 1.5.0  0
+runtest 1.5.0 1.5.1 -1
+runtest 1.5.1 1.5.0  1
+
+# mixed length
+runtest 1.5   1.5.1 -1
+runtest 1.5.1 1.5    1
+
+# with pkgrel, simple
+runtest 1.5.0-1 1.5.0-1  0
+runtest 1.5.0-1 1.5.0-2 -1
+runtest 1.5.0-1 1.5.1-1 -1
+runtest 1.5.0-2 1.5.1-1 -1
+
+# with pkgrel, mixed lengths
+runtest 1.5-1   1.5.1-1 -1
+runtest 1.5-2   1.5.1-1 -1
+runtest 1.5-2   1.5.1-2 -1
+
+# mixed pkgrel inclusion
+runtest 1.5   1.5-1 0
+runtest 1.5-1 1.5   0
+
+#END TESTS
+
+if [ $failure -eq 0 ]; then
+	echo "All tests successful"
+	exit 0
+fi
+
+echo "$failure failed tests"
+exit 1
-- 
1.5.5.3





More information about the pacman-dev mailing list