summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2007-12-02 17:24:22 -0600
committerDan McGee <dan@archlinux.org>2007-12-02 17:24:22 -0600
commit7341d0954194149d5659dbd33ea4b2f9f005b54a (patch)
treee80f6dfeb1a73079543b2123910fe0061fe5c764
parent32e625db1436db6ac73851d39024329afc06ece1 (diff)
downloadpacman-7341d0954194149d5659dbd33ea4b2f9f005b54a.tar.xz
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 <dan@archlinux.org>
-rw-r--r--lib/libalpm/add.c2
-rw-r--r--lib/libalpm/deps.c2
-rw-r--r--pactest/tests/pacman001.py5
-rw-r--r--pactest/tests/pacman002.py5
-rw-r--r--pactest/tests/pacman003.py5
-rw-r--r--pactest/tests/pacman004.py5
6 files changed, 22 insertions, 2 deletions
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")