Rename test.prop to test.props and create test.prop that takes only 1 Arbitrary.

[waynevanson]

Rename test.prop to test.props and create test.prop that takes only 1 Arbitrary.

A breaking change.

I don't like having to create a tuple when I use the test integrations (vitest).

Current

it.prop([fc.string()])(string => {})

Proposed

it.prop(fc.string())(string => {})
it.props([fc.string()])(string => {})

There's also a world where another overload to the test function is provided which is non breaking.

What do you think?

I can implement the latter if you wish.

[gruhn]

Hmm, prop is short for property. props sounds like it's defining multiple properties. Not that it's a single property parametrized over multiple arbitraries .

[dubzzz]

I'd have said the same

[waynevanson]

That sounds fair enough.

How about having overload for it.prop(arbitrary) that can be used instead of it.prop(arbitrary)?

[dubzzz]

Actually we do already have it.prop([a1, a2]) and it.prop({ a1, a2 }), introducing a it.prop(a1, a2) would probably be too much. It should/could probably replace it.prop([a1, a2]) if relevant.