How to stub unzipper? about node-unzipper HOT 4 CLOSED

I was actually going to suggest doing something just like that: stub in your own stream implementation (Readable, PassThrough, etc.) as the returned value from unzipper and emit events to trigger different function calls. The only downside is because you're keeping your functions private it requires more setup and stubbing to test each private method instead of exposing them for testing by exporting the function separately.

from node-unzipper.

Seems like that's a bit of a hassle. Would it be easier to pull out each of your event handlers and test them more individually? Testing code like that feels like its closer to an "integration" test since you're verifying different units of work will work together, which would make more sense to actually use unzipper as part of the test.

from node-unzipper.

I'm not an expert in testing, so I may have mixed unit and integration tests.
My event handlers are private methods and I didn't plan to test it (I know there is a debate about it).
Actually, the only thing I would like to test is the callback (with or without error):

if(isValid) {
    readableStream.pipe(unzipper.Parse())
        .on('entry', (entry) => {
            // Filter and write files like in the doc
            ...
        })
        .on('finish', () => {
            callback();
        });
} else {
    callback(new Error());
}

If stubbing unzipper is too complicated, I'll try to use it with a simple .zip file for my tests.

from node-unzipper.

Oops! My bad... Actually my first idea works well but I called the event emit at the wrong place 🤦‍♂️
Here is my solution:

describe('...', () => {
   let readableStream = null;
   let Parse = null;

   beforeEach(() => {
      readableStream = new stream.Readable({ read: () => null });
      Parse = sinon.stub(unzipper, 'Parse');
      Parse.returns(readableStream);
   });
   
   afterEach(() => {
      Parse.restore();
   });

   it('...', done => {
      myFunctionToTest(..., err => {
         expect(err).to.be.undefined;
         done();
      });
      readableStream.emit('finish');
   });
});

Sorry and thanks for you help @camlegleiter

from node-unzipper.