Fix broken tests, add sleep to allow container to die after kill signal

upserve · Jul 8, 2024 · b6d8e81 · b6d8e81
1 parent ad537ed
commit b6d8e81
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/spec/docker/container_spec.rb b/spec/docker/container_spec.rb
@@ -619,6 +619,7 @@
 
     it 'kills the container' do
       subject.kill
+      sleep(1)
       expect(described_class.all.map(&:id)).to be_none { |id|
         id.start_with?(subject.id)
       }
@@ -637,6 +638,7 @@
       }
       it 'kills the container' do
         subject.kill(:signal => "SIGTERM")
+        sleep(1)
         expect(described_class.all.map(&:id)).to be_any { |id|
           id.start_with?(subject.id)
         }
@@ -645,6 +647,7 @@
         }
 
         subject.kill(:signal => "SIGKILL")
+        sleep(1)
         expect(described_class.all.map(&:id)).to be_none { |id|
           id.start_with?(subject.id)
         }

diff --git a/spec/docker/image_spec.rb b/spec/docker/image_spec.rb
@@ -643,14 +643,7 @@
 
         expect(image.id).to_not be_nil
 
-        expected = [
-          'Created',
-          'Size'
-        ]
-
-        expected << 'VirtualSize' unless ::Docker.podman?
-
-        expected.each do |key|
+        %w(Created Size).each do |key|
           expect(image.info).to have_key(key)
         end
       end