replaced navigator with the angularjs equivalent .navigator, made the function browserSupportsCookies available for public access as cookie.isSupported and wrote tests to verify the functionality

Author: Arthur Helmel

dist/angular-local-storage.js

@@ -1,6 +1,6 @@
 /**
  * An Angular module that gives you access to the browsers local storage
- * @version v0.1.3 - 2014-10-14
+ * @version v0.1.3 - 2014-10-30
  * @link https://github.com/grevory/angular-local-storage
  * @author grevory <greg@gregpike.ca>
  * @license MIT License, http://www.opensource.org/licenses/MIT
@@ -286,16 +286,16 @@ angularLocalStorage.provider('localStorageService', function() {
     };
 
     // Checks the browser to see if cookies are supported
-    var browserSupportsCookies = function() {
+    var browserSupportsCookies = (function() {
       try {
-        return navigator.cookieEnabled ||
+        return $window.navigator.cookieEnabled ||
           ("cookie" in $document && ($document.cookie.length > 0 ||
           ($document.cookie = "test").indexOf.call($document.cookie, "test") > -1));
       } catch (e) {
           $rootScope.$broadcast('LocalStorageModule.notification.error', e.message);
           return false;
       }
-    };
+    }());
 
     // Directly adds a value to cookies
     // Typically used as a fallback is local storage is not available in the browser
@@ -308,7 +308,7 @@ angularLocalStorage.provider('localStorageService', function() {
         value = toJson(value);
       }
 
-      if (!browserSupportsCookies()) {
+      if (!browserSupportsCookies) {
         $rootScope.$broadcast('LocalStorageModule.notification.error', 'COOKIES_NOT_SUPPORTED');
         return false;
       }
@@ -344,7 +344,7 @@ angularLocalStorage.provider('localStorageService', function() {
     // Directly get a value from a cookie
     // Example use: localStorageService.cookie.get('library'); // returns 'angular'
     var getFromCookies = function (key) {
-      if (!browserSupportsCookies()) {
+      if (!browserSupportsCookies) {
         $rootScope.$broadcast('LocalStorageModule.notification.error', 'COOKIES_NOT_SUPPORTED');
         return false;
       }
@@ -437,6 +437,7 @@ angularLocalStorage.provider('localStorageService', function() {
       deriveKey: deriveQualifiedKey,
       length: lengthOfLocalStorage,
       cookie: {
+        isSupported: browserSupportsCookies,
         set: addToCookies,
         add: addToCookies, //DEPRECATED
         get: getFromCookies,

dist/angular-local-storage.min.js

@@ -259,16 +259,16 @@ angularLocalStorage.provider('localStorageService', function() {
     };
 
     // Checks the browser to see if cookies are supported
-    var browserSupportsCookies = function() {
+    var browserSupportsCookies = (function() {
       try {
-        return navigator.cookieEnabled ||
+        return $window.navigator.cookieEnabled ||
           ("cookie" in $document && ($document.cookie.length > 0 ||
           ($document.cookie = "test").indexOf.call($document.cookie, "test") > -1));
       } catch (e) {
           $rootScope.$broadcast('LocalStorageModule.notification.error', e.message);
           return false;
       }
-    };
+    }());
 
     // Directly adds a value to cookies
     // Typically used as a fallback is local storage is not available in the browser
@@ -281,7 +281,7 @@ angularLocalStorage.provider('localStorageService', function() {
         value = toJson(value);
       }
 
-      if (!browserSupportsCookies()) {
+      if (!browserSupportsCookies) {
         $rootScope.$broadcast('LocalStorageModule.notification.error', 'COOKIES_NOT_SUPPORTED');
         return false;
       }
@@ -317,7 +317,7 @@ angularLocalStorage.provider('localStorageService', function() {
     // Directly get a value from a cookie
     // Example use: localStorageService.cookie.get('library'); // returns 'angular'
     var getFromCookies = function (key) {
-      if (!browserSupportsCookies()) {
+      if (!browserSupportsCookies) {
         $rootScope.$broadcast('LocalStorageModule.notification.error', 'COOKIES_NOT_SUPPORTED');
         return false;
       }
@@ -410,6 +410,7 @@ angularLocalStorage.provider('localStorageService', function() {
       deriveKey: deriveQualifiedKey,
       length: lengthOfLocalStorage,
       cookie: {
+        isSupported: browserSupportsCookies,
         set: addToCookies,
         add: addToCookies, //DEPRECATED
         get: getFromCookies,

src/angular-local-storage.js

@@ -62,6 +62,12 @@ describe('localStorageService', function() {
     };
   }
 
+  function expectCookieSupporting(expected) {
+    return function(localStorageService) {
+      expect(localStorageService.cookie.isSupported).toEqual(expected);
+    };
+  }
+
   function expectDomain(domain) {
     return function($document, localStorageService) {
       localStorageService.set('foo','bar'); //Should trigger first time
@@ -417,7 +423,10 @@ describe('localStorageService', function() {
     beforeEach(module('LocalStorageModule', function($provide) {
       $provide.value('$window', {
         localStorage: false,
-        sessionStorage: false
+        sessionStorage: false,
+        navigator: {
+          cookieEnabled: true
+        }
       });
       $provide.value('$document', {
         cookie: ''
@@ -428,6 +437,10 @@ describe('localStorageService', function() {
       expectSupporting(false)
     ));
 
+    it('cookie.isSupported should be true if cookies are enabled', inject(
+      expectCookieSupporting(true)
+    ));
+
     it('fallback storage type should be cookie', inject(
       expectStorageTyping('cookie')
     ));
@@ -504,4 +517,22 @@ describe('localStorageService', function() {
     };
   });
 
+  //cookie disabled
+  describe('No Cookie', function() {
+
+    beforeEach(module('LocalStorageModule', function($provide) {
+      $provide.value('$window', {
+        navigator: {
+          cookieEnabled: false
+        }
+      });
+      $provide.value('$document', {
+
+      });
+    }));
+
+    it('cookie.isSupported should be false if cookies are disabled', inject(
+      expectCookieSupporting(false)
+    ));
+  });
 });