chore(read-chunk): assert size >= 0

@vates/read-chunk/index.js

@@ -20,10 +20,10 @@ const assert = require('assert')
 const readChunk = (stream, size) =>
   stream.closed || stream.readableEnded
     ? Promise.resolve(null)
-    : size === 0
-    ? Promise.resolve(Buffer.alloc(0))
     : new Promise((resolve, reject) => {
         if (size !== undefined) {
+          assert(size > 0)
+
           // per Node documentation:
           // > The size argument must be less than or equal to 1 GiB.
           assert(size < 1073741824)

@vates/read-chunk/index.test.js

@@ -19,6 +19,11 @@ const rejectionOf = promise =>
   )
 
 describe('readChunk', () => {
+  it('rejects if size is less than or equal to 0', async () => {
+    const error = await rejectionOf(readChunk(makeStream([]), 0))
+    assert.strictEqual(error.code, 'ERR_ASSERTION')
+  })
+
   it('rejects if size is greater than or equal to 1 GiB', async () => {
     const error = await rejectionOf(readChunk(makeStream([]), 1024 * 1024 * 1024))
     assert.strictEqual(error.code, 'ERR_ASSERTION')
@@ -51,10 +56,6 @@ describe('readChunk', () => {
     it('returns less data if stream ends', async () => {
       assert.deepEqual(await readChunk(makeStream(['foo', 'bar']), 10), Buffer.from('foobar'))
     })
-
-    it('returns an empty buffer if the specified size is 0', async () => {
-      assert.deepEqual(await readChunk(makeStream(['foo', 'bar']), 0), Buffer.alloc(0))
-    })
   })
 
   describe('with object stream', () => {