[quaff]

Add missing methods:

1. param(int jdbcIndex, @Nullable Object value)
2. param(int jdbcIndex, @Nullable Object value, int sqlType)
3. param(String name, @Nullable Object value, int sqlType)

[sbrannen]

Thanks for the suggestions and the PR.

I've requested a small fix.

[sbrannen]

The new tests in JdbcClientIntegrationTests use add() more than once, resulting in a multi-statement batch update. However, the new unit tests in JdbcClientIndexedParameterTests and JdbcClientNamedParameterTests only have a single add(), resulting in a single-statement batch.

Please introduce additional unit tests analogous to the current new ones but with multiple statements (e.g., 2) per batch update, and verify setObject(...) is invoked with the right values for both rows in order, plus containsExactly(...).

[quaff]

The new tests in JdbcClientIntegrationTests use add() more than once, resulting in a multi-statement batch update. However, the new unit tests in JdbcClientIndexedParameterTests and JdbcClientNamedParameterTests only have a single add(), resulting in a single-statement batch.

Please introduce additional unit tests analogous to the current new ones but with multiple statements (e.g., 2) per batch update, and verify setObject(...) is invoked with the right values for both rows in order, plus containsExactly(...).

Updated tests with @ParameterizedTest.

[sbrannen]

Thanks for making the requested changes.

I have now requested an additional minor simplification.

[sbrannen]

This has been merged into main.

Thanks

[sbrannen]

See also:

• Introduce entry-based DSL for JdbcClient batch updates #37341