Merge pull request #10 from isocroft/patch-2

fix: fixed sendSMS url endpoint path

Author: stitchng

CHANGELOG.md

@@ -1,3 +1,9 @@
+<a name="0.1.0"></a>
+# 0.1.0 (2021-12-15)
+
+#### Bug Fixes
+- Fixed incorrect SMS Delivery endpoint URL path
+
 <a name="0.1.0-alpha.1"></a>
 # 0.1.0-alpha.1 (2021-05-13)
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "infobip-nodejs",
-  "version": "0.1.0-alpha.1",
+  "version": "0.1.0",
   "description": "A NodeJS Wrapper for InfoBip",
   "main": "index.js",
   "files": [

src/InfoBip/index.js

@@ -93,10 +93,10 @@ const setPathName = (config, values) => {
     offset) {
     let _value = values[string]
     return isTypeOf(
-      _value,
-      config.route_params[string]
+      config.route_params[string],
+      _value
     )
-      ? _value
+      ? config.route_params[string]
       : null
   })
 }
@@ -313,8 +313,7 @@ class InfoBip extends Mockable {
 
     let reqVerb = config.method.toLowerCase()
 
-    return this._mock !== null ? this._mock['sendSMSBinary'].bind(this, params) : got[reqVerb].bind(
-      got,
+    return this._mock !== null ? this._mock['sendSMSBinary'].bind(this, params) : got[reqVerb](
       `${this.baseUrl}${pathname}`,
       this.httpConfig
     )
@@ -324,9 +323,9 @@ class InfoBip extends Mockable {
     let config = {
       send_json: true,
       method: 'POST',
-      path: '/sms/2/text/advanced',
-      route_params: null,
-      params: { messages$: Array, bulkId: String }
+      path: '/sms/2/text/{:type}',
+      route_params: { type: typeof params.messages === 'undefined' ? 'single' : 'advanced' },
+      params: { messages: Array, bulkId: String, from: String, to: String, text: String, type: String }
     }
 
     if (config.route_params !== null ||
@@ -339,22 +338,24 @@ class InfoBip extends Mockable {
     let payload = setInputValues(config, params)
     let pathname = setPathName(config, params)
 
-    for (let messageIndex in params.messages) {
-      if (params.messages.hasOwnProperty(messageIndex)) {
-        let message = params.messages[messageIndex] || []
-        let destinations = message.destinations || []
+    if (params.messages) {
+      for (let messageIndex in params.messages) {
+        if (params.messages.hasOwnProperty(messageIndex)) {
+          let message = params.messages[messageIndex] || []
+          let destinations = message.destinations || []
 
-        if (!isTypeOf(message.from, ['string']) ||
-          !isTypeOf(message.text, ['string'])) {
-          throw new TypeError('infobip api: request payload for [from, text]; sendSMS() not of correct type')
-        }
+          if (!isTypeOf(message.from, ['string']) ||
+            !isTypeOf(message.text, ['string'])) {
+            throw new TypeError('infobip api: request payload for [from, text]; sendSMS() not of correct type')
+          }
 
-        for (let destinationIndex in destinations) {
-          if (destinations.hasOwnProperty(destinationIndex)) {
-            let destination = destinations[destinationIndex] || {}
+          for (let destinationIndex in destinations) {
+            if (destinations.hasOwnProperty(destinationIndex)) {
+              let destination = destinations[destinationIndex] || {}
 
-            if (!isTypeOf(destination.to, ['array', 'string'])) {
-              throw new TypeError('infobip api: request payload for [to]; SMS not of correct type')
+              if (!isTypeOf(destination.to, ['array', 'string'])) {
+                throw new TypeError('infobip api: request payload for [to]; SMS not of correct type')
+              }
             }
           }
         }
@@ -379,8 +380,7 @@ class InfoBip extends Mockable {
 
     let reqVerb = config.method.toLowerCase()
 
-    return this._mock !== null ? this._mock['sendSMS'].bind(this, params) : got[reqVerb].bind(
-      got,
+    return this._mock !== null ? this._mock['sendSMS'].bind(this, params) : got[reqVerb](
       `${this.baseUrl}${pathname}`,
       this.httpConfig
     )

test/infobip.test.js

@@ -23,7 +23,7 @@ describe('InfoBip Instance Test(s)', function () {
 
   it('should throw an error if [send] method is called without required arguments', function () {
     try {
-      instance.send()
+      instance.sendSMS()
     } catch (err) {
       should.exist(err)
     }