test: remove unused params

[vakhid-shagmanov]

No description provided.

[vakhid-shagmanov]

why test .expect('Supports: text/plain, text/html, application/json') failed ?

[krzysdz]

why test .expect('Supports: text/plain, text/html, application/json') failed ?

Number of function parameters matters for declaring error handling middleware. function(err, req, res, next) - must have 4 parameters (including next, even if unused), otherwise it will be considered a normal middleware.

[vakhid-shagmanov]

@krzysdz even if not used?

[krzysdz]

Yes, number of function parameters is the only thing Express can use to distinguish normal and error-handling middleware, because both are just passed to router.use().

Official docs: https://expressjs.com/en/guide/error-handling.html#writing-error-handlers
Another question regarding the number of parameters and error handling middleware: #5065

[vakhid-shagmanov]

@krzysdz thanks for answering the questions