shadabahmed/logstasher — pull request #57 by torrancew (2014-11-11 18:57:25+00:00)
Title: Prove that timestamps are not working properly in test suite

This commit uncovers the same test failure that's present in #54, only without the features of that PR. Effectively, it proves that the string that was mocking timestamp was covering up strange behavior. To make matters even stranger, this behavior does not seem to follow through to actual Rails applications in either, Rails 3 or 4 - timestamps have a subsecond precision in both. This is looking an awful lot like some type of test pollution or other form of environment-based cause.

There is no need to merge this PR, but I feel it should be brought to the attention of the developers, given that it was cited as a blocking factor for merging #54. In the mean time, I will provide a new commit for #54, that expects one timestamp in Rails 3, and another in Rails 4 - I think the proper solution is to remove this strange behavior (possibly an errant monkey patch somewhere in the Rails dependency trees?), but I have not been successful in that approach.