Merge pull request #4 from uphold/feature/update-me-with-otp

Add otp argument to .updateMe()
uphold · May 24, 2017 · 5bd5854 · 5bd5854
2 parents 04d0e11 + 7f3ec20
commit 5bd5854
Show file tree

Hide file tree

Showing 3 changed files with 39 additions and 4 deletions.
diff --git a/docs/actions/user/update-me.md b/docs/actions/user/update-me.md
@@ -2,9 +2,12 @@
 
 Updates the authenticated user's details.
 
+The `otp` argument will add the `OTP-TOKEN` header to the request.
+
 | Argument  | Type   | Required | Description                                              |
 |:----------|:-------|:---------|:---------------------------------------------------------|
 | `body`    | Object | Yes      | Request body                                             |
+| `otp`     | String | No       | OTP token                                                |
 | `options` | Object | No       | Any options you may want to pass to [`.api()`](/sdk#api) |
 
 The `body` argument accepts the following keys:

diff --git a/src/core/actions/user.js b/src/core/actions/user.js
@@ -4,8 +4,8 @@ export function getMe(options) {
   return this.api('/me', options);
 }
 
-export function updateMe({ address, birthdate, country, firstName, identity, lastName, settings, state, username }, options) {
-  return this.api('/me', merge({
+export function updateMe({ address, birthdate, country, firstName, identity, lastName, settings, state, username }, otp, options) {
+  options = merge({
     body: {
       address,
       birthdate,
@@ -18,5 +18,14 @@ export function updateMe({ address, birthdate, country, firstName, identity, las
       username
     },
     method: 'patch'
-  }, options));
+  }, options);
+
+  if (otp) {
+    options.headers = {
+      'otp-token': otp,
+      ...options.headers
+    };
+  }
+
+  return this.api('/me', options);
 }
diff --git a/test/core/actions/user.spec.js b/test/core/actions/user.spec.js
@@ -29,7 +29,7 @@ describe('UserActions', () => {
         settings: 'qex',
         state: 'qix',
         username: 'qox'
-      }, { fiz: 'faz' })
+      }, false, { fiz: 'faz' })
         .then(result => {
           expect(result).toBe('foo');
           expect(sdk.api).toBeCalledWith('/me', {
@@ -49,5 +49,28 @@ describe('UserActions', () => {
           });
         });
     });
+
+    it('should make a request to `PATCH /me` with otp', () => {
+      return sdk.updateMe({
+        address: 'bar'
+      }, 'biz', {
+        headers: {
+          baz: 'buz'
+        }
+      })
+        .then(result => {
+          expect(result).toBe('foo');
+          expect(sdk.api).toBeCalledWith('/me', {
+            body: {
+              address: 'bar'
+            },
+            headers: {
+              baz: 'buz',
+              'otp-token': 'biz'
+            },
+            method: 'patch'
+          });
+        });
+    });
   });
 });