summaryrefslogtreecommitdiffstats
path: root/features
diff options
context:
space:
mode:
authorintrigeri <intrigeri@boum.org>2019-03-05 10:30:11 +0000
committerintrigeri <intrigeri@boum.org>2019-03-05 10:30:11 +0000
commit886fc3b52337e82ead2b8dd26b86c6e6da21a2f5 (patch)
tree9517fc8eccb31f58071703e8fd0180af4b69d273 /features
parent399f80f2179e81409c3646139fab03f0f76a57ed (diff)
Test suite: fix buggy checks (refs: #14596)
They were previously no-ops as we did not act based on their return value.
Diffstat (limited to 'features')
-rw-r--r--features/step_definitions/additional_software_packages.rb4
1 files changed, 2 insertions, 2 deletions
diff --git a/features/step_definitions/additional_software_packages.rb b/features/step_definitions/additional_software_packages.rb
index d6b0e0d..c3a87ee 100644
--- a/features/step_definitions/additional_software_packages.rb
+++ b/features/step_definitions/additional_software_packages.rb
@@ -50,8 +50,8 @@ end
Then /^Additional Software is correctly configured for package "([^"]*)"$/ do |package|
assert($vm.file_exist?(ASP_CONF), "ASP configuration file not found")
step 'all persistence configuration files have safe access rights'
- $vm.execute("ls /live/persistence/TailsData_unlocked/apt/cache/#{package}_*.deb").success?
- $vm.execute("ls /live/persistence/TailsData_unlocked/apt/lists/*_Packages").success?
+ $vm.execute_successfully("ls /live/persistence/TailsData_unlocked/apt/cache/#{package}_*.deb")
+ $vm.execute_successfully("ls /live/persistence/TailsData_unlocked/apt/lists/*_Packages")
try_for(30) do
$vm.execute("grep --line-regexp --fixed-strings #{package} #{ASP_CONF}").success?
end