Skip to content
This repository was archived by the owner on Jan 27, 2021. It is now read-only.

Conversation

@lcoq
Copy link

@lcoq lcoq commented Jun 2, 2015

The test does not fail but starts an infinite loop which updates the persons updated_at where id is NULL

@bobbus
Copy link

bobbus commented Jun 18, 2015

Encouter this one too. Would you accept a PR trying to fix this bug ?

@mtuckergh
Copy link

HI @Icoq and @bobbus. Apologies for not responding sooner. We absolutely do accept PRs fixing issues. My only feedback here is that the Spec should also be finite, meaning it can check something like expect(something).to receive(:some_method).exactly(n).times. I'm sure the spec included here is just to demonstrate that an infinite loop is occurring, but wanted to call this out just to make sure we're on the same page.
Appreciate any help on tackling this!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants