Skip to content

chore: update score_test_scenarios deps to score/ paths - #512

Merged
umaucher merged 2 commits into
eclipse-score:mainfrom
qorix-group:Subramanian-K812_use_score_test_scenarios_paths
Oct 16, 2026
Merged

umaucher merged 2 commits into
eclipse-score:mainfrom
qorix-group:Subramanian-K812_use_score_test_scenarios_paths

Conversation

@Subramanian-K812

Copy link
Copy Markdown
Contributor

Summary

Upstream PR eclipse-score/testing_tools#31
("Move test scenario libraries under score/") relocated the test scenario libraries:

  • test_scenarios_cpp/ → score/test_scenarios_cpp/
  • test_scenarios_rust/ → score/test_scenarios_rust/

This aligns score_test_scenarios with the score/ layout used by all other S-CORE modules.
This PR updates the two persistency BUILD files that depend on these targets to reference the
new paths.

What's changed

File Change
score/kvs/tests/test_scenarios/BUILD @score_test_scenarios//test_scenarios_cpp → @score_test_scenarios//score/test_scenarios_cpp
score/kvs/rust_kvs/tests/test_scenarios/BUILD @score_test_scenarios//test_scenarios_rust → @score_test_scenarios//score/test_scenarios_rust

Context

Once this PR is merged, eclipse-score/reference_integration can bump score_test_scenarios
to the PR-31 merge commit (c16400a3) and score_persistency to this commit cleanly,
without requiring a local patch override.

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Starting local Bazel server (8.7.0) and connecting to it...
INFO: Invocation ID: 00ce2053-221f-4de1-aacc-f3a926cb6ee0
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 2 packages loaded
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Loading: 2 packages loaded
    currently loading: 
Analyzing: target //:license-check (3 packages loaded, 0 targets configured)
Analyzing: target //:license-check (3 packages loaded, 0 targets configured)

Analyzing: target //:license-check (34 packages loaded, 14 targets configured)

Analyzing: target //:license-check (91 packages loaded, 14 targets configured)

Analyzing: target //:license-check (140 packages loaded, 1705 targets configured)

Analyzing: target //:license-check (143 packages loaded, 3234 targets configured)

Analyzing: target //:license-check (143 packages loaded, 3234 targets configured)

Analyzing: target //:license-check (143 packages loaded, 3234 targets configured)

Analyzing: target //:license-check (143 packages loaded, 3234 targets configured)

Analyzing: target //:license-check (143 packages loaded, 3234 targets configured)

Analyzing: target //:license-check (155 packages loaded, 6447 targets configured)

Analyzing: target //:license-check (157 packages loaded, 8985 targets configured)

Analyzing: target //:license-check (160 packages loaded, 9664 targets configured)

Analyzing: target //:license-check (160 packages loaded, 9664 targets configured)

Analyzing: target //:license-check (160 packages loaded, 9664 targets configured)

Analyzing: target //:license-check (160 packages loaded, 9664 targets configured)

Analyzing: target //:license-check (166 packages loaded, 15647 targets configured)

Analyzing: target //:license-check (167 packages loaded, 15771 targets configured)

INFO: Analyzed target //:license-check (168 packages loaded, 15897 targets configured).
[12 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions, 1 running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 50 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
[15 / 16] Building license.check.license_check.jar (); 0s disk-cache, multiplex-worker
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 40.720s, Critical Path: 2.96s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
[main] INFO Querying Eclipse Foundation for license data for 42 items.
[main] INFO Found 32 items.
[main] INFO Querying ClearlyDefined for license data for 10 items.
[main] INFO Found 10 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO crate/cratesio/-/lazy_static/1.5.1
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.

@umaucher

Copy link
Copy Markdown
Contributor

to use the new features from eclipse-score/testing_tools#31 you need to update https://github.com/eclipse-score/persistency/blob/main/MODULE.bazel#L29 to a version providing it

@Subramanian-K812

Subramanian-K812 commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor Author

@umaucher There is currently no registry version of score_test_scenarios providing the score/ paths. eclipse-score/testing_tools#31 merged to eclipse-score/testing_tools at commit c16400a3 but no new registry version (e.g., v0.5.0) has been published. Used git_override (same pattern as score_tools in this file) as an interim solution — will replace with a version bump once the registry release is available.

@Subramanian-K812
Subramanian-K812 force-pushed the Subramanian-K812_use_score_test_scenarios_paths branch from 7737d31 to 6412d78 Compare October 16, 2026 04:49
@Subramanian-K812
Subramanian-K812 deployed to workflow-approval October 16, 2026 04:49 — with GitHub Actions Active
@Subramanian-K812
Subramanian-K812 deployed to workflow-approval October 16, 2026 04:49 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-512: https://eclipse-score.github.io/persistency/pr-512/

@umaucher

Copy link
Copy Markdown
Contributor

when "touching" the MODULE.bazel file you also need to ensure to update the according *.lock file bazel mod tidy or bazel mod deps --lockfile_mode=update should do the trick

@Subramanian-K812
Subramanian-K812 force-pushed the Subramanian-K812_use_score_test_scenarios_paths branch from 6412d78 to 5ca5d01 Compare October 16, 2026 08:49
@Subramanian-K812
Subramanian-K812 force-pushed the Subramanian-K812_use_score_test_scenarios_paths branch from 5ca5d01 to 3f71e20 Compare October 16, 2026 08:49
@Subramanian-K812
Subramanian-K812 deployed to workflow-approval October 16, 2026 08:50 — with GitHub Actions Active
@Subramanian-K812
Subramanian-K812 deployed to workflow-approval October 16, 2026 08:50 — with GitHub Actions Active
@umaucher
umaucher added this pull request to the merge queue Oct 16, 2026
Merged via the queue into eclipse-score:main with commit 86ac1e9 Oct 16, 2026
19 checks passed

This branch was successfully deployed

1 active deployment
workflow-approval — 3f71e20f Deployed Oct 16, 2026 by Subramanian-K812 via qnx-build (per-x86_64-qnx) / approval #950
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants