mirror of
https://github.com/scylladb/scylladb.git
synced 2026-05-31 03:56:42 +00:00
f165b396fd
A recent Seastar update deprecated smp::count and introduced this_smp_shard_count() as a replacement. One difference is that this_smp_shard_count() wants to run on a reactor thread. This poses a problem for non-reactor tests (BOOST_AUTO_TEST_CASE) that nevertheless use a schema, as the schema_builder constructor references smp::count. If we replace it with this_smp_shard_count() then it will crash when running without a reactor. To fix, remove the implicit this_smp_shard_count() call from raw_schema's constructor and require callers to pass shard_count explicitly to schema_builder. This allows tests that don't run on a reactor thread to construct schemas without crashing. Production code and reactor-based tests pass this_smp_shard_count(). Non-reactor test files (expr_test, keys_test, nonwrapping_interval_test, wrapping_interval_test, bti_key_translation_test, range_tombstone_list_test) pass a fixed shard count of 1. Note: sstable_test.cc is a Seastar test file (SEASTAR_THREAD_TEST_CASE) but also contains one plain BOOST_AUTO_TEST_CASE (test_empty_key_view_comparison) that constructs a schema_builder without a reactor context. This test also receives a fixed shard count of 1.
115 lines
3.9 KiB
C++
115 lines
3.9 KiB
C++
/*
|
|
* Copyright (C) 2021-present ScyllaDB
|
|
*
|
|
* Modified by ScyllaDB
|
|
*/
|
|
|
|
/*
|
|
* SPDX-License-Identifier: LicenseRef-ScyllaDB-Source-Available-1.1
|
|
*/
|
|
|
|
#undef SEASTAR_TESTING_MAIN
|
|
#include <seastar/testing/test_case.hh>
|
|
#include <seastar/testing/thread_test_case.hh>
|
|
#include "test/lib/cql_test_env.hh"
|
|
#include "db/virtual_table.hh"
|
|
#include "db/system_keyspace.hh"
|
|
#include "db/config.hh"
|
|
#include "schema/schema_builder.hh"
|
|
#include "test/lib/cql_assertions.hh"
|
|
|
|
namespace db {
|
|
|
|
class test_table : public virtual_table {
|
|
public:
|
|
test_table() : virtual_table(build_schema()) {}
|
|
|
|
static schema_ptr build_schema() {
|
|
auto id = generate_legacy_id(system_keyspace::NAME, "test");
|
|
return schema_builder(this_smp_shard_count(), system_keyspace::NAME, "test", std::make_optional(id))
|
|
.with_column("pk", int32_type, column_kind::partition_key)
|
|
.with_column("ck", int32_type, column_kind::clustering_key)
|
|
.with_column("v", int32_type)
|
|
.build();
|
|
}
|
|
|
|
mutation_source as_mutation_source() override {
|
|
throw std::runtime_error("Not implemented");
|
|
}
|
|
|
|
void test_set_cell() {
|
|
mutation m(_s, partition_key::from_single_value(*_s, data_value(666).serialize_nonnull()));
|
|
row& cr = m.partition().clustered_row(*_s, clustering_key::from_single_value(*_s, data_value(10).serialize_nonnull())).cells();
|
|
|
|
set_cell(cr, "v", 8);
|
|
|
|
auto result_cell = cr.cell_at(0).as_atomic_cell(column_definition("v", int32_type, column_kind::regular_column));
|
|
auto result = result_cell.serialize();
|
|
|
|
BOOST_REQUIRE(result[result.size() - 1] == 8);
|
|
|
|
BOOST_CHECK_THROW(set_cell(cr, "nonexistent_column", 20), std::runtime_error);
|
|
}
|
|
};
|
|
|
|
}
|
|
|
|
BOOST_AUTO_TEST_SUITE(virtual_table_test)
|
|
|
|
SEASTAR_TEST_CASE(test_set_cell) {
|
|
auto table = db::test_table();
|
|
table.test_set_cell();
|
|
|
|
return make_ready_future<>();
|
|
}
|
|
|
|
SEASTAR_THREAD_TEST_CASE(test_system_config_table_read) {
|
|
do_with_cql_env_thread([] (cql_test_env& env) {
|
|
auto res = env.execute_cql("SELECT * FROM system.config WHERE name = 'partitioner';").get();
|
|
assert_that(res).is_rows().with_size(1).with_row({
|
|
{ utf8_type->decompose(sstring("partitioner")) },
|
|
{ utf8_type->decompose(sstring("default")) },
|
|
{ utf8_type->decompose(sstring("string")) },
|
|
{ utf8_type->decompose(format("\"{}\"", env.local_db().get_config().partitioner())) }
|
|
});
|
|
}).get();
|
|
}
|
|
|
|
SEASTAR_THREAD_TEST_CASE(test_system_config_table_update) {
|
|
if (smp::count < 2) {
|
|
fmt::print("This test should be run with at least 2 CPUs\n");
|
|
return;
|
|
}
|
|
|
|
do_with_cql_env_thread([] (cql_test_env& env) {
|
|
auto value = env.local_db().get_config().failure_detector_timeout_in_ms();
|
|
smp::invoke_on_others([&env, value] {
|
|
BOOST_REQUIRE_EQUAL(env.local_db().get_config().failure_detector_timeout_in_ms(), value);
|
|
}).get();
|
|
|
|
env.execute_cql(format("UPDATE system.config SET value = '{}' WHERE name = 'failure_detector_timeout_in_ms';", value + 10)).get();
|
|
|
|
smp::invoke_on_others([&env, value] {
|
|
BOOST_REQUIRE_EQUAL(env.local_db().get_config().failure_detector_timeout_in_ms(), value + 10);
|
|
}).get();
|
|
}).get();
|
|
}
|
|
|
|
SEASTAR_THREAD_TEST_CASE(test_system_config_table_set_empty) {
|
|
do_with_cql_env_thread([] (cql_test_env& env) {
|
|
env.execute_cql(format("UPDATE system.config SET value = '' WHERE name = 'allowed_repair_based_node_ops';")).get();
|
|
BOOST_REQUIRE_EQUAL(env.local_db().get_config().allowed_repair_based_node_ops(), sstring());
|
|
}).get();
|
|
}
|
|
|
|
SEASTAR_THREAD_TEST_CASE(test_system_config_table_no_live_update) {
|
|
do_with_cql_env_thread([] (cql_test_env& env) {
|
|
BOOST_REQUIRE_THROW(
|
|
env.execute_cql("UPDATE system.config SET value = 'foo' WHERE name = 'cluster_name';").get(),
|
|
exceptions::mutation_write_failure_exception
|
|
);
|
|
}).get();
|
|
}
|
|
|
|
BOOST_AUTO_TEST_SUITE_END()
|