Expected 1-3 arguments, but got 2

TypeScript Version: 2.6.0-dev.20171015

Code

declare function f(x: number): number;
declare function f(x: number, y: number, z: number): number;

f(1, 2);

Expected behavior:

Expected 1 or 3 arguments, but got 2.

Actual behavior:

src/a.ts(4,1): error TS2554: Expected 1-3 arguments, but got 2.

[lmcarreiro]

But what if it was:

declare function f(x: number): number;
declare function f(x: number, y: number, z: number): number;
declare function f(x: number, y: number, z: number, a: number): number;
declare function f(x: number, y: number, z: number, a: number, b: number): number;
...
declare function f(x: number, y: number, z: number, a: number, b: number, ...): number;

f(1, 2);

The message would be Expected 1 or 3 or 4 or 5 or ... or N arguments, but got 2. ???

Based on existing error messages, it would be ideal to say Expected 1 or at least 3 arguments, but got 2..
Of course, doing this in a localizable way without defining too many different diagnostics will be more difficult; might be better to use commas instead of or (except for the last one) so that the template would be Expected {0} or at least {1} arguments and fill in {0} with a comma-separated list if necessary.

[DanielRosenwasser]

Consider something like No overload expects {0} arguments. The most likely overloads that match expect either {1} arguments or at least {2} arguments.

@DanielRosenwasser Good, assuming that "most likely" would mean the next lowest and next highest numbers of arguments. So if overloads accepted 1, 2, or 5 arguments and you provide 3 or 4, say "2 or 5".

[DanielRosenwasser]

Yup, that's what I should've been explicit about, thanks for clarifying. 😃

[sandersn]

Note that this has occurred at least once in real code: DefinitelyTyped/DefinitelyTyped#19119.

[gusdecool]

this also happened on https://www.npmjs.com/package/@types/urijs library when we are calling addQuery() with 2 arguments

import * as URI from 'urijs';

const path = new URI(`/some-url`);
path.addQuery('pickUpCoordinate', 'foo-query'); // this will throw TS2554 expected 1 argument but got 2 arguments