From 7341d0954194149d5659dbd33ea4b2f9f005b54a Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Sun, 2 Dec 2007 17:24:22 -0600 Subject: Add 4 new pactests to get better code coverage by pactest I started playing around with gcov today and it showed a few places in the code that we don't test at all. This is the start of ensuring that we execute most of the code in our codebase. Signed-off-by: Dan McGee --- lib/libalpm/add.c | 2 +- lib/libalpm/deps.c | 2 +- pactest/tests/pacman001.py | 5 +++++ pactest/tests/pacman002.py | 5 +++++ pactest/tests/pacman003.py | 5 +++++ pactest/tests/pacman004.py | 5 +++++ 6 files changed, 22 insertions(+), 2 deletions(-) create mode 100644 pactest/tests/pacman001.py create mode 100644 pactest/tests/pacman002.py create mode 100644 pactest/tests/pacman003.py create mode 100644 pactest/tests/pacman004.py diff --git a/lib/libalpm/add.c b/lib/libalpm/add.c index 4ea3c869..3a573ddd 100644 --- a/lib/libalpm/add.c +++ b/lib/libalpm/add.c @@ -428,7 +428,7 @@ static int extract_single_file(struct archive *archive, } } } - /* else if(S_ISLNK(entrymode)) { */ + /* else if(S_ISLNK(entrymode)) */ /* case 5,8: don't need to do anything special */ } diff --git a/lib/libalpm/deps.c b/lib/libalpm/deps.c index 350df6b8..7b7d1097 100644 --- a/lib/libalpm/deps.c +++ b/lib/libalpm/deps.c @@ -593,7 +593,7 @@ int _alpm_resolvedeps(pmdb_t *local, alpm_list_t *dbs_sync, pmpkg_t *syncpkg, missdepstring, miss->target); free(missdepstring); if(data) { - MALLOC(miss, sizeof(pmdepmissing_t), 0); + MALLOC(miss, sizeof(pmdepmissing_t),/*nothing*/); if(!miss) { pm_errno = PM_ERR_MEMORY; FREELIST(*data); diff --git a/pactest/tests/pacman001.py b/pactest/tests/pacman001.py new file mode 100644 index 00000000..d467e3f2 --- /dev/null +++ b/pactest/tests/pacman001.py @@ -0,0 +1,5 @@ +self.description = "Test command line option (--version)" + +self.args = "--version" + +self.addrule("PACMAN_RETCODE=2") diff --git a/pactest/tests/pacman002.py b/pactest/tests/pacman002.py new file mode 100644 index 00000000..c0217259 --- /dev/null +++ b/pactest/tests/pacman002.py @@ -0,0 +1,5 @@ +self.description = "Test command line option (--help)" + +self.args = "--help" + +self.addrule("PACMAN_RETCODE=2") diff --git a/pactest/tests/pacman003.py b/pactest/tests/pacman003.py new file mode 100644 index 00000000..b5275943 --- /dev/null +++ b/pactest/tests/pacman003.py @@ -0,0 +1,5 @@ +self.description = "Test command line option (-S --help)" + +self.args = "-S --help" + +self.addrule("PACMAN_RETCODE=2") diff --git a/pactest/tests/pacman004.py b/pactest/tests/pacman004.py new file mode 100644 index 00000000..d8d6b102 --- /dev/null +++ b/pactest/tests/pacman004.py @@ -0,0 +1,5 @@ +self.description = "Test command line option (-v)" + +self.args = "-v" + +self.addrule("PACMAN_RETCODE=1") -- cgit v1.1-4-g5e80