Add note to README, run prettier (#123)

* Add note to README, run prettier

* Don't need circleci anymore

README.md

@@ -11,6 +11,10 @@ module dependencies by using this.
 [![Greenkeeper badge](https://badges.greenkeeper.io/clarkbw/jest-localstorage-mock.svg)](https://greenkeeper.io/)
 [![Twitter](https://img.shields.io/twitter/url/https/github.com/clarkbw/jest-localstorage-mock.svg?style=social)](https://twitter.com/intent/tweet?text=Wow:&url=%5Bobject%20Object%5D)
 
+# Jest 24+
+
+Note that with `jest@24` and above this project potentially duplicating functionality.
+
 ## Install
 
 This should only be installed as a development dependency (`devDependencies`) as

__tests__/index.js

@@ -1,5 +1,5 @@
 describe('storage', () =>
-  [localStorage, sessionStorage].map(storage => {
+  [localStorage, sessionStorage].map((storage) => {
     // https://html.spec.whatwg.org/multipage/webstorage.html#storage
     beforeEach(() => {
       storage.clear();

__tests__/localstorage.test.js

@@ -1,20 +1,20 @@
-import { LocalStorage } from "../src/localstorage";
+import { LocalStorage } from '../src/localstorage';
 
-describe("localstorage", () => {
+describe('localstorage', () => {
   let localStorage;
 
   beforeEach(() => {
     localStorage = new LocalStorage(jest);
   });
 
-  describe("getItem", () => {
-    it("should return null if the item is undefined", () => {
-      expect(localStorage.getItem("item")).toBeNull();
+  describe('getItem', () => {
+    it('should return null if the item is undefined', () => {
+      expect(localStorage.getItem('item')).toBeNull();
     });
 
     it("should return '' instead of null", () => {
-      localStorage.setItem("item", "");
-      expect(localStorage.getItem("item")).toBe("");
+      localStorage.setItem('item', '');
+      expect(localStorage.getItem('item')).toBe('');
     });
   });
 });

__tests__/setup.test.js

@@ -9,7 +9,7 @@ describe('setup', () => {
       'localStorage',
       '_sessionStorage',
       'sessionStorage',
-    ].forEach(globalKey => {
+    ].forEach((globalKey) => {
       if (restore) {
         delete global[globalKey];
         global[globalKey] = orignalImpGlobsl[globalKey];
@@ -31,7 +31,7 @@ describe('setup', () => {
     restoreGlobals();
   });
 
-  ['_localStorage', '_sessionStorage'].forEach(gKey => {
+  ['_localStorage', '_sessionStorage'].forEach((gKey) => {
     it(`[${gKey}] should define a property on the global object with writable false`, () => {
       require('../src/setup');
       expect(global[gKey.replace('_', '')].constructor.name).toBe(

src/localstorage.js

@@ -2,7 +2,7 @@ export class LocalStorage {
   constructor(jest) {
     Object.defineProperty(this, 'getItem', {
       enumerable: false,
-      value: jest.fn(key => this[key] !== undefined ? this[key] : null),
+      value: jest.fn((key) => (this[key] !== undefined ? this[key] : null)),
     });
     Object.defineProperty(this, 'setItem', {
       enumerable: false,
@@ -13,14 +13,14 @@ export class LocalStorage {
     });
     Object.defineProperty(this, 'removeItem', {
       enumerable: false,
-      value: jest.fn(key => {
+      value: jest.fn((key) => {
         delete this[key];
       }),
     });
     Object.defineProperty(this, 'clear', {
       enumerable: false,
       value: jest.fn(() => {
-        Object.keys(this).map(key => delete this[key]);
+        Object.keys(this).map((key) => delete this[key]);
       }),
     });
     Object.defineProperty(this, 'toString', {
@@ -31,7 +31,7 @@ export class LocalStorage {
     });
     Object.defineProperty(this, 'key', {
       enumerable: false,
-      value: jest.fn(idx => Object.keys(this)[idx] || null),
+      value: jest.fn((idx) => Object.keys(this)[idx] || null),
     });
   } // end constructor