Add support for storageService objects

Implements #11  to add support for the storageService configuration value to be an object that supports the expected storage API.

Author: justinsa

.gitignore

@@ -1,3 +1,4 @@
 node_modules/
 *.log
 temp/
+package-lock.json

README.md

@@ -76,9 +76,9 @@ All properties (own and inherited) of the extensions object will be available as
 
 If you do not provide a storage service then a simple, in-memory dictionary will be used.
 
-You can provide any storage service that supports the following API:
+You can provide any storage service or object that supports the following API:
 
-  1. ```mixed get(key)```
+  1. ```any get(key)```
   2. ```boolean has(key)```
   3. ```void remove(key)```
   4. ```void set(key, value)```
@@ -93,6 +93,16 @@ app.config(['$authenticationProvider', function ($authenticationProvider) {
 }]);
 ```
 
+or an object that provides the expected functionality:
+
+```JAVASCRIPT
+app.config(['$authenticationProvider', function ($authenticationProvider) {
+  $authenticationProvider.configure({
+    storageService: new CustomStorageService()
+  });
+}]);
+```
+
 The ng-authentication-service was designed in tandem with the [ng-local-storage-service](https://github.com/justinsa/angular-local-storage-service).
 
 ## API

ng-authentication-service.js

@@ -63,7 +63,7 @@
       var storeService;
       var storageService = function () {
         if (storeService === undefined) {
-          if (!_.isString(configuration.storageService)) {
+          if (_.isNil(configuration.storageService)) {
             // Use a simple, in-memory storage option
             storeService = {
               dictionary: {},
@@ -82,12 +82,18 @@
               }
             };
             return storeService;
+          } else if (_.isString(configuration.storageService)) {
+            // Get the service from the $injector
+            if (!$injector.has(configuration.storageService)) {
+              $log.error('No matching service registered in Angular: ', configuration.storageService);
+              return undefined;
+            }
+            storeService = $injector.get(configuration.storageService);            
+          } else {
+            // The configuration object is the service
+            storeService = configuration.storageService;
           }
-          if (!$injector.has(configuration.storageService)) {
-            $log.error('No matching service registered in Angular: ', configuration.storageService);
-            return undefined;
-          }
-          storeService = $injector.get(configuration.storageService);
+
           _.each(['get', 'has', 'remove', 'set'], function (fnName) {
             if (!_.hasIn(storeService, fnName) || !_.isFunction(storeService[fnName])) {
               $log.error('storageService is missing function: ', fnName);

tests.js

@@ -67,6 +67,28 @@ describe('$authentication', function () {
     })
   );
 
+  describe('with object storageService', function () {
+    var storage = {
+      get: function () {},
+      has: function () {},
+      remove: function () {},
+      set: function () {},
+      test: function () { return 'test'; }
+    };
+
+    beforeEach(
+      module('authentication.service', function ($authenticationProvider) {
+        $authenticationProvider.configure({
+          storageService: storage
+        });
+      })
+    );
+
+    it('should use the provided storage object', inject(function ($authentication) {
+      $authentication.store().test().should.equal('test');
+    }));
+  });
+
   describe('with extensions', function () {
     beforeEach(function () {
       module('authentication.service', function ($authenticationProvider) {
@@ -82,7 +104,7 @@ describe('$authentication', function () {
       });
     });
 
-    it('should expose the provided fields that do not conflict with the existing api',
+    it('should expose the provided fields that do not conflict with the existing API',
       inject(function ($authentication) {
         $authentication.key1.should.equal('key1');
         $authentication.key2.should.be.a.Function();