Merge pull request #1293 from supercaracal/fix-a-test

redis · Sep 9, 2024 · 9938411 · 9938411
2 parents 6afa62b + a46f9db
commit 9938411
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -57,6 +57,7 @@ jobs:
         run: make start
       - name: Test
         run: bundle exec rake test:redis test:distributed
+        timeout-minutes: 3
       - name: Shutting down Redis
         run: make stop
 

diff --git a/test/redis/ssl_test.rb b/test/redis/ssl_test.rb
@@ -16,6 +16,7 @@ def test_verified_ssl_connection
     RedisMock.start({ ping: proc { "+PONG" } }, ssl_server_opts("trusted")) do |port|
       redis = Redis.new(host: "127.0.0.1", port: port, ssl: true, ssl_params: { ca_file: ssl_ca_file })
       assert_equal redis.ping, "PONG"
+      redis.close
     end
   end
 
@@ -41,6 +42,7 @@ def test_ssl_blocking
     RedisMock.start({}, ssl_server_opts("trusted")) do |port|
       redis = Redis.new(host: "127.0.0.1", port: port, ssl: true, ssl_params: { ca_file: ssl_ca_file })
       assert_equal redis.set("boom", "a" * 10_000_000), "OK"
+      redis.close
     end
   end