fixxx

degica · Apr 1, 2024 · 4609d82 · 4609d82
1 parent 513c2fd
commit 4609d82
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/lib/kaiser/cli.rb b/lib/kaiser/cli.rb
@@ -598,7 +598,7 @@ def network
 
     def container_dead?(container)
       x = JSON.parse(`docker inspect #{container} 2>/dev/null`)
-      return true if x.empty? || x[0]['State']['Running'] == false
+      x.empty? || x[0]['State']['Running'] == false
     end
 
     def if_container_dead(container)

diff --git a/lib/kaiser/cmds/shutdown.rb b/lib/kaiser/cmds/shutdown.rb
@@ -14,7 +14,7 @@ def usage
       end
 
       def execute(_opts)
-        Config.config[:shared_names].each do |_, container_name|
+        Config.config[:shared_names].each_value do |container_name|
           killrm container_name
         end
 

diff --git a/spec/cli_spec.rb b/spec/cli_spec.rb
@@ -51,19 +51,19 @@
     it 'for x86 machines returns normal selenium' do
       cli = Kaiser::Cli.new
       stub_const('RUBY_PLATFORM', 'x86_64-linux')
-      expect(cli.selenium_node_image).to eq('selenium/standalone-chrome-debug')
+      expect(cli.send(:selenium_node_image)).to eq('selenium/standalone-chrome-debug')
     end
 
     it 'for arm machines on mac' do
       cli = Kaiser::Cli.new
       stub_const('RUBY_PLATFORM', 'arm64-darwin23')
-      expect(cli.selenium_node_image).to eq('seleniarm/standalone-chromium')
+      expect(cli.send(:selenium_node_image)).to eq('seleniarm/standalone-chromium')
     end
 
     it 'for arm machines on linux' do
       cli = Kaiser::Cli.new
       stub_const('RUBY_PLATFORM', 'arch64-linux')
-      expect(cli.selenium_node_image).to eq('seleniarm/standalone-chromium')
+      expect(cli.send(:selenium_node_image)).to eq('seleniarm/standalone-chromium')
     end
   end
 end