Added support for error callbacks

Author: Kumanan Rajamanikkam

lib/swagger-spec.js

@@ -239,16 +239,19 @@
     });
     describe("constructor", function() {
       beforeEach(function() {
-        var body, callback, headers, operation;
+        var body, callback, error, headers, operation;
         headers = {
           'mock': 'true'
         };
         body = null;
         callback = function() {
           return 'mock callback';
         };
+        error = function() {
+          return 'mock error';
+        };
         operation = wordnik2.word.operations.getExamples;
-        return window.request = new SwaggerRequest("GET", "http://google.com", headers, body, callback, operation);
+        return window.request = new SwaggerRequest("GET", "http://google.com", headers, body, callback, error, operation);
       });
       return it("sticks the API key into the headers, if present in the parent Api configuration", function() {
         return runs(function() {

lib/swagger.js

@@ -199,8 +199,10 @@ describe 'SwaggerRequest', ->
       body = null
       callback = ->
         'mock callback'
+      error = ->
+          'mock error'
       operation = wordnik2.word.operations.getExamples
-      window.request = new SwaggerRequest("GET", "http://google.com", headers, body, callback, operation)
+      window.request = new SwaggerRequest("GET", "http://google.com", headers, body, callback, error, operation)
 
     it "sticks the API key into the headers, if present in the parent Api configuration", ->
       runs ->
@@ -235,4 +237,36 @@ describe 'SwaggerRequest', ->
       window.user?
 
     runs ->
-      expect(window.user.username).toBe("kareem#{random}")
+      expect(window.user.username).toBe("kareem#{random}")
+      
+  # it "supports PUT requests", ->
+  #       window.chorus = new SwaggerApi
+  #         discoveryUrl: "http://chorus-dev.nik.io/api/resources.json"
+  #         success: ->
+  # 
+  #             # Use a random suffix to pass uniqueness validations.
+  #             window.random = Math.floor(Math.random()*1000000)
+  #                 
+  #             siteParts = 
+  #               name: name
+  #               url : url
+  #               description : desc
+  # 
+  #             window.chorus.sites.createSite { body: siteParts }, (site) ->
+  #               updateSiteParts = 
+  #                 id: site.id 
+  #                 partnerId: site.partnerId
+  #                 name: site.name
+  #                 url : site.url
+  #                 settings: 
+  #                   interBlogRecommendations: true
+  #                   intraBlogRecommendations: false
+  #                         
+  #               window.chorus.sites.updateSite {body: updateSiteParts}, (response) ->
+  #                 window.site = response
+  # 
+  #       waitsFor ->
+  #         window.site?
+  # 
+  #       runs ->
+  #         expect(window.site.interBlogRecommendations).toBe(true)

src/swagger-spec.coffee

@@ -95,10 +95,19 @@ class SwaggerOperation
 
     # Store a named reference to this operation on the parent resource
     # getDefinitions() maps to getDefinitionsData.do()
-    @resource[@nickname]= (args, callback) =>
-      @do(args, callback)
+    @resource[@nickname]= (args, callback, error) =>
+      @do(args, callback, error)
 
-  do: (args={}, callback = (data) -> console.log(data) ) =>
+  do: (args={}, callback, error) =>
+  
+    # Define a default error handler
+    unless error?
+      error = (xhr, textStatus, error) -> console.log xhr, textStatus, error
+
+    # Define a default success handler
+    # TODO MAYBE: Call this success instead of callback
+    unless callback?
+      callback = (date) -> console.log data
 
     # Pull headers out of args    
     if args.headers?
@@ -110,7 +119,7 @@ class SwaggerOperation
       body = args.body
       delete args.body
 
-    new SwaggerRequest(@httpMethod, @urlify(args), headers, body, callback, this)
+    new SwaggerRequest(@httpMethod, @urlify(args), headers, body, callback, error, this)
 
   urlify: (args) ->
 
@@ -142,10 +151,11 @@ class SwaggerOperation
 
 class SwaggerRequest
 
-  constructor: (@type, @url, @headers, @body, @callback, @operation) ->
+  constructor: (@type, @url, @headers, @body, @callback, @error, @operation) ->
     throw "SwaggerRequest type is required (get/post/put/delete)." unless @type?
     throw "SwaggerRequest url is required." unless @url?
     throw "SwaggerRequest callback is required." unless @callback?
+    throw "SwaggerRequest error callback is required." unless @error?
     throw "SwaggerRequest operation is required." unless @operation?
 
     # console.log "new SwaggerRequest: %o", this
@@ -165,7 +175,7 @@ class SwaggerRequest
         data: JSON.stringify(@body)
         dataType: 'json'
         error: (xhr, textStatus, error) ->
-          console.log xhr, textStatus, error
+          @error(xhr, textStatus, error)
         success: (data) =>
           @callback(data)