fix: resolve with full response

hfreire · Jun 23, 2017 · e57666b · e57666b
1 parent b59bdf2
commit e57666b
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 21 deletions.
diff --git a/src/happn-wrapper.js b/src/happn-wrapper.js
@@ -132,7 +132,6 @@ class HappnWrapper {
 
     return this._getRequestCircuitBreaker.exec(options)
       .then((response) => handleResponse(response))
-      .then(({ data }) => data)
   }
 
   getAccount () {
@@ -154,7 +153,6 @@ class HappnWrapper {
 
     return this._getRequestCircuitBreaker.exec(options)
       .then((response) => handleResponse(response))
-      .then(({ data }) => data)
   }
 
   getUpdates (limit = 10, offset = 0) {
@@ -174,7 +172,6 @@ class HappnWrapper {
 
     return this._getRequestCircuitBreaker.exec(options)
       .then((response) => handleResponse(response))
-      .then(({ data }) => data)
   }
 
   sendMessage () {

diff --git a/test/happn-wrapper.js b/test/happn-wrapper.js
@@ -135,8 +135,7 @@ describe('Happn Wrapper', () => {
   describe('when getting recommendations', () => {
     const userId = 'my-user-id'
     const statusCode = 200
-    const data = {}
-    const body = { data: {} }
+    const body = {}
     const response = { statusCode, body }
 
     beforeEach(() => {
@@ -178,8 +177,8 @@ describe('Happn Wrapper', () => {
 
     it('should resolve with response body as data', () => {
       return subject.getRecommendations()
-        .then((_data) => {
-          _data.should.be.eql(data)
+        .then((data) => {
+          data.should.be.eql(body)
         })
     })
   })
@@ -209,8 +208,7 @@ describe('Happn Wrapper', () => {
   describe('when getting account', () => {
     const userId = 'my-user-id'
     const statusCode = 200
-    const data = {}
-    const body = { data }
+    const body = {}
     const response = { statusCode, body }
 
     beforeEach(() => {
@@ -237,17 +235,16 @@ describe('Happn Wrapper', () => {
 
     it('should resolve with response body as data', () => {
       return subject.getAccount()
-        .then((_data) => {
-          _data.should.be.eql(data)
+        .then((data) => {
+          data.should.be.equal(body)
         })
     })
   })
 
   describe('when getting user', () => {
     const userId = 'my-user-id'
     const statusCode = 200
-    const data = {}
-    const body = { data }
+    const body = {}
     const response = { statusCode, body }
 
     beforeEach(() => {
@@ -273,8 +270,8 @@ describe('Happn Wrapper', () => {
 
     it('should resolve with response body as data', () => {
       return subject.getUser(userId)
-        .then((_data) => {
-          _data.should.be.eql(data)
+        .then((data) => {
+          data.should.be.equal(body)
         })
     })
   })
@@ -299,8 +296,7 @@ describe('Happn Wrapper', () => {
   describe('when getting update', () => {
     const userId = 'my-user-id'
     const statusCode = 200
-    const data = {}
-    const body = { data }
+    const body = {}
     const response = { statusCode, body }
 
     beforeEach(() => {
@@ -342,8 +338,8 @@ describe('Happn Wrapper', () => {
 
     it('should resolve with response body as data', () => {
       return subject.getUpdates()
-        .then((_data) => {
-          _data.should.be.eql(data)
+        .then((data) => {
+          data.should.be.eql(body)
         })
     })
   })
@@ -397,8 +393,8 @@ describe('Happn Wrapper', () => {
 
     it('should resolve with response body as data', () => {
       return subject.like(userIdToLike)
-        .then((_data) => {
-          _data.should.be.equal(body)
+        .then((data) => {
+          data.should.be.equal(body)
         })
     })
   })