adding a couple more test cases

dist/angular-locker.js

@@ -128,11 +128,15 @@
                     };
 
                     /**
+                     * The driver instance
+                     *
                      * @type {Storage}
                      */
                     this._driver = this._resolveDriver(options.driver);
 
                     /**
+                     * The namespace value
+                     *
                      * @type {String}
                      */
                     this._namespace = options.namespace;
@@ -289,8 +293,8 @@
                         if (! this._checkSupport()) _error('The browser does not support localStorage');
 
                         if (! this._exists(key)) return false;
-                        this._driver.removeItem(this._getPrefix(key));
 
+                        this._driver.removeItem(this._getPrefix(key));
                         this._event('locker.item.forgotten', { key: key });
 
                         return true;
@@ -334,11 +338,12 @@
                      *
                      * @param  {Mixed}  key
                      * @param  {Mixed}  value
+                     * @param  {Mixed}  def
                      * @return {Boolean}
                      */
-                    add: function (key, value) {
+                    add: function (key, value, def) {
                         if (! this.has(key)) {
-                            this.put(key, value);
+                            this.put(key, value, def);
                             return true;
                         }
 

src/angular-locker.js

@@ -128,11 +128,15 @@
                     };
 
                     /**
+                     * The driver instance
+                     *
                      * @type {Storage}
                      */
                     this._driver = this._resolveDriver(options.driver);
 
                     /**
+                     * The namespace value
+                     *
                      * @type {String}
                      */
                     this._namespace = options.namespace;
@@ -289,8 +293,8 @@
                         if (! this._checkSupport()) _error('The browser does not support localStorage');
 
                         if (! this._exists(key)) return false;
-                        this._driver.removeItem(this._getPrefix(key));
 
+                        this._driver.removeItem(this._getPrefix(key));
                         this._event('locker.item.forgotten', { key: key });
 
                         return true;
@@ -334,11 +338,12 @@
                      *
                      * @param  {Mixed}  key
                      * @param  {Mixed}  value
+                     * @param  {Mixed}  def
                      * @return {Boolean}
                      */
-                    add: function (key, value) {
+                    add: function (key, value, def) {
                         if (! this.has(key)) {
-                            this.put(key, value);
+                            this.put(key, value, def);
                             return true;
                         }
 

test/spec/angular-locker.spec.js

@@ -378,7 +378,7 @@ describe('angular-locker', function () {
 
             it('should switch drivers when chained', function () {
                 module(function(lockerProvider) {
-                    lockerProvider.defaults.driver = 'local';
+                    lockerProvider.defaults().driver = 'local';
                 });
 
                 inject(function (locker) {
@@ -701,6 +701,28 @@ describe('angular-locker', function () {
                 expect( locker.getDriver() ).toEqual($window.localStorage);
                 expect( locker.driver('session').getDriver() ).toEqual($window.sessionStorage);
             }));
+
+            it('should return the same instance if driver or namespace not changed', inject(function ($window, locker) {
+                expect( locker.getDriver() ).toEqual($window.localStorage);
+
+                var localInstance1 = locker;
+                var localInstance2 = locker.driver('local');
+                var sessionInstance = locker.driver('session');
+
+                expect( localInstance1 ).toEqual(localInstance2);
+                expect( localInstance1 ).not.toEqual(sessionInstance);
+            }));
+
+            it('should return the same instance if driver or namespace not changed', inject(function ($window, locker) {
+                expect( locker.getNamespace() ).toEqual('locker');
+
+                var defaultNamespace1 = locker;
+                var defaultNamespace2 = locker.namespace('locker');
+                var fooNamespace = locker.namespace('foo');
+
+                expect( defaultNamespace1 ).toEqual(defaultNamespace2);
+                expect( defaultNamespace2 ).not.toEqual(fooNamespace);
+            }));
         });
 
     });