balvig / utensils

Rspec stuff we use over and over again
MIT License
11 stars 9 forks source link

Set ActiveRecord strategy explicitly #4

Closed kinopyo closed 8 years ago

kinopyo commented 8 years ago

Seems like there're issues with El Capitan that doesn't cleanup the mongoid unless we explicitly set the strategy for it

kinopyo commented 8 years ago

oh.. @balvig do you think it's better to separate it to another file like database_cleaner/mongoid.rb..? wondering if he "all in one file" would raise some error if the project doesn't have mongoid storage :sweat_smile:

balvig commented 8 years ago

Thank you @kinopyo

do you think it's better to separate it to another file

Hmmm, could you maybe try it out in a non-mongoid project? :bow:

kinopyo commented 8 years ago

unfortunately keeping in same file would cause error for non mongoid project :sweat: , moved to another file

:recycle:

kinopyo commented 8 years ago

Update: removed the mongoid setting part but keep the active_record strategy specific, best way to keep it compatible with other projects :smile:

balvig commented 8 years ago

Thanks @kinopyo merged and released 1.4.3

kinopyo commented 8 years ago

:clap: