chore: address review comments

Author: achingbrain

test/exporter-sharded.spec.js

@@ -193,19 +193,15 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(32)
+        expect(exported.length).to.equal(32)
 
-          const dir = exported.shift()
+        const dir = exported.shift()
 
-          expect(dir.type).to.equal('dir')
+        expect(dir.type).to.equal('dir')
 
-          exported.forEach(file => expect(file.type).to.equal('file'))
+        exported.forEach(file => expect(file.type).to.equal('file'))
 
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        cb()
       }
     ], done)
   })
@@ -220,17 +216,13 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(1)
+        expect(exported.length).to.equal(1)
 
-          const file = exported.shift()
+        const file = exported.shift()
 
-          expect(file.name).to.deep.equal('file-14')
+        expect(file.name).to.deep.equal('file-14')
 
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        cb()
       }
     ], done)
   })
@@ -245,17 +237,13 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(1)
+        expect(exported.length).to.equal(1)
 
-          const file = exported.shift()
+        const file = exported.shift()
 
-          expect(file.name).to.deep.equal('file-30')
+        expect(file.name).to.deep.equal('file-30')
 
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        cb()
       }
     ], done)
   })
@@ -270,17 +258,13 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(1)
+        expect(exported.length).to.equal(1)
 
-          const file = exported.shift()
+        const file = exported.shift()
 
-          expect(file.name).to.deep.equal('file-2567')
+        expect(file.name).to.deep.equal('file-2567')
 
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        cb()
       }
     ], done)
   })
@@ -297,17 +281,11 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(1)
-
-          const entry = exported.pop()
+        expect(exported.length).to.equal(1)
 
-          expect(entry.name).to.deep.equal('baz')
+        const entry = exported.pop()
 
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        expect(entry.name).to.deep.equal('baz')
       }
     ], done)
   })
@@ -324,17 +302,13 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(1)
+        expect(exported.length).to.equal(1)
 
-          const entry = exported.pop()
+        const entry = exported.pop()
 
-          expect(entry.name).to.deep.equal('bar')
+        expect(entry.name).to.deep.equal('bar')
 
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        cb()
       }
     ], done)
   })
@@ -351,18 +325,14 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(5)
-
-          expect(exported[1].name).to.equal('foo')
-          expect(exported[2].name).to.equal('bar')
-          expect(exported[3].name).to.equal('baz')
-          expect(exported[4].name).to.equal('file-1')
-
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        expect(exported.length).to.equal(5)
+
+        expect(exported[1].name).to.equal('foo')
+        expect(exported[2].name).to.equal('bar')
+        expect(exported[3].name).to.equal('baz')
+        expect(exported[4].name).to.equal('file-1')
+
+        cb()
       }
     ], done)
   })
@@ -379,20 +349,16 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(35)
+        expect(exported.length).to.equal(35)
 
-          expect(exported[1].name).to.equal('foo')
-          expect(exported[2].name).to.equal('bar')
-          expect(exported[3].name).to.equal('baz')
-          expect(exported[4].name).to.equal('file-14')
+        expect(exported[1].name).to.equal('foo')
+        expect(exported[2].name).to.equal('bar')
+        expect(exported[3].name).to.equal('baz')
+        expect(exported[4].name).to.equal('file-14')
 
-          exported.slice(4).forEach(file => expect(file.type).to.equal('file'))
+        exported.slice(4).forEach(file => expect(file.type).to.equal('file'))
 
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        cb()
       }
     ], done)
   })
@@ -431,17 +397,13 @@ describe('exporter sharded', function () {
         )
       },
       (exported, cb) => {
-        try {
-          expect(exported.length).to.equal(1)
+        expect(exported.length).to.equal(1)
 
-          const entry = exported.pop()
+        const entry = exported.pop()
 
-          expect(entry.name).to.deep.equal('file-1')
+        expect(entry.name).to.deep.equal('file-1')
 
-          cb()
-        } catch (e) {
-          cb(e)
-        }
+        cb()
       }
     ], done)
   })