toptal / chewy

High-level Elasticsearch Ruby framework based on the official elasticsearch-ruby client
MIT License
1.88k stars 366 forks source link

Fix id conversion issue in delayed_sidekiq strategy #964

Open sundus-y opened 2 weeks ago

sundus-y commented 2 weeks ago

Ensure ids extracted from Redis remain strings, preventing UUID issues. Previously, ids were being converted to integers, causing problems with UUIDs in the delayed_sidekiq strategy.

This update also enhances the test suite:

Due to SQLite's lack of UUID support, a stub_uuid_model method is added. This method stubs models with UUIDs, using SecureRandom.uuid for the primary key.


Before submitting the PR make sure the following are checked: