Idea, test more cases:

• pass invalid table name to KDbTableOrQuerySchema so there is no table found; -1 result expected

• create an empty KDbQuerySchema() and pass to KDbTableOrQuerySchema; -1 result expected because it has no connection assigned

• create an KDbQuerySchema(kdb_t) query; expected result == 4 because it's a "SELECT * ..." query

• test 3 other overloads of KDb::recordCount()
  • int recordCount(KDbConnection* conn, const KDbEscapedString& sql) (with valid and invalid SQL)
  • int recordCount(const KDbTableSchema& tableSchema);
  • int recordCount(KDbQuerySchema* querySchema, const QList<QVariant>& params = QList<QVariant>());

• test parameter queries (i.e. calls with const QList<QVariant>& params set)

• test all the overloads with nullptr connection or table/query arguments, -1 expected (actually there will be some crashes, will fix this :) )

Let's use QCOMPARE() for any comparisons of values other than bool.

Then "Number of records in default persons table is 4." is implied.

Please create on stack, memory leak otherwise.

So:

KDbTableOrQuerySchema tableOrQuery(utils.connection.data(), "persons");

not needed, always present

Not needed