Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Register non ActiveRecord::Base classes correctly #19

Merged
merged 2 commits into from
Mar 20, 2013
Merged

Register non ActiveRecord::Base classes correctly #19

merged 2 commits into from
Mar 20, 2013

Conversation

mptre
Copy link
Contributor

@mptre mptre commented Mar 19, 2013

After upgrading to v4.7.1 my before and after hooks stopped working while running my specs for all classes that didn't inherit from ActiveRecord::Base (classes stored in lib for instance).

I assume there's a valid reason for the ActiveRecord::Base inherit check?

@metaskills
Copy link
Owner

I assume there's a valid reason for the ActiveRecord::Base inherit check?

Now that I look at it, I am not sure why that is there. I certainly like your test added and it should be allowed. BTW, that require_relative will have to go for the test to pass in 1.8.7.

@@ -0,0 +1,16 @@
require 'test_helper_dummy'
require_relative '../dummy_app/lib/library'
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could be require "#{Dummy::Application.root}/lib/library"

metaskills added a commit that referenced this pull request Mar 20, 2013
Register non ActiveRecord::Base classes correctly
@metaskills metaskills merged commit 7b4383e into metaskills:master Mar 20, 2013
@metaskills
Copy link
Owner

Released v4.7.2. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants