-
Notifications
You must be signed in to change notification settings - Fork 33
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
Fixtures #7
Comments
Right now the closest thing is
This should make up for many of the features a fixture adapter might have, however I would like to give this more thought and come up with an official story. I will keep this issue open to track. |
+1 I'd like to try epf instead of Ember Data, but I need to be able to use Fixtures. |
Just leaving a note here for future googlers. The // works
session.merge(App.Post.create({id: '1', title: 'testing'}));
// doesn't work
session.merge(App.Post.create({id: 1, title: 'testing'})); Perhaps this should be documented - or I actually feel that a numeric session.load(App.Post, 1); // works with numeric argument! The behind-the-scenes coercion of |
@heartsentwined I agree w/ that. We should coerce during merge. Could you open up a separate issue? |
New issue over at #27 |
Just as a side note for people looking into testing, I am working on tests for an EPF project and am quite successfully using sinonjs with the normal rest adapter for testing. IMO this could ultimately produce a smoother transition from a local solution to a real backend (I have heard horror stories about people transitioning from ED's fixture adapter to a real rest adapter). |
@ghempton : I can't bring the |
@IGuider I am having the same issue, did you manage to resolve it? |
@anthillape no, I returned to ember-data |
As I said in my previous comment, I would recommend using sinon.js rather than Ep.LocalAdapter. I would give the same recommendation over ED's fixture adapter as well. |
Will it have FixtureAdapter for testing?
The text was updated successfully, but these errors were encountered: