Calling didCreateModel with a hash should update the data property (a…

…nd therefore also invalidate any DS.attrs).
joewest · Dec 31, 2011 · f548be3 · f548be3
1 parent 9cdeaa9
commit f548be3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/ember-data/lib/system/store.js b/packages/ember-data/lib/system/store.js
@@ -474,7 +474,7 @@ DS.Store = SC.Object.extend({
       clientId = get(model, 'clientId');
 
       data[clientId] = hash;
-      model.set('data', hash);
+      set(model, 'data', hash);
 
       idToClientIdMap[id] = clientId;
       idList.push(id);
@@ -496,6 +496,7 @@ DS.Store = SC.Object.extend({
 
     clientId = get(model, 'clientId');
     data[clientId] = hash;
+    set(model, 'data', hash);
 
     idToClientIdMap[id] = clientId;
     idList.push(id);

diff --git a/packages/ember-data/tests/adapter_test.js b/packages/ember-data/tests/adapter_test.js
@@ -313,7 +313,7 @@ test("by default, commit calls deleteMany once per type", function() {
 });
 
 test("by default, createMany calls create once per record", function() {
-  expect(6);
+  expect(8);
   var count = 1;
 
   adapter.create = function(store, type, model) {
@@ -329,8 +329,11 @@ test("by default, createMany calls create once per record", function() {
 
     var hash = get(model, 'data');
     hash.id = count;
+    hash.updatedAt = "now";
 
     store.didCreateModel(model, hash);
+    equal(get(model, 'updatedAt'), "now", "the model should receive the new information");
+
     count++;
   };