Merge pull request #1098 from matrix-org/rav/test_rts_login

Groundwork for tests including a teamserver login

Author: richvdh

.eslintignore.errorfiles

@@ -146,7 +146,6 @@ src/Roles.js
 src/RoomListSorter.js
 src/RoomNotifs.js
 src/Rooms.js
-src/RtsClient.js
 src/ScalarAuthClient.js
 src/ScalarMessaging.js
 src/SdkConfig.js

src/Lifecycle.js

@@ -265,7 +265,11 @@ function _handleRestoreFailure(e) {
 
 let rtsClient = null;
 export function initRtsClient(url) {
-    rtsClient = new RtsClient(url);
+    if (url) {
+        rtsClient = new RtsClient(url);
+    } else {
+        rtsClient = null;
+    }
 }
 
 /**

src/RtsClient.js

@@ -1,5 +1,7 @@
 import 'whatwg-fetch';
 
+let fetchFunction = fetch;
+
 function checkStatus(response) {
     if (!response.ok) {
         return response.text().then((text) => {
@@ -31,7 +33,7 @@ const request = (url, opts) => {
         opts.body = JSON.stringify(opts.body);
         opts.headers['Content-Type'] = 'application/json';
     }
-    return fetch(url, opts)
+    return fetchFunction(url, opts)
         .then(checkStatus)
         .then(parseJson);
 };
@@ -64,7 +66,7 @@ export default class RtsClient {
                     client_secret: clientSecret,
                 },
                 method: 'POST',
-            }
+            },
         );
     }
 
@@ -74,7 +76,7 @@ export default class RtsClient {
                 qs: {
                     team_token: teamToken,
                 },
-            }
+            },
         );
     }
 
@@ -91,7 +93,12 @@ export default class RtsClient {
                 qs: {
                     user_id: userId,
                 },
-            }
+            },
         );
     }
+
+    // allow fetch to be replaced, for testing.
+    static setFetch(fn) {
+        fetchFunction = fn;
+    }
 }

src/components/structures/MatrixChat.js

@@ -263,11 +263,8 @@ module.exports = React.createClass({
         window.addEventListener('resize', this.handleResize);
         this.handleResize();
 
-        if (this.props.config.teamServerConfig &&
-            this.props.config.teamServerConfig.teamServerURL
-        ) {
-            Lifecycle.initRtsClient(this.props.config.teamServerConfig.teamServerURL);
-        }
+        const teamServerConfig = this.props.config.teamServerConfig || {};
+        Lifecycle.initRtsClient(teamServerConfig.teamServerURL);
 
         // if the user has followed a login or register link, don't reanimate
         // the old creds, but rather go straight to the relevant page

src/index.js

@@ -15,6 +15,7 @@ limitations under the License.
 */
 
 import Skinner from './Skinner';
+import RtsClient from './RtsClient';
 
 module.exports.loadSkin = function(skinObject) {
     Skinner.load(skinObject);
@@ -27,3 +28,7 @@ module.exports.resetSkin = function() {
 module.exports.getComponent = function(componentName) {
     return Skinner.getComponent(componentName);
 };
+
+module.exports.setFetch = function(fetchFunction) {
+    RtsClient.setFetch(fetchFunction);
+};