fix: missing verification and tests (#6)

* add tests * add verification
powerchordlabs · Nov 8, 2016 · 5af975f · 5af975f
1 parent beb1f01
commit 5af975f
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 0 deletions.
diff --git a/src/index.js b/src/index.js
@@ -284,6 +284,11 @@ const validators = {
       throw new InvalidArgumentError('id')
     }
   },
+  entity_byHostname(hostname) {
+    if(isInvalidString(hostname)) {
+      throw new InvalidArgumentError('hostname');
+    }
+  },
   content_getMany(type, filter) {
     if(isInvalidString(type)) {
       throw new InvalidArgumentError('type')

diff --git a/test/index.js b/test/index.js
@@ -278,6 +278,27 @@ describe('forteApi', () => {
 
   describe('api.organizations', () => {
 
+    describe('.getOneByHostname(hostname)', () => {
+      const invalidHostnames = [null, undefined, {}]
+      invalidHostnames.forEach((hostname) => {
+        it(`should throw for hostname '${hostname}'`, () => {
+          assert.throws(() => { api.organizations.getOneByHostname(hostname) }, InvalidArgumentError)
+        })
+      })
+
+      const validHostnames = ['www.domain.com', 'dealer.domain.com']
+      validHostnames.forEach((hostname) => {
+        const expected = expectedUri(ApiPaths.organizations.getOneByHostname(hostname))
+        it(`should GET uri: ${expected}`, () => {
+          const getOneByHostnameMock = mockapi.get(expected, 200)
+
+          return api.organizations.getOneByHostname(hostname).then(() => {
+            getOneByHostnameMock.done()
+          })
+        })
+      })
+    })
+
     describe('.getMany(filter)', () => {
       const invalidFilters = [null, undefined, {}]
       invalidFilters.forEach((filter) => {