Real concurrency spec for ActiveRecord

[marshall-lee]

This one should fail. It should prove that current implementation is not actually concurrent. It ignores :pool ActiveRecord parameter making ConnectionPool useless.

[marshall-lee]

It fails as expected. RSpec outputs just what i say in #190:

 Failure/Error: expect(Widget.connection_pool.connections.size).to eq(10)

   expected: 10
        got: 1

[marshall-lee]

It happens because in current implementation current_connection_id is still Thread.current.object_id. Now lets try to change it to Fiber.current.object_id

[marshall-lee]

Not it fails too but for another reason:

 ActiveRecord::ConnectionTimeoutError:
   could not obtain a database connection within 5.000 seconds (waited 5.000 seconds)

So current connection pool just sucks

[marshall-lee]

Sorry, it works. I just realized that ActiveRecord's connection pool contains only one connection but this connection is actually a pool itself.