Fix have_enqueued_sidekiq_job timezone problem

wspurgin / rspec-sidekiq

RSpec for Sidekiq

https://github.com/wspurgin/rspec-sidekiq

Other

663 stars 132 forks source link

Fix have_enqueued_sidekiq_job timezone problem #146

Closed a0s closed 4 years ago

a0s commented 7 years ago

It was bad idea to compare string representation's of times without timezone considering. This should be fix https://github.com/philostler/rspec-sidekiq/issues/133

coveralls commented 7 years ago

Coverage remained the same at 79.501% when pulling af4e228dae122886f38e78bf4076418f50299c5d on a0s:patch-1 into 2cd15b0fe2b172243e8002c4aefa39696c42d52f on philostler:develop.

packrat386 commented 4 years ago

Sorry this took to so long to get to. #153 has been pulled into the latest release (3.1.0), which should address this issue. Gonna close this PR for now, but feel free to re-open if you're still seeing problems.