sascha
/
query
1
0
Fork 0
Code Issues 4 Pull Requests Actions Packages Projects Releases Wiki Activity

Compare commits

base: sascha:ed6366fef652557e0e5afeb9e2a9167f5dc39797
Branches Tags
sascha:main
sascha:feature/matador-ng
..
compare: sascha:2060883d59a0c9dc90c5e970471dbed59a369094
Branches Tags
sascha:feature/matador-ng
sascha:main

No commits in common. "ed6366fef652557e0e5afeb9e2a9167f5dc39797" and "2060883d59a0c9dc90c5e970471dbed59a369094" have entirely different histories.
ed6366fef6 ... 2060883d59

5 changed files with 46 additions and 99 deletions
Show Stats Expand all files Collapse all files

19
include/matador/sql/column_generator.hpp
View File

@ -73,10 +73,7 @@ public:
void on_revision(const char *id, unsigned long long &rev);
template<typename Type>
void on_attribute(const char *id, Type &x, const utils::field_attributes &attr = utils::not_null_attributes);
template<typename Type>
void on_attribute(const char *id, std::optional<Type> &x, const utils::field_attributes &attr = utils::null_attributes);
void on_attribute(const char *id, Type &x, const utils::field_attributes &attr = utils::null_attributes);
template<class Pointer>
void on_belongs_to(const char *id, Pointer &x, utils::cascade_type)
@ -109,23 +106,13 @@ private:
template<typename V>
void column_generator::on_primary_key(const char *id, V &x, typename std::enable_if<std::is_integral<V>::value && !std::is_same<bool, V>::value>::type*)
{
on_attribute(id, x, { utils::constraints::PRIMARY_KEY | utils::constraints::NOT_NULL });
on_attribute(id, x, { utils::constraints::PRIMARY_KEY });
}
template<typename Type>
void column_generator::on_attribute(const char *id, Type &x, const utils::field_attributes &attr)
{
if (attr.options() == utils::constraints::NONE) {
columns_.push_back(column{id, x, { attr.size(), utils::constraints::NOT_NULL}});
} else {
columns_.emplace_back(id, x, attr);
}
}
template<typename Type>
void column_generator::on_attribute(const char *id, std::optional<Type> &x, const utils::field_attributes &attr)
{
columns_.emplace_back(id, data_type_traits<Type>::builtin_type(attr.size()), attr);
columns_.emplace_back(id, x, attr);
}
}

6
include/matador/utils/access.hpp
View File

@ -4,7 +4,6 @@
#include "matador/utils/cascade_type.hpp"
#include <string>
#include <optional>
namespace matador::utils {
@ -45,11 +44,6 @@ void attribute(Operator &op, const char *id, Type &value, const field_attributes
op.on_attribute(id, value, attr);
}
template<class Operator, class Type>
void attribute(Operator &op, const char *id, std::optional<Type> &value, const field_attributes &attr) {
op.on_attribute(id, value, attr);
}
template<class Operator, class Type>
void attribute(Operator &op, const char *id, Type &value) {
op.on_attribute(id, value);

2
src/sql/column_generator.cpp
View File

@ -10,7 +10,7 @@ column_generator::column_generator(std::vector<column> &columns, const table_rep
void column_generator::on_primary_key(const char *id, std::string &pk, size_t size)
{
on_attribute(id, pk, { size, utils::constraints::PRIMARY_KEY | utils::constraints::NOT_NULL });
on_attribute(id, pk, { size, utils::constraints::PRIMARY_KEY });
}
void column_generator::on_revision(const char *id, unsigned long long int &x)

46
test/ColumnGeneratorTest.cpp
View File

@ -4,53 +4,29 @@
#include "matador/sql/table_repository.hpp"
#include "models/product.hpp"
#include "models/optional.hpp"
using namespace matador::sql;
using namespace matador::utils;
TEST_CASE("Generate columns from object", "[column generator]") {
table_repository repo;
auto columns = column_generator::generate<matador::test::product>(repo);
const std::vector<column> expected_columns = {
column{ "product_name", data_type_t::type_varchar, { constraints::PRIMARY_KEY | constraints::NOT_NULL } },
column{ "supplier_id", data_type_t::type_unsigned_long, constraints::FOREIGN_KEY },
column{ "category_id", data_type_t::type_unsigned_long, constraints::FOREIGN_KEY },
column{ "quantity_per_unit", data_type_t::type_varchar, constraints::NOT_NULL },
column{ "unit_price", data_type_t::type_unsigned_int, constraints::NOT_NULL },
column{ "units_in_stock", data_type_t::type_unsigned_int, constraints::NOT_NULL },
column{ "units_in_order", data_type_t::type_unsigned_int, constraints::NOT_NULL },
column{ "reorder_level", data_type_t::type_unsigned_int, constraints::NOT_NULL },
column{ "discontinued", data_type_t::type_bool, constraints::NOT_NULL }
const std::vector<std::string> expected_columns = {
"product_name",
"supplier_id",
"category_id",
"quantity_per_unit",
"unit_price",
"units_in_stock",
"units_in_order",
"reorder_level",
"discontinued"
};
REQUIRE(!columns.empty());
REQUIRE(columns.size() == expected_columns.size());
for (size_t i = 0; i != expected_columns.size(); ++i) {
REQUIRE(expected_columns[i].name() == columns[i].name());
REQUIRE(expected_columns[i].attributes().options() == columns[i].attributes().options() );
REQUIRE(expected_columns[i].type() == columns[i].type() );
}
}
TEST_CASE("Generate columns from object with nullable columns", "[column generator]") {
table_repository repo;
auto columns = column_generator::generate<matador::test::optional>(repo);
const std::vector<column> expected_columns = {
column{ "id", data_type_t::type_unsigned_long, { constraints::PRIMARY_KEY | constraints::NOT_NULL } },
column{ "name", data_type_t::type_varchar, null_attributes },
column{ "age", data_type_t::type_unsigned_int, null_attributes }
};
REQUIRE(!columns.empty());
REQUIRE(columns.size() == expected_columns.size());
for (size_t i = 0; i != expected_columns.size(); ++i) {
REQUIRE(expected_columns[i].name() == columns[i].name());
REQUIRE(expected_columns[i].attributes().options() == columns[i].attributes().options() );
REQUIRE(expected_columns[i].type() == columns[i].type() );
REQUIRE(expected_columns[i] == columns[i].name());
}
}

72
test/SessionTest.cpp
View File

@ -1,12 +1,10 @@
#include <catch2/catch_test_macros.hpp>
#include <catch2/catch_template_test_macros.hpp>
#include <catch2/generators/catch_generators.hpp>
#include <matador/sql/column.hpp>
#include <matador/sql/condition.hpp>
#include <matador/sql/session.hpp>
#include "Databases.hpp"
#include "models/product.hpp"
#include "models/airplane.hpp"
#include "models/flight.hpp"
@ -15,34 +13,23 @@
using namespace matador::sql;
using namespace matador::test;
template<class Type>
class SessionTestFixture
{
public:
SessionTestFixture()
: pool_(Type::dns, 4), session_(pool_)
{}
~SessionTestFixture() = default;
TEST_CASE("Create table with foreign key relation", "[session]") {
auto dns = GENERATE(as<std::string>{},
"sqlite://sqlite.db",
"postgres://test:test123@127.0.0.1:5432/matador_test" );
matador::sql::session &session()
{ return session_; }
private:
matador::sql::connection_pool<matador::sql::connection> pool_;
matador::sql::session session_;
};
TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Create table with foreign key relation", "[session]", Sqlite, Postgres) {
auto &s = SessionTestFixture<TestType>::session();
connection_pool<connection> pool(dns, 4);
session s(pool);
s.create()
.template table<airplane>("airplane")
.table<airplane>("airplane")
.execute();
REQUIRE(s.table_exists("airplane"));
s.create()
.template table<flight>("flight")
.table<flight>("flight")
.execute();
REQUIRE(s.table_exists("flight"));
@ -54,11 +41,12 @@ TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Create table with foreign key rel
REQUIRE(!s.table_exists("airplane"));
}
TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Execute select statement with where clause", "[session]", Postgres, Sqlite) {
auto &s = SessionTestFixture<TestType>::session();
TEST_CASE("Execute select statement with where clause", "[session]") {
connection_pool<connection> pool("sqlite://sqlite.db", 4);
session s(pool);
s.create()
.template table<person>("person")
.table<person>("person")
.execute();
person george{7, "george", 45};
@ -69,7 +57,7 @@ TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Execute select statement with whe
REQUIRE(res == 1);
// fetch person as record
auto result_record = s.template select<person>()
auto result_record = s.select<person>()
.from("person")
.where("id"_col == 7)
.fetch_all();
@ -78,20 +66,20 @@ TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Execute select statement with whe
REQUIRE(i.size() == 3);
REQUIRE(i.at(0).name() == "id");
REQUIRE(i.at(0).type() == data_type_t::type_unsigned_long);
REQUIRE(i.at(0).template as<long long>() == george.id);
REQUIRE(i.at(0).as<long long>() == george.id);
REQUIRE(i.at(1).name() == "name");
REQUIRE(i.at(1).type() == data_type_t::type_varchar);
REQUIRE(i.at(1).template as<std::string>() == george.name);
REQUIRE(i.at(1).as<std::string>() == george.name);
REQUIRE(i.at(2).name() == "age");
REQUIRE(i.at(2).type() == matador::sql::data_type_t::type_unsigned_int);
REQUIRE(i.at(2).template as<long long>() == george.age);
REQUIRE(i.at(2).as<long long>() == george.age);
}
// fetch person as person
auto result_person = s.template select<person>()
auto result_person = s.select<person>()
.from("person")
.where("id"_col == 7)
.template fetch_all<person>();
.fetch_all<person>();
for (const auto& i : result_person) {
REQUIRE(i.id == 7);
@ -102,8 +90,9 @@ TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Execute select statement with whe
s.drop().table("person").execute();
}
TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Execute insert statement", "[session]", Sqlite, Postgres) {
auto &s = SessionTestFixture<TestType>::session();
TEST_CASE("Execute insert statement", "[session]") {
connection_pool<connection> pool("sqlite://sqlite.db", 4);
session s(pool);
s.create()
.table("person", {
@ -123,15 +112,16 @@ TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Execute insert statement", "[sess
s.drop().table("person").execute();
}
TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Select statement with foreign key", "[session]", Sqlite, Postgres) {
auto &s = SessionTestFixture<TestType>::session();
TEST_CASE("Select statement with foreign key", "[session]") {
connection_pool<connection> pool("sqlite://sqlite.db", 4);
session s(pool);
s.create()
.template table<airplane>("airplane")
.table<airplane>("airplane")
.execute();
s.create()
.template table<flight>("flight")
.table<flight>("flight")
.execute();
std::vector<entity<airplane>> planes {
@ -141,19 +131,19 @@ TEMPLATE_TEST_CASE_METHOD(SessionTestFixture, "Select statement with foreign key
};
for (const auto &plane : planes) {
auto res = s.insert().template into<airplane>("airplane").values(*plane).execute();
auto res = s.insert().into<airplane>("airplane").values(*plane).execute();
REQUIRE(res == 1);
}
auto count = s.select({count_all()}).from("airplane").template fetch_value<int>();
auto count = s.select({count_all()}).from("airplane").fetch_value<int>();
REQUIRE(count == 3);
flight f4711{4, planes.at(1), "hans"};
auto res = s.insert().template into<flight>("flight").values(f4711).execute();
auto res = s.insert().into<flight>("flight").values(f4711).execute();
REQUIRE(res == 1);
auto f = *s.template select<flight>().from("flight").template fetch_all<flight>().begin();
auto f = *s.select<flight>().from("flight").fetch_all<flight>().begin();
REQUIRE(f.id == 4);
REQUIRE(f.pilot_name == "hans");
REQUIRE(f.airplane.get() != nullptr);