This fixes #25

If a user provides a non-existing dest path, the promise would simply resolve. The image would not have been written, since the dest path is not an existing path.
This is unexpected behavior. The Promise should be rejected if the WriteStream errors.

Another example: There dest path could point to a locked file, which would prevent the WriteStream from overwriting the existing file. I'm pretty sure, the promise would resolve.

Adding .on("error", err => reject(err)) will help.

Please be aware, that i haven't tested my change thoroughly.

This fixes #27

Michael Kemp authored Apr 11, 2022 and Sébastien Demanou committed Apr 11, 2022

This fixes #28

This fixes #28

This fixes #29