Permalink
Browse files

adding a test for clearing stale connections

  • Loading branch information...
1 parent 7ba54ff commit 8d17f533187a791b4a42f45542c56c5f965e6fad @tenderlove tenderlove committed Jul 16, 2010
Showing with 31 additions and 0 deletions.
  1. +31 −0 activerecord/test/cases/connection_pool_test.rb
@@ -0,0 +1,31 @@
+require "cases/helper"
+
+module ActiveRecord
+ module ConnectionAdapters
+ class ConnectionPoolTest < ActiveRecord::TestCase
+ def test_clear_stale_cached_connections!
+ pool = ConnectionPool.new ActiveRecord::Base.connection_pool.spec
+
+ threads = [
+ Thread.new { pool.connection },
+ Thread.new { pool.connection }]
+
+ threads.map { |t| t.join }
+
+ pool.extend Module.new {
+ attr_accessor :checkins
+ def checkin conn
+ @checkins << conn
+ conn.object_id
+ end
+ }
+ pool.checkins = []
+
+ cleared_threads = pool.clear_stale_cached_connections!
+ assert((cleared_threads - threads.map { |x| x.object_id }).empty?,
+ "threads should have been removed")
+ assert_equal pool.checkins.length, threads.length
+ end
+ end
+ end
+end

0 comments on commit 8d17f53

Please sign in to comment.