Merge pull request #29 from richmolj/master

 Make setJWT update localStorage when appropriate

Author: richmolj

package.json

@@ -1,6 +1,6 @@
 {
   "name": "jsorm",
-  "version": "0.4.1",
+  "version": "0.5.0",
   "description": "Javascript ORM",
   "main": "_bundles/jsorm.js",
   "module": "lib-esm/index.js",

src/model.ts

@@ -63,6 +63,10 @@ export default class Model {
 
   static setJWT(token: string) : void {
     this.getJWTOwner().jwt = token;
+
+    if (Config.jwtLocalStorage) {
+      Config.localStorage.setItem(Config.jwtLocalStorage, token)
+    }
   }
 
   static getJWT() : string {

src/util/refresh-jwt.ts

@@ -3,8 +3,9 @@ import Config from '../configuration';
 
 export default function refreshJWT(klass: typeof Model, serverResponse: Response) : void {
   let jwt = serverResponse.headers.get('X-JWT');
-  let localStorage = Config.localStorage;
+  if (!jwt) return
 
+  let localStorage = Config.localStorage;
   if (localStorage) {
     let localStorageKey = Config.jwtLocalStorage;
     if (localStorageKey) {

test/unit/model-test.ts

@@ -132,6 +132,25 @@ describe('Model', function() {
       Author.setJWT('n3wt0k3n');
       expect(ApplicationRecord.jwt).to.eq('n3wt0k3n');
     });
+
+    describe.only('when localStorage is configured', function() {
+      beforeEach(function() {
+        Config.jwtLocalStorage = 'jwt'
+        Config.localStorage = { setItem: sinon.spy() }
+      })
+
+      afterEach(function() {
+        Config.jwtLocalStorage = undefined
+        Config.localStorage = undefined
+      })
+
+      it('adds to localStorage', function() {
+        Author.setJWT('n3wt0k3n');
+        let called = Config.localStorage.setItem
+          .calledWith('jwt', 'n3wt0k3n');
+        expect(called).to.eq(true);
+      })
+    })
   });
 
   describe('#fetchOptions', function() {