Quick pactest changes:

* Minor change to utils.py(grep) to make it cleaner
* Added scriptlet check to verify scriptlet events happen

2 files changed, 20 insertions, 9 deletions

diff --git a/pactest/tests/scriptlet001.py b/pactest/tests/scriptlet001.py
new file mode 100644
index 00000000..a749a114
--- /dev/null
+++ b/pactest/tests/scriptlet001.py
@@ -0,0 +1,15 @@
+# quick note here - chroot() is expected to fail.  We're not checking the
+# validity of the scripts, only that they fire (or try to)
+self.description = "Scriptlet test (pre/post install)"
+
+p1 = pmpkg("dummy")
+p1.files = ['etc/dummy.conf']
+p1.install['pre_install'] = "ls /etc";
+p1.install['post_install'] = "ls /etc";
+self.addpkg(p1)
+
+self.args = "-U %s" % p1.filename()
+
+self.addrule("PACMAN_RETCODE=0")
+self.addrule("PACMAN_OUTPUT=pre_install")
+self.addrule("PACMAN_OUTPUT=post_install")
diff --git a/pactest/util.py b/pactest/util.py
index e445605e..574a59be 100755
--- a/pactest/util.py
+++ b/pactest/util.py
@@ -236,16 +236,12 @@ def diffmtime(mt1, mt2):
 #
 
 def grep(filename, pattern):
-    found = 0
-    fd = file(filename, "r")
-    while 1 and not found:
-        line = fd.readline()
-        if not line:
-            break
+    lines = file(filename, 'r').readlines()
+    for line in lines:
+        if not line: break
         if line.find(pattern) != -1:
-            found = 1
-    fd.close()
-    return found
+            return True
+    return False
 
 def mkdir(dir):
     if os.path.isdir(dir):