notahat / machinist

Fixtures aren't fun. Machinist is.
MIT License
1.12k stars 134 forks source link

Updated Titanous' DataMapper support to work on current 2.0 master #79

Closed emmanuel closed 13 years ago

emmanuel commented 13 years ago

Hi,

I've updated @titanous fork which implements DataMapper support. I issued a pull request to @titanous, but haven't heard anything back (it's been a while), so I'm offering these changes directly back to you.

On top of the DataMapper support, I switch the default testing DB from MySQL to SQLite so that tests can be run with no external dependencies.

Finally, I moved the ActiveRecord models in the specs into an AR module, for clarity and to ensure no collisions with models from other ORMs (the DataMapper spec models are namespaced in a DM module).

Thanks for Machinist!

emmanuel commented 13 years ago

Well, looks like @d11wtq beat me to the punch with notahat/machinist#85, so I'm going to rebase this work on top of his to separate out my 'extra' changes from the DataMapper support.