From 6135fea66c3e804e34e28ae5f058e3a874d5dff2 Mon Sep 17 00:00:00 2001
From: SleepWalker <mybox@udf.su>
Date: Mon, 12 Feb 2018 22:56:10 +0200
Subject: [PATCH] #389: do not send token with /api/options request

---
 src/services/api/options.js      |  5 +++--
 src/services/api/options.test.js | 38 ++++++++++++++++++++++++++++++++
 2 files changed, 41 insertions(+), 2 deletions(-)
 create mode 100644 src/services/api/options.test.js

diff --git a/src/services/api/options.js b/src/services/api/options.js
index 40666d5..6002a25 100644
--- a/src/services/api/options.js
+++ b/src/services/api/options.js
@@ -1,14 +1,15 @@
+// @flow
 import request from 'services/request';
 
 let options;
 
 export default {
-    get() {
+    get(): Promise<{ reCaptchaPublicKey: string }> {
         if (options) {
             return Promise.resolve(options);
         }
 
-        return request.get('/api/options').then((resp) => {
+        return request.get('/api/options', {}, { token: null }).then((resp) => {
             options = resp;
 
             return resp;
diff --git a/src/services/api/options.test.js b/src/services/api/options.test.js
new file mode 100644
index 0000000..bb6aaa9
--- /dev/null
+++ b/src/services/api/options.test.js
@@ -0,0 +1,38 @@
+import expect from 'unexpected';
+import sinon from 'sinon';
+import request from 'services/request';
+import options from './options';
+
+describe('services/api/options', () => {
+    const expectedResp = {foo: 'bar'};
+
+    beforeEach(() => {
+        sinon.stub(request, 'get')
+            .named('request.get')
+            .returns(Promise.resolve(expectedResp));
+    });
+
+    afterEach(() => {
+        request.get.restore();
+    });
+
+    it('should request options without token', () =>
+        options.get().then((resp) => {
+            expect(resp, 'to be', expectedResp);
+            expect(request.get, 'to have a call satisfying', [
+                '/api/options',
+                {},
+                { token: null }
+            ]);
+        })
+    );
+
+    it('should cache options', () =>
+        // NOTE: this is bad practice, but we are relying on the state from
+        // the previous test
+        options.get().then((resp) => {
+            expect(resp, 'to be', expectedResp);
+            expect(request.get, 'was not called');
+        })
+    );
+});