Tests not running on a clean checkout or unpacked gem about recaptcha

[schmidt]

This is especially important, since I wanted to test may changes in http://github.com/finnlabs/recaptcha/commit/96e237ecb60414ce27252f9a272e10951f323eab

[ambethia]

That's interesting, there should be no problem requiring rails/version if rubygems is loaded, and the rails gem installed. I do know that some of the tests are broken in the master branch with the introduction of using flash instead of the session to pass the error message back to the user.

[schmidt]

Thanks for the hint. I digged a little deeper. It seems to be related to 1.8.7 - when using 1.8.6 the tests are run successfully.

I will try to find a workaround.

[schmidt]

Why is the

require 'rails/version'

actually needed. The tests run fine without it. Also, I cannot find any reference to Rails::VERSION in the source of Recaptcha.

[ambethia]

It probably referenced at some point, but got factored out of the tests. I'm planning on putting some time aside in the coming days to cleanup the tests and fix up the code in preparation for rails 3.

[schmidt]

Okay, with your help, I figured it out

• http://github.com/finnlabs/recaptcha/commit/ebf84c4 allows me to run the tests on 1.8.7
• http://github.com/finnlabs/recaptcha/commit/98524d3 updates the tests to use flash instead of session
• http://github.com/finnlabs/recaptcha/commit/4e03219 updates an expectation to the current wording in the lib

[schmidt]

Please also consider integrating my changes to the configuration options before starting any refactoring. http://github.com/finnlabs/recaptcha/commit/96e237ecb60414ce27252f9a272e10951f323eab

[jjulian]

+1 for the schmidt configuration changes! Setting the ENV is currently not easy when you have multiple environments and multiple developers running under non-localhost domains.