<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On 8 September 2016 at 02:26, Edwin Plauchu <span dir="ltr">&lt;<a href="mailto:edwin.plauchu.camacho@linux.intel.com" target="_blank">edwin.plauchu.camacho@linux.intel.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":5uj" class="a3s aXjCH m157076699ff3e4f4">     def test_rpm_query_nonroot(self):<br>
-        (status, output) = self.target.run(&#39;useradd test1&#39;)<br>
+        test_user = &#39;test1&#39;<br>
+        (status, output) = self.target.run(&quot;useradd %s&quot; % test_user)<br>
         self.assertTrue(status == 0, msg=&quot;Failed to create new user: &quot; + output)<br>
-        (status, output) = self.target.run(&#39;su -c id test1&#39;)<br>
+        (status, output) = self.target.run(&quot;su -c id %s&quot; % test_user)<br>
         self.assertTrue(&#39;(test1)&#39; in output, msg=&quot;Failed to execute as new user&quot;)<br>
-        (status, output) = self.target.run(&#39;su -c &quot;rpm -qa&quot; test1 &#39;)<br>
+        (status, output) = self.target.run(&quot;su -c \&quot;rpm -qa\&quot; %s &quot; % test_user)<br>
         self.assertEqual(status, 0, msg=&quot;status: %s. Cannot run rpm -qa: %s&quot; % (status, output))<br>
+        (status, output) = self.target.run(&quot;userdel -r %s&quot; % test_user)<br>
+        self.assertTrue(status == 0, msg=&quot;Failed to erase user: %s&quot; % output)</div></blockquote></div><br>The user removal should be in a try/finally block around the test so that it always executes even if the test fails.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Ross</div></div>