Skip to content

Fixes persistency test_scenarios integration in reference_integration #228

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
rmaddikery wants to merge 6 commits into from
Closed

Fixes persistency test_scenarios integration in reference_integration #228

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
e6e6490
Removes deprecated feature flag in mw/log
rmaddikery Jan 22, 2026
7e14ecb
Fix mw/log target deps while importing logging
rmaddikery Jan 22, 2026
ea41063
Fix: Use v1.10.2 for aspect_rules_lint
rmaddikery Feb 10, 2026
eede0ab
Fixes starlark buildifier formatting
rmaddikery Feb 11, 2026
c26079f
Fixes documentation generation for score_process dep
rmaddikery Feb 11, 2026
600468f
Fix module deps after baselibs new release 0.2.4
rmaddikery Feb 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .bazelrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ build --java_runtime_version=remotejdk_17
build --tool_java_runtime_version=remotejdk_17

build --@score_baselibs//score/json:base_library=nlohmann
build --@score_baselibs//score/mw/log/flags:KRemote_Logging=False

# Clippy linting (enabled by default)
build --aspects=@score_rust_policies//clippy:linters.bzl%clippy_strict
Expand Down
11 changes: 9 additions & 2 deletions MODULE.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,18 +77,25 @@ bazel_dep(name = "google_benchmark", version = "1.9.4", dev_dependency = True)
bazel_dep(name = "platforms", version = "1.0.0")

## S-CORE bazel registry
bazel_dep(name = "score_baselibs", version = "0.1.2")
bazel_dep(name = "score_baselibs", version = "0.2.4")

bazel_dep(name = "score_bazel_platforms", version = "0.0.4")

# Override score_docs_as_code to resolve compatibility level conflicts between dependencies
bazel_dep(name = "score_docs_as_code", version = "3.0.0", dev_dependency = True)
single_version_override(
Copy link
Contributor

@vinodreddy-g vinodreddy-g Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we need single_version_override here?

Copy link
Contributor

@PiotrKorkus PiotrKorkus Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

moreover it cannot be used. The only place where it should be used is reference_integration repo

module_name = "score_docs_as_code",
version = "3.0.0",
)

bazel_dep(name = "score_platform", version = "0.5.3", dev_dependency = True)
bazel_dep(name = "score_process", version = "1.4.0", dev_dependency = True)

bazel_dep(name = "score_python_basics", version = "0.3.4")
bazel_dep(name = "score_tooling", version = "1.1.0")

# ToDo: implicit dependencies for score_tooling, but needed directly here??
bazel_dep(name = "aspect_rules_lint", version = "2.0.0")
bazel_dep(name = "aspect_rules_lint", version = "1.10.2")
Copy link
Contributor

@vinodreddy-g vinodreddy-g Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why was the version downgraded?

bazel_dep(name = "buildifier_prebuilt", version = "8.2.0.2")

## temporary overrides / tools
Expand Down
2 changes: 1 addition & 1 deletion src/cpp/src/BUILD
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ cc_library(
"//src/cpp/src/internal:error",
"@score_baselibs//score/filesystem",
"@score_baselibs//score/json",
"@score_baselibs//score/mw/log",
"@score_baselibs//score/mw/log:frontend",
"@score_baselibs//score/result",
],
)
2 changes: 2 additions & 0 deletions src/cpp/tests/BUILD
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ cc_test(
"@score_baselibs//score/filesystem",
"@score_baselibs//score/filesystem:mock",
"@score_baselibs//score/json:mock",
"@score_baselibs//score/mw/log:backend_stub_testutil",
"@score_baselibs//score/result",
],
)
Expand All @@ -47,6 +48,7 @@ cc_test(
"@google_benchmark//:benchmark",
"@score_baselibs//score/filesystem",
"@score_baselibs//score/json",
"@score_baselibs//score/mw/log:backend_stub_testutil",
"@score_baselibs//score/result",
],
)
1 change: 1 addition & 0 deletions tests/test_scenarios/cpp/BUILD
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ cc_binary(
visibility = ["//visibility:public"],
deps = [
"//src/cpp/src:kvs_cpp",
"@score_baselibs//score/mw/log:console_only_backend",
"@score_test_scenarios//test_scenarios_cpp",
],
)
2 changes: 1 addition & 1 deletion tests/test_scenarios/cpp/src/cit/snapshots.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ class SnapshotMaxCount : public Scenario
void run(const std::string& input) const final
{
auto obj{get_object(input)};
auto count{get_field<int32_t>(obj, "count")};
[[maybe_unused]] auto count{get_field<int32_t>(obj, "count")};
auto params{KvsParameters::from_json(input)};

auto kvs{kvs_instance(params)};
Expand Down