8000 kvs: add unit tests for safety measurement by sbachmann-qorix · Pull Request #13 · eclipse-score/inc_mw_per · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

kvs: add unit tests for safety measurement #13

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

Jump to bottom
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

kvs: add unit tests for safety measurement #13

wants to merge 7 commits into from

Conversation

sbachmann-qorix
Copy link
Contributor

No description provided.

@sbachmann-qorix
kvs: add unit tests, allow to set the storage dir and remove public f…
b676717 
…rom internal KVS elements

Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix
devenv: update lock-file
5426d82 
Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix sbachmann-qorix self-assigned this May 16, 2025
@github-actions GitHub Actions
Copy link
github-actions bot commented May 16, 2025 •
edited
Loading

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.2.1) and connecting to it...
INFO: Invocation ID: f96b6289-6de9-435b-8571-2d438b6cf7f1
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'googletest', the root module requires module version googletest@1.14.0, but got googletest@1.14.0.bcr.1 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (48 packages loaded, 9 targets configured)

Analyzing: target //:license-check (81 packages loaded, 179 targets configured)

Analyzing: target //:license-check (109 packages loaded, 1995 targets configured)

Analyzing: target //:license-check (116 packages loaded, 2154 targets configured)

Analyzing: target //:license-check (116 packages loaded, 2154 targets configured)

Analyzing: target //:license-check (122 packages loaded, 4495 targets configured)

Analyzing: target //:license-check (124 packages loaded, 4739 targets configured)

DEBUG: Rule 'toolchains_llvm++llvm+llvm_toolchain_llvm' indicated that a canonical reproducible form can be obtained by modifying arguments _action_listener = <unknown object com.google.devtools.build.lib.packages.Attribute$LabelListLateBoundDefault>, _config_dependencies = [], _configure = False, _environ = [], _original_name = "llvm_toolchain_llvm"
DEBUG: Repository toolchains_llvm++llvm+llvm_toolchain_llvm instantiated at:
  <builtin>: in <toplevel>
Repository rule llvm defined at:
  /home/runner/.bazel/external/toolchains_llvm+/toolchain/rules.bzl:27:23: in <toplevel>
Analyzing: target //:license-check (124 packages loaded, 4739 targets configured)

INFO: Analyzed target //:license-check (125 packages loaded, 6677 targets configured).
[8 / 13] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/score_dash_license_checker+/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local ... (2 actions, 1 running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 15 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
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: 196.980s, Critical Path: 0.77s
INFO: 13 processes: 3 disk cache hit, 9 internal, 1 processwrapper-sandbox.
INFO: Build completed successfully, 13 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 15 items.
[main] INFO Found 5 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO cargo/cargo/-/fuchsia-cprng/0.1.1
[main] INFO cargo/cargo/-/libc/0.2.172
[main] INFO cargo/cargo/-/rand/0.4.6
[main] INFO cargo/cargo/-/rand_core/0.3.1
[main] INFO cargo/cargo/-/rand_core/0.4.2
[main] INFO cargo/cargo/-/remove_dir_all/0.5.3
[main] INFO cargo/cargo/-/rust-kvs/0.1.0
[main] INFO cargo/cargo/-/tempdir/0.3.7
[main] INFO cargo/cargo/-/winapi-x86_64-pc-windows-gnu/0.4.0
[main] INFO cargo/cargo/-/winapi/0.3.9
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.
[main] INFO A review is required for cargo/cargo/-/winapi/0.3.9.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21182 .
[main] INFO A review is required for cargo/cargo/-/fuchsia-cprng/0.1.1.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21183 .
[main] INFO A review is required for cargo/cargo/-/tempdir/0.3.7.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21184 .
[main] INFO A review is required for cargo/cargo/-/remove_dir_all/0.5.3.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21185 .
[main] INFO A review is required for cargo/cargo/-/libc/0.2.172.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21186 .
[main] INFO A review is required for cargo/cargo/-/winapi-x86_64-pc-windows-gnu/0.4.0.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21187 .
[main] INFO A review is required for cargo/cargo/-/rand_core/0.3.1.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21188 .
[main] INFO A review is required for cargo/cargo/-/rand_core/0.4.2.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21189 .
[main] INFO A review is required for cargo/cargo/-/rand/0.4.6.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/21190 .
[main] INFO A review is required for cargo/cargo/-/rust-kvs/0.1.0.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/20866 .

@sbachmann-qorix
kvs: add tests to increase code coverage
3a6e855 
Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
vinodreddy-g
vinodreddy-g reviewed May 20, 2025
View reviewed changes
@sbachmann-qorix
kvs: move tempdir to development dependencies
7668520 
Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix
kvs: fixed clippy lints
de700be 
Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix
kvs: rename converage tests to test
656bbe7 
Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@sbachmann-qorix
kvs: increase region and branch coverage via tests
371c95c 
Command: cargo llvm-cov --html --open --branch --tests

Functions: 100.00% (89/89)
Lines:      96.79% (603/623)
Regions:    96.32% (1047/1087)
Branches:   88.10% (37/42)

Signed-off-by: Sven Bachmann <sven.bachmann.ext@qorix.ai>
@guysagnes guysagnes self-requested a review May 22, 2025 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vinodreddy-g vinodreddy-g vinodreddy-g left review comments

@PandaeDo PandaeDo Awaiting requested review from PandaeDo

@guysagnes guysagnes Awaiting requested review from guysagnes

At least 1 approving review is required to merge this pull request.

Assignees

@sbachmann-qorix sbachmann-qorix

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sbachmann-qorix @vinodreddy-g
0