Update: arrow functions simplified

c-vetter · Feb 22, 2017 · d24a98b · d24a98b
1 parent 4182655
commit d24a98b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 11 deletions.
diff --git a/lib/copy/__tests__/copy.test.js b/lib/copy/__tests__/copy.test.js
@@ -18,9 +18,7 @@ describe('fs-extra', () => {
     fse.emptyDir(TEST_DIR, done)
   })
 
-  afterEach(done => {
-    fse.remove(TEST_DIR, done)
-  })
+  afterEach(done => fse.remove(TEST_DIR, done))
 
   describe('+ copy()', () => {
     it('should return an error if src and dest are the same', done => {

diff --git a/lib/json/__tests__/output-json-sync.test.js b/lib/json/__tests__/output-json-sync.test.js
@@ -36,10 +36,7 @@ describe('json', () => {
         const file = path.join(TEST_DIR, 'this-dir', 'does-not', 'exist', 'really', 'file.json')
         assert(!fs.existsSync(file))
 
-        const replacer = (k, v) => {
-          if (v === 'JP') return 'Jon Paul'
-          return v
-        }
+        const replacer = (k, v) => v === 'JP' ? 'Jon Paul' : v
         const data = {name: 'JP'}
 
         outputJsonSync(file, data, { replacer })

diff --git a/lib/json/__tests__/output-json.test.js b/lib/json/__tests__/output-json.test.js
@@ -39,10 +39,7 @@ describe('json', () => {
         const file = path.join(TEST_DIR, 'this-dir', 'does-not', 'exist', 'really', 'file.json')
         assert(!fs.existsSync(file))
 
-        const replacer = (k, v) => {
-          if (v === 'JP') return 'Jon Paul'
-          return v
-        }
+        const replacer = (k, v) => v === 'JP' ? 'Jon Paul' : v
         const data = {name: 'JP'}
 
         outputJson(file, data, { replacer }, err => {