get_query_n_bad should use named parameter to dbGetQuery

[hannes]

get_query_n_bad calls dbGetQuery(con, query, -2) etc. However, in DBI the definition of dbGetQuery is function(conn, statement, ..., n = -1L) {. Hence, n is never modified for the default implementation.

[krlmlr]

Thanks. In the dev version, I see:

DBItest/R/spec-result-get-query.R

Lines 82 to 91 in 9e0e123

	get_query_n_bad = function(ctx) {
	with_connection({
	query <- trivial_query()
	expect_error(dbGetQuery(con, query, n = -2))
	expect_error(dbGetQuery(con, query, n = 1.5))
	expect_error(dbGetQuery(con, query, n = integer()))
	expect_error(dbGetQuery(con, query, n = 1:3))
	expect_error(dbGetQuery(con, query, n = NA_integer_))
	})
	},

[github-actions]

This old thread has been automatically locked. If you think you have found something related to this, please open a new issue and link to this old issue if necessary.