fs.unlink throw exception if path is not defined

[armandciejak]

• Version: v4.4.3
• Platform: Linux 005753 3.14.28 deps: update openssl to 1.0.1j #1 SMP PREEMPT Thu Feb 18 14:13:50 CET 2016 armv7l GNU/Linux
• Subsystem: fs

When calling fs.unlink with path argument undefined the function throw an exception. I expect the function to return all errors with the callback as documented.

Here is how to reproduce the problem using the REPL:

> var fs = require('fs');
> var p;
> fs.unlink(p, function(error) {console.log('Async error: ' + error.message);});
TypeError: path must be a string
    at TypeError (native)
    at Object.fs.unlink (fs.js:927:11)
    at repl:1:4
    at REPLServer.defaultEval (repl.js:262:27)
    at bound (domain.js:287:14)
    at REPLServer.runBound [as eval] (domain.js:300:12)
    at REPLServer.<anonymous> (repl.js:431:12)
    at emitOne (events.js:82:20)
    at REPLServer.emit (events.js:169:7)
    at REPLServer.Interface._onLine (readline.js:211:10)

[cjihrig]

Node's policy is to throw on obvious programming errors. In this case, fs.unlink() expects a string.