diff options
| author | intrigeri <intrigeri@boum.org> | 2019-03-05 10:30:11 +0000 |
|---|---|---|
| committer | intrigeri <intrigeri@boum.org> | 2019-03-05 10:30:11 +0000 |
| commit | 886fc3b52337e82ead2b8dd26b86c6e6da21a2f5 (patch) | |
| tree | 9517fc8eccb31f58071703e8fd0180af4b69d273 | |
| parent | 399f80f2179e81409c3646139fab03f0f76a57ed (diff) | |
Test suite: fix buggy checks (refs: #14596)
They were previously no-ops as we did not act based on their return value.
| -rw-r--r-- | features/step_definitions/additional_software_packages.rb | 4 |
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 |