summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoss Burton <ross.burton@arm.com>2023-07-05 11:50:01 +0100
committerRichard Purdie <richard.purdie@linuxfoundation.org>2023-07-10 11:36:34 +0100
commit6d01e5f369e2f12a264ca613a1f165bec05424d9 (patch)
tree0d688d5a3dd40ba1de75860d8164cb2ac6554713
parent90cc6af0ba0f4f4cf54821875f11c4139bc5405a (diff)
downloadpoky-6d01e5f369e2f12a264ca613a1f165bec05424d9.tar.gz
oeqa/runtime/cases/rpm: fix wait_for_no_process_for_user failure case
str.format() doesn't use % notation, update the formatting to work. assertTrue() is a member of self not a global, and assertTrue(True) will always pass. Change this to just self.fail() as this is the failure case. (From OE-Core rev: 017f3a0b1265c1a3b69c20bdb56bbf446111977e) Signed-off-by: Ross Burton <ross.burton@arm.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r--meta/lib/oeqa/runtime/cases/rpm.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/meta/lib/oeqa/runtime/cases/rpm.py b/meta/lib/oeqa/runtime/cases/rpm.py
index fa86eb0537..a4ba4e6769 100644
--- a/meta/lib/oeqa/runtime/cases/rpm.py
+++ b/meta/lib/oeqa/runtime/cases/rpm.py
@@ -59,8 +59,8 @@ class RpmBasicTest(OERuntimeTestCase):
59 return 59 return
60 time.sleep(1) 60 time.sleep(1)
61 user_pss = [ps for ps in output.split("\n") if u + ' ' in ps] 61 user_pss = [ps for ps in output.split("\n") if u + ' ' in ps]
62 msg = "There're %s 's process(es) still running: %s".format(u, "\n".join(user_pss)) 62 msg = "User %s has processes still running: %s" % (u, "\n".join(user_pss))
63 assertTrue(True, msg=msg) 63 self.fail(msg=msg)
64 64
65 def unset_up_test_user(u): 65 def unset_up_test_user(u):
66 # ensure no test1 process in running 66 # ensure no test1 process in running