Skip to content

Commit 40fe82b

Browse files
committed
cpp impl: Update BUILD Files and Workflow
Update impl and tests BUILD Files and Workflow
1 parent ebcf0ee commit 40fe82b

File tree

3 files changed

+7
-13
lines changed

3 files changed

+7
-13
lines changed

.github/workflows/check.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@ jobs:
7070
run: |
7171
bazel coverage //:test_kvs_cpp \
7272
--collect_code_coverage \
73-
--instrument_test_targets \
7473
--combined_report=lcov \
7574
--experimental_generate_llvm_lcov \
7675
--nocache_test_results \

src/cpp/BUILD

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,11 @@
1515

1616
cc_library(
1717
name = "kvs_cpp",
18-
srcs = glob(["src/*.cpp"]),
19-
hdrs = glob([
20-
"inc/**/*.hpp",
21-
]),
18+
srcs = [
19+
"inc/internal/kvs_helper.hpp",
20+
"src/kvs.cpp",
21+
],
22+
hdrs = ["inc/kvs.hpp"],
2223
includes = ["inc"],
2324
visibility = ["//:__pkg__"],
2425
deps = [

src/cpp/tests/BUILD

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ cc_test(
3232
"@googletest//:gtest_main",
3333
"@score-baselibs//score/filesystem:filesystem",
3434
"@score-baselibs//score/filesystem:mock",
35-
"@score-baselibs//score/json",
3635
"@score-baselibs//score/json:mock",
37-
"@score-baselibs//score/memory:string_comparison_adaptor",
3836
"@score-baselibs//score/result:result",
3937
],
4038
)
@@ -44,17 +42,13 @@ cc_test(
4442
size = "small",
4543
srcs = [
4644
"bm_kvs.cpp",
47-
"//src/cpp:kvs_srcs",
48-
],
49-
copts = [
50-
"-Isrc/cpp/inc",
5145
],
5246
visibility = ["//:__pkg__"],
5347
deps = [
48+
"//:kvs_cpp",
5449
"@google_benchmark//:benchmark",
5550
"@score-baselibs//score/filesystem:filesystem",
56-
"@score-baselibs//score/json:interface",
57-
"@score-baselibs//score/json:json_parser",
51+
"@score-baselibs//score/json",
5852
"@score-baselibs//score/result:result",
5953
],
6054
)

0 commit comments

Comments
 (0)