diff options
author | Dan McGee <dan@archlinux.org> | 2010-10-27 04:30:22 +0200 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2011-01-22 18:14:42 +0100 |
commit | 946f4af7f376c22d33bb24ff997c99231a05c0e7 (patch) | |
tree | 672c482cb56c1fc286bbe2cd776b9f2b3c45d417 /test | |
parent | 2c6be06bbab70a2433450290c66cc8b3af24e1d4 (diff) | |
download | pacman-946f4af7f376c22d33bb24ff997c99231a05c0e7.tar.gz pacman-946f4af7f376c22d33bb24ff997c99231a05c0e7.tar.xz |
pactest: pylint changes for pmrule
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'test')
-rwxr-xr-x | test/pacman/pmrule.py | 23 | ||||
-rwxr-xr-x | test/pacman/util.py | 3 |
2 files changed, 12 insertions, 14 deletions
diff --git a/test/pacman/pmrule.py b/test/pacman/pmrule.py index 89ae3f49..c69e3cd3 100755 --- a/test/pacman/pmrule.py +++ b/test/pacman/pmrule.py @@ -16,8 +16,9 @@ # along with this program. If not, see <http://www.gnu.org/licenses/>. import os -from util import * -from stat import * +import stat + +import util class pmrule: """Rule object @@ -54,11 +55,11 @@ class pmrule: if retcode != int(key): success = 0 elif case == "OUTPUT": - logfile = os.path.join(root, LOGFILE) + logfile = os.path.join(root, util.LOGFILE) if not os.access(logfile, os.F_OK): print "LOGFILE not found, cannot validate 'OUTPUT' rule" success = 0 - elif not grep(os.path.join(root, LOGFILE), key): + elif not util.grep(os.path.join(root, util.LOGFILE), key): success = 0 else: print "PACMAN rule '%s' not found" % case @@ -68,8 +69,8 @@ class pmrule: if not newpkg: success = 0 else: - vprint("\tnewpkg.checksum : %s" % newpkg.checksum) - vprint("\tnewpkg.mtime : %s" % newpkg.mtime) + util.vprint("\tnewpkg.checksum : %s" % newpkg.checksum) + util.vprint("\tnewpkg.mtime : %s" % newpkg.mtime) if case == "EXIST": success = 1 elif case == "MODIFIED": @@ -124,8 +125,8 @@ class pmrule: if not os.path.isfile(filename): success = 0 else: - mode = os.lstat(filename)[ST_MODE] - if int(value,8) != S_IMODE(mode): + mode = os.lstat(filename)[stat.ST_MODE] + if int(value, 8) != stat.S_IMODE(mode): success = 0 elif case == "TYPE": if value == "dir": @@ -138,13 +139,13 @@ class pmrule: if not os.path.islink(filename): success = 0 elif case == "PACNEW": - if not os.path.isfile("%s%s" % (filename, PM_PACNEW)): + if not os.path.isfile("%s.pacnew" % filename): success = 0 elif case == "PACORIG": - if not os.path.isfile("%s%s" % (filename, PM_PACORIG)): + if not os.path.isfile("%s.pacorig" % filename): success = 0 elif case == "PACSAVE": - if not os.path.isfile("%s%s" % (filename, PM_PACSAVE)): + if not os.path.isfile("%s.pacsave" % filename): success = 0 else: print "FILE rule '%s' not found" % case diff --git a/test/pacman/util.py b/test/pacman/util.py index f77da52b..be155e95 100755 --- a/test/pacman/util.py +++ b/test/pacman/util.py @@ -29,9 +29,6 @@ PM_SYNCDBPATH = "var/lib/pacman/sync" PM_LOCK = "var/lib/pacman/db.lck" PM_CACHEDIR = "var/cache/pacman/pkg" PM_EXT_PKG = ".pkg.tar.gz" -PM_PACNEW = ".pacnew" -PM_PACORIG = ".pacorig" -PM_PACSAVE = ".pacsave" # Pacman PACCONF = "etc/pacman.conf" |