Readme: Testing with rspec. #12

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

sadfuzzy commented May 23, 2012

No description provided.

Owner

ledermann commented May 23, 2012

Hm, thanks for your work, but I don't like the idea of testing gem features in the application. The gem already includes some tests, so there is no need to test it in the application again.

@ledermann ledermann closed this May 23, 2012

Contributor

sadfuzzy commented May 23, 2012

The idea is not to test the Gem, but to overcome the bug of not created/assumed read marks.
When you create factories with dependent model, that acts_as_unread, the read marks are not created if gap between created_at times is less than 0.002sec (if i'm not mistaken).

This fix helps to figure this out and make your test pass every time you run it.

Contributor

sadfuzzy commented May 24, 2012

Do you agree?

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