Skip to content

Commit 89d1eba

Browse files
dotnwatdotnwat
authored
Merge pull request #23552 from dotnwat/bazel-test-backfill
cluster: bazel test backfill
2 parents 8a9060e + ba1efaa commit 89d1eba

File tree

5 files changed

+178
-9
lines changed

5 files changed

+178
-9
lines changed

‎src/v/cluster/tests/BUILD

+165
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,17 @@ redpanda_test_cc_library(
1010
],
1111
)
1212

13+
redpanda_test_cc_library(
14+
name = "leader_balancer_test_utils",
15+
hdrs = [
16+
"leader_balancer_constraints_utils.h",
17+
"leader_balancer_test_utils.h",
18+
],
19+
include_prefix = "cluster/tests",
20+
deps = [
21+
],
22+
)
23+
1324
redpanda_test_cc_library(
1425
name = "topic_table_fixture",
1526
hdrs = [
@@ -80,3 +91,157 @@ redpanda_cc_gtest(
8091
"@seastar//:testing",
8192
],
8293
)
94+
95+
redpanda_cc_btest(
96+
name = "leader_balancer_test",
97+
timeout = "short",
98+
srcs = [
99+
"leader_balancer_test.cc",
100+
],
101+
deps = [
102+
":leader_balancer_test_utils",
103+
"//src/v/cluster",
104+
"//src/v/model",
105+
"//src/v/test_utils:seastar_boost",
106+
"@abseil-cpp//absl/container:flat_hash_map",
107+
"@abseil-cpp//absl/container:flat_hash_set",
108+
"@boost//:test",
109+
],
110+
)
111+
112+
redpanda_cc_btest(
113+
name = "leader_balancer_constraints_test",
114+
timeout = "short",
115+
srcs = [
116+
"leader_balancer_constraints_test.cc",
117+
],
118+
deps = [
119+
":leader_balancer_test_utils",
120+
"//src/v/cluster",
121+
"//src/v/model",
122+
"//src/v/raft",
123+
"//src/v/test_utils:seastar_boost",
124+
"@abseil-cpp//absl/container:flat_hash_map",
125+
"@abseil-cpp//absl/container:flat_hash_set",
126+
"@boost//:test",
127+
],
128+
)
129+
130+
redpanda_cc_btest(
131+
name = "metrics_reporter_test",
132+
timeout = "short",
133+
srcs = [
134+
"metrics_reporter_test.cc",
135+
],
136+
deps = [
137+
"//src/v/cluster",
138+
"//src/v/test_utils:seastar_boost",
139+
"@boost//:test",
140+
],
141+
)
142+
143+
redpanda_cc_gtest(
144+
name = "plugin_table_test",
145+
timeout = "short",
146+
srcs = [
147+
"plugin_table_test.cc",
148+
],
149+
deps = [
150+
"//src/v/cluster",
151+
"//src/v/model",
152+
"//src/v/test_utils:gtest",
153+
"//src/v/utils:uuid",
154+
"@googletest//:gtest",
155+
"@seastar",
156+
],
157+
)
158+
159+
redpanda_cc_gtest(
160+
name = "plugin_frontend_validation_test",
161+
timeout = "short",
162+
srcs = [
163+
"plugin_frontend_validation_test.cc",
164+
],
165+
deps = [
166+
"//src/v/cluster",
167+
"//src/v/model",
168+
"//src/v/raft",
169+
"//src/v/test_utils:gtest",
170+
"//src/v/utils:uuid",
171+
"@abseil-cpp//absl/container:flat_hash_map",
172+
"@boost//:range",
173+
"@googletest//:gtest",
174+
"@seastar",
175+
],
176+
)
177+
178+
redpanda_cc_gtest(
179+
name = "namespaced_cache_test",
180+
timeout = "short",
181+
srcs = [
182+
"namespaced_cache_test.cc",
183+
],
184+
deps = [
185+
"//src/v/cluster",
186+
"//src/v/config",
187+
"//src/v/container:intrusive",
188+
"//src/v/test_utils:gtest",
189+
"@abseil-cpp//absl/container:flat_hash_map",
190+
"@googletest//:gtest",
191+
"@seastar",
192+
],
193+
)
194+
195+
redpanda_cc_gtest(
196+
name = "cluster_recovery_table_test",
197+
timeout = "short",
198+
srcs = [
199+
"cluster_recovery_table_test.cc",
200+
],
201+
deps = [
202+
"//src/v/cluster",
203+
"//src/v/test_utils:gtest",
204+
"@googletest//:gtest",
205+
"@seastar",
206+
],
207+
)
208+
209+
redpanda_cc_gtest(
210+
name = "partition_leaders_table_test",
211+
timeout = "short",
212+
srcs = [
213+
"partition_leaders_table_test.cc",
214+
],
215+
deps = [
216+
"//src/v/cluster",
217+
"//src/v/model",
218+
"//src/v/raft:fundamental",
219+
"//src/v/random:generators",
220+
"//src/v/test_utils:gtest",
221+
"//src/v/test_utils:random",
222+
"@abseil-cpp//absl/container:flat_hash_map",
223+
"@fmt",
224+
"@googletest//:gtest",
225+
"@seastar",
226+
],
227+
)
228+
229+
redpanda_cc_gtest(
230+
name = "shard_placement_table_test",
231+
timeout = "moderate",
232+
srcs = [
233+
"shard_placement_table_test.cc",
234+
],
235+
deps = [
236+
"//src/v/cluster",
237+
"//src/v/container:chunked_hash_map",
238+
"//src/v/features",
239+
"//src/v/ssx:event",
240+
"//src/v/storage",
241+
"//src/v/storage:resources",
242+
"//src/v/test_utils:gtest",
243+
"//src/v/test_utils:random",
244+
"//src/v/utils:prefix_logger",
245+
"@seastar",
246+
],
247+
)

‎src/v/cluster/tests/leader_balancer_test_utils.h

+3-3
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@ namespace leader_balancer_test_utils {
1919
*/
2020
static cluster::leader_balancer_strategy::index_type make_cluster_index(
2121
int node_count,
22-
int shards_per_node,
23-
int groups_per_shard,
24-
int replica_count) {
22+
unsigned shards_per_node,
23+
unsigned groups_per_shard,
24+
unsigned replica_count) {
2525
cluster::leader_balancer_strategy::index_type index;
2626

2727
std::vector<model::broker_shard> shards;

‎src/v/cluster/tests/partition_leaders_table_test.cc

+4-4
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,12 @@
1010
*/
1111

1212
#include "cluster/commands.h"
13+
#include "cluster/data_migrated_resources.h"
1314
#include "cluster/logger.h"
1415
#include "cluster/partition_leaders_table.h"
1516
#include "cluster/producer_state.h"
1617
#include "cluster/topic_table.h"
1718
#include "cluster/types.h"
18-
#include "data_migrated_resources.h"
1919
#include "gmock/gmock.h"
2020
#include "model/fundamental.h"
2121
#include "model/metadata.h"
@@ -53,10 +53,10 @@ struct test_fixture : public seastar_test {
5353
ss::future<size_t> count_leaderless_partitions() {
5454
size_t leaderless_cnt = 0;
5555
co_await leaders.for_each_leader([&leaderless_cnt](
56-
model::topic_namespace_view tp_ns,
57-
model::partition_id pid,
56+
model::topic_namespace_view,
57+
model::partition_id,
5858
std::optional<model::node_id> leader,
59-
model::term_id term) {
59+
model::term_id) {
6060
if (!leader) {
6161
leaderless_cnt++;
6262
}

‎src/v/cluster/tests/plugin_frontend_validation_test.cc

+2-2
Original file line numberDiff line numberDiff line change
@@ -55,15 +55,15 @@ struct transform_config {
5555

5656
ss::sstring make_string(size_t size) {
5757
ss::sstring str;
58-
for (int i = 0; i < size; ++i) {
58+
for (size_t i = 0; i < size; ++i) {
5959
str.append("a", 1);
6060
}
6161
return str;
6262
}
6363

6464
absl::flat_hash_map<ss::sstring, ss::sstring> make_env_map(size_t size) {
6565
absl::flat_hash_map<ss::sstring, ss::sstring> env;
66-
for (int i = 0; i < size; ++i) {
66+
for (size_t i = 0; i < size; ++i) {
6767
env.insert({ss::format("var_{}", i), "bar"});
6868
}
6969
return env;

‎src/v/storage/BUILD

+4
Original file line numberDiff line numberDiff line change
@@ -207,6 +207,10 @@ redpanda_cc_library(
207207
"@seastar",
208208
],
209209
include_prefix = "storage",
210+
visibility = [
211+
"//src/v/cluster/tests:__pkg__",
212+
"//src/v/storage/tests:__pkg__",
213+
],
210214
deps = [
211215
"//src/v/base",
212216
"//src/v/config",

0 commit comments

Comments
 (0)