feat: add attribute invoice refund order as per latest api

Author: rahulbile

lib/models/invoice.js

@@ -63,25 +63,30 @@ class Invoice extends RecurlyData {
 
   fetchPDF(callback) {
     if (!this.href) {
-      throw (new Error('cannot fetch a record without an href'))
+      throw new Error('cannot fetch a record without an href')
     }
 
-    this.get(this.href, { }, {
-      headers: {
-        Accept: 'application/pdf'
+    this.get(
+      this.href,
+      {},
+      {
+        headers: {
+          Accept: 'application/pdf'
+        },
+        encoding: null,
+        noParse: true
       },
-      encoding: null,
-      noParse: true
-    }, (err, response, payload) => {
-      if (err) {
-        return callback(err)
+      (err, response, payload) => {
+        if (err) {
+          return callback(err)
+        }
+        if (response.statusCode === 404) {
+          return callback(new Error('not_found'))
+        }
+
+        callback(err, payload)
       }
-      if (response.statusCode === 404) {
-        return callback(new Error('not_found'))
-      }
-
-      callback(err, payload)
-    })
+    )
   }
 
   refund(options, callback) {
@@ -90,16 +95,20 @@ class Invoice extends RecurlyData {
       options = null
     }
 
-    options = _.defaults({
-      amount_in_cents: undefined,
-      refund_apply_order: undefined
-    }, options, { refund_apply_order: 'credit' })
+    options = _.defaults(
+      {
+        amount_in_cents: undefined,
+        refund_method: undefined
+      },
+      options,
+      { refund_method: 'credit_first' }
+    )
 
     debug('Got options as %o', options)
 
     let uri = _.get(this, 'a.refund.href')
     if (!uri && !this.invoice_number) {
-      throw (new Error('cannot refund an invoice without an invoice_number'))
+      throw new Error('cannot refund an invoice without an invoice_number')
     }
 
     uri = uri || `${Invoice.ENDPOINT}/${this.invoice_number}/refund`
@@ -108,7 +117,7 @@ class Invoice extends RecurlyData {
     debug('Calling refund uri %s with body %o', uri, body)
 
     this.post(uri, body, (err, response, payload) => {
-      const error = handleRecurlyError(err, response, payload, [ 201 ])
+      const error = handleRecurlyError(err, response, payload, [201])
       if (error) {
         return callback(error)
       }
@@ -120,13 +129,15 @@ class Invoice extends RecurlyData {
   markSuccessful(callback) {
     let uri = _.get(this, 'a.mark_successful.href')
     if (!uri && !this.invoice_number) {
-      throw (new Error('cannot mark invoice as successful without an invoice_number'))
+      throw new Error(
+        'cannot mark invoice as successful without an invoice_number'
+      )
     }
 
     uri = uri || `${Invoice.ENDPOINT}/${this.invoice_number}/mark_successful`
 
     this.put(uri, null, (err, response, payload) => {
-      const error = handleRecurlyError(err, response, payload, [ 200 ])
+      const error = handleRecurlyError(err, response, payload, [200])
       if (error) {
         return callback(error)
       }
@@ -138,13 +149,13 @@ class Invoice extends RecurlyData {
   markFailed(callback) {
     let uri = _.get(this, 'a.mark_failed.href')
     if (!uri && !this.invoice_number) {
-      throw (new Error('cannot mark invoice as failed without an invoice_number'))
+      throw new Error('cannot mark invoice as failed without an invoice_number')
     }
 
     uri = uri || `${Invoice.ENDPOINT}/${this.invoice_number}/mark_failed`
 
     this.put(uri, null, (err, response, payload) => {
-      const error = handleRecurlyError(err, response, payload, [ 200 ])
+      const error = handleRecurlyError(err, response, payload, [200])
       if (error) {
         return callback(error)
       }