danielestevez commented on this pull request.


> @@ -103,7 +103,7 @@ public void simpleRootTestWithSudoPassword() {
       expect(sshClient.getUsername()).andReturn("tester");
       expect(sshClient.getHostAddress()).andReturn("somewhere.example.com");
       expect(
-            sshClient.exec("sudo -S sh <<'RUN_SCRIPT_AS_ROOT_SSH'\n" + 
"testpassword!\n" + "echo $USER\n"
+            sshClient.exec("sudo -S sh <<'RUN_SCRIPT_AS_ROOT_SSH'\n" + 
"'testpassword!'\n" + "echo $USER\n"

@demobox 
That line of code 
https://github.com/jclouds/jclouds/pull/1205#discussion-diff-187774732R118 is 
already covered in simpleRootTest and testUserAddAsRoot tests

This PR adds a conflictive password for all tests 
https://github.com/jclouds/jclouds/pull/1207


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1205#discussion_r188094382

Reply via email to