Skip to content

[score/time] Improve unit test coverage - #146

Open
lavrovvalera wants to merge 1 commit into
eclipse-score:mainfrom
lavrovvalera:vala_improve_time_coverage_up_to_90
Open

[score/time] Improve unit test coverage#146
lavrovvalera wants to merge 1 commit into
eclipse-score:mainfrom
lavrovvalera:vala_improve_time_coverage_up_to_90

Conversation

@lavrovvalera

Copy link
Copy Markdown
Contributor
  • Add missing tests for score/time
  • Rename mock-based clock tests to *_adapter_test to clarify scope
  • Update .bazelrc coverage config

Closes #44

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

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.6.0) and connecting to it...
INFO: Invocation ID: 3a73a913-7d53-4206-b479-be63848aa5d3
Computing main repo mapping: 
WARNING: /home/runner/work/time/time/MODULE.bazel:13:7: The attribute 'compatibility_level' in module() is a no-op and will be removed in a future Bazel release. Please remove it from your MODULE.bazel file.
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 3 packages loaded
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Loading: 3 packages loaded
    currently loading: 
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)

Analyzing: target //:license-check (38 packages loaded, 10 targets configured)

Analyzing: target //:license-check (87 packages loaded, 10 targets configured)

Analyzing: target //:license-check (89 packages loaded, 10 targets configured)

Analyzing: target //:license-check (139 packages loaded, 460 targets configured)

Analyzing: target //:license-check (153 packages loaded, 3442 targets configured)

Analyzing: target //:license-check (153 packages loaded, 6931 targets configured)

Analyzing: target //:license-check (154 packages loaded, 9141 targets configured)

Analyzing: target //:license-check (165 packages loaded, 9284 targets configured)

Analyzing: target //:license-check (165 packages loaded, 9284 targets configured)

Analyzing: target //:license-check (165 packages loaded, 9284 targets configured)

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

Analyzing: target //:license-check (169 packages loaded, 11296 targets configured)

INFO: Analyzed target //:license-check (170 packages loaded, 11422 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 2 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: 30.762s, Critical Path: 2.55s
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>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@lavrovvalera
lavrovvalera force-pushed the vala_improve_time_coverage_up_to_90 branch from 3df2a98 to 773a93c Compare August 3, 2026 14:32
- Add missing tests for score/time
- Rename mock-based clock tests to *_adapter_test to clarify scope
- Update .bazelrc coverage config
@pawelrutkaq
pawelrutkaq force-pushed the vala_improve_time_coverage_up_to_90 branch from 773a93c to 69ec4b7 Compare August 7, 2026 08:16
@pawelrutkaq
pawelrutkaq deployed to workflow-approval August 7, 2026 08:16 — with GitHub Actions Active
@pawelrutkaq
pawelrutkaq temporarily deployed to workflow-approval August 7, 2026 08:16 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Improvement: Increase unit test coverage to 90% line + branch: score/time/ API

2 participants