Implement keys function, issue #31

tsironis · Feb 29, 2016 · ceee274 · avgerin0s · Feb 29, 2016 · tsironis
1 parent 626aa8c
commit ceee274
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 5 deletions.
diff --git a/lockr.js b/lockr.js
@@ -124,13 +124,16 @@
     return Lockr.smembers(key).indexOf(value) > -1;
   };
 
-  Lockr.getAll = function () {
+  Lockr.keys = function() {
     var keys = Object.keys(localStorage);
-    
-    keys = keys.filter(function (key) {
-      return key.indexOf(Lockr.prefix) !== -1
+
+    return keys.filter(function (key) {
+      return key.indexOf(Lockr.prefix) !== -1;
     });
+  };
 
+  Lockr.getAll = function () {
+    var keys = Lockr.keys();
     return keys.map(function (key) {
       return Lockr.get(key.substr(Lockr.prefix.length));
     });

diff --git a/lockr.min.js b/lockr.min.js
diff --git a/specs/lockrSpecs.js b/specs/lockrSpecs.js
@@ -62,6 +62,19 @@ describe('Lockr.get', function () {
     expect(value).toEqual(0);
   });
 
+  it('gets Lockr keys', function() {
+    Lockr.flush();
+    Lockr.set('one', 1);
+    Lockr.set('two', 2);
+    Lockr.set('three', 3);
+    Lockr.set('four', 4);
+
+    var keys = Lockr.keys();
+
+    expect(keys.length).toBe(4);
+    expect(keys).toContain('one', 'two', 'three', 'four');
+  });
+
   it('gets all contents of the localStorage', function () {
     var contents = Lockr.getAll();
 
@@ -187,4 +200,18 @@ describe('Lockr::Prefixed', function() {
     Lockr.set("justlikeyou", true);
     expect("imaprefixjustlikeyou" in localStorage).toEqual(true);
   });
+  it('gets Lockr keys (with prefix)', function() {
+    Lockr.flush();
+
+    Lockr.set('one', 1);
+    Lockr.set('two', 2);
+    Lockr.set('three', 3);
+    Lockr.set('four', 4);
+
+    var keys = Lockr.keys();
+
+    expect(keys.length).toBe(4);
+    expect(keys).toContain('imaprefixone', 'imaprefixtwo', 'imaprefixthree', 'imaprefixfour');
+  });
+
 });