Fix test-installed-snaps.js by ensuring sudo is set to true for insta…

…ll or remove operations (#150)
canonical · Feb 7, 2017 · f97ee18 · f97ee18
1 parent e6499d0
commit f97ee18
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/tests/run-tests.sh b/tests/run-tests.sh
@@ -58,7 +58,7 @@ if [ ! -d "$modules_dir" ]; then
   modules_dir="../node_modules"
 fi
 
-echo 'Run selenium tests for snapweb service on given IP'
+echo "Run selenium tests for snapweb on $host:$port with user:$user/$sudo"
 USER=$user HOST=$host PORT=$port SUDO=$sudo $modules_dir/.bin/wdio -b https://$2:4201
 exitcode=$?
 

diff --git a/tests/utils/snap-cmd-util.js b/tests/utils/snap-cmd-util.js
@@ -103,11 +103,11 @@ SnapUtil.prototype.getToken = function() {
 }
 
 SnapUtil.prototype.installSnap = function(name) {
-    return this._promisifyExec('snap install ' + name, this.isSudoRequired);
+  return this._promisifyExec('snap install ' + name, true);
 }
 
 SnapUtil.prototype.removeSnap = function(name) {
-    return this._promisifyExec('snap remove ' + name, this.isSudoRequired);
+  return this._promisifyExec('snap remove ' + name, true);
 }
 
 SnapUtil.prototype.snapVersion = function() {