Rest persistence should send JSON encoded data.

gwak · Sep 15, 2010 · 6d8d9cf · 6d8d9cf
1 parent 0f81b7e
commit 6d8d9cf
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 3 deletions.
diff --git a/src/model_rest_persistence.js b/src/model_rest_persistence.js
@@ -46,7 +46,7 @@ Model.RestPersistence = function(resource, methods) {
       } else {
         params = null;
       }
-      return params;
+      return JSON.stringify(params)
     },
 
     read: function(callback) {
@@ -84,6 +84,7 @@ Model.RestPersistence = function(resource, methods) {
       return jQuery.ajax({
         type: method,
         url: url,
+        contentType: "application/json",
         dataType: "json",
         data: data,
         dataFilter: function(data, type) {

diff --git a/test/public/javascripts/json.js b/test/public/javascripts/json.js
diff --git a/test/tests/model_rest_persistence.js b/test/tests/model_rest_persistence.js
@@ -125,7 +125,7 @@ test("create", function() {
 
   equals(request.type, "POST");
   equals(request.url, "/posts");
-  same(request.data, { post: { title: "Foo", body: "..." } });
+  same(JSON.parse(request.data), { post: { title: "Foo", body: "..." } });
 });
 
 test("create - 422 response (failed validations)", function() {
@@ -192,7 +192,7 @@ test("update", function() {
 
   equals(request.type, "PUT");
   equals(request.url, "/posts/1");
-  same(request.data, { post: { title: "Bar", body: "..." } });
+  same(JSON.parse(request.data), { post: { title: "Bar", body: "..." } });
 });
 
 test("update - blank response (Rails' `head :ok`)", function() {

diff --git a/test/views/index.erb b/test/views/index.erb
@@ -8,6 +8,7 @@
 
 <!-- Dependencies -->
 <script src="javascripts/jquery-1.4.1.min.js"></script>
+<script src="javascripts/json.js"></script>
 
 <!-- js-model -->
 <script src="/js-model.js"></script>