Skip to content

Feature/azb 101279 quality pack targets fix component tests - #238

Merged
NEOatNHNG merged 43 commits into
eclipse-score:mainfrom
etas-contrib:feature/AZB-101279-Quality-pack-targets
Oct 7, 2026
Merged

NEOatNHNG merged 43 commits into
eclipse-score:mainfrom
etas-contrib:feature/AZB-101279-Quality-pack-targets

Conversation

@GabrielPinheiro7891

@GabrielPinheiro7891 GabrielPinheiro7891 commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Add test suites for unit_tests

GabrielPinheiro7891 and others added 30 commits August 6, 2026 15:27
Remove feature requirements as they belong to eclipse-score/score repo
Co-authored-by: Ryan Steel <str1yok@bosch.com>
Signed-off-by: Gabriel Pinheiro <gabriel.pinheiro@etas.com>
The requirement describes type safety enforcement across time domains,
which is functional behavior rather than an interface contract.
Co-authored-by: Björn Hornburg <Bjoern.Hornburg@de.bosch.com>
Signed-off-by: Gabriel Pinheiro <gabriel.pinheiro@etas.com>
Co-authored-by: Björn Hornburg <Bjoern.Hornburg@de.bosch.com>
Signed-off-by: Gabriel Pinheiro <gabriel.pinheiro@etas.com>
@github-actions

github-actions Bot commented Oct 5, 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: 99092f62-bb6e-445d-9a10-00ccdfa3d55c
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'platforms', the root module requires module version platforms@1.0.0, but got platforms@1.1.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_platform', the root module requires module version score_platform@0.7.1, but got score_platform@0.7.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'rules_oci', the root module requires module version rules_oci@2.2.7, but got rules_oci@2.3.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
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: 3 packages loaded
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/time/time/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/time/time/BUILD
INFO: Elapsed time: 16.504s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

Comment thread BUILD Outdated
Comment thread BUILD
Comment on lines +58 to +61
"//score/time:unit_test_suite_host",
"//score/time_daemon:unit_test_suite_host",
"//score/time_slave:unit_test_suite_host",
"//score/ts_client/src:unit_test_suite_host",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why unit_test_suite_host? I just see the names without "host".

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

_host on the name is added by cc_unit_test_suites_for_host_and_qnx. So thats the correct name:

bazel query '//score/time:unit_test_suite_host'
//score/time:unit_test_suite_host

bazel query '//score/time:unit_test_suite'
ERROR: no such target '//score/time:unit_test_suite': target 'unit_test_suite' not declared in package 'score/time' defined by /home/gpn1brg/test/score_time/score/time/BUILD

Comment thread BUILD Outdated
"//score/time/high_res_steady_time/src:high_res_steady_clock_test",
"//score/time/steady_time/src:steady_clock_test",
"//score/time/system_time/src:system_clock_test",
"//score/time/high_res_steady_time/src:high_res_steady_clock_adapter_test",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Imho the xyz_clock_adapter_tests are unit tests, not component tests.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually all tests are tagged as "unit". Removed completely test_suite component_tests.

It runs now only with unit_tests as it should:

bazel test //:unit_tests
INFO: Analyzed 49 targets (0 packages loaded, 0 targets configured).
INFO: Found 49 test targets...
INFO: Elapsed time: 0.219s, Critical Path: 0.00s
INFO: 1 process: 49 action cache hit, 1 internal.
INFO: Build completed successfully, 1 total action
//score/time/clock/src:clock_status_test (cached) PASSED in 0.2s
//score/time/high_res_steady_time/src:high_res_steady_clock_adapter_test (cached) PASSEDin 0.0s
//score/time/high_res_steady_time/src:high_res_steady_clock_integration_test (cached) PASSED in 0.1s
//score/time/high_res_steady_time/src/details/system_clock:high_res_steady_clock_backend_impl_test (cached) PASSED in 0.1s
//score/time/ptp/src:pdelay_measurement_data_test (cached) PASSED in 0.0s
//score/time/ptp/src:time_slave_sync_data_test (cached) PASSED in 0.0s
//score/time/steady_time/src:steady_clock_adapter_test (cached) PASSED in 0.0s
//score/time/steady_time/src:steady_clock_integration_test (cached) PASSED in 0.1s
//score/time/steady_time/src/details/steady_time_impl:steady_clock_backend_impl_test (cached) PASSED in 0.0s
//score/time/system_time/src:system_clock_adapter_test (cached) PASSED in 0.0s
//score/time/system_time/src:system_clock_integration_test (cached) PASSED in 0.1s
//score/time/system_time/src/details/system_time_impl:system_clock_backend_impl_test (cached) PASSED in 0.1s
//score/time/vehicle_time/src:vehicle_clock_test (cached) PASSED in 0.0s
//score/time/vehicle_time/src:vehicle_time_status_test (cached) PASSED in 0.0s
//score/time/vehicle_time/src/details/td_impl:vehicle_clock_backend_impl_test (cached) PASSED in 0.1s
//score/time_daemon/src/application/job_runner:job_runner_test (cached) PASSED in 3.3s
//score/time_daemon/src/common/data_flow:data_flow_test (cached) PASSED in 0.0s
//score/time_daemon/src/common/data_types:ptp_time_info_test (cached) PASSED in 0.0s
//score/time_daemon/src/common/machines:machines_test (cached) PASSED in 5.5s
//score/time_daemon/src/control_flow_divider/core:control_flow_divider_test (cached) PASSED in 0.8s
//score/time_daemon/src/control_flow_divider/ptp:factory_test (cached) PASSED in 0.2s
//score/time_daemon/src/control_flow_divider/ptp:ptp_control_flow_divider_test (cached) PASSED in 1.2s
//score/time_daemon/src/ipc/core:publisher_impl_test (cached) PASSED in 0.1s
//score/time_daemon/src/ipc/core:receiver_impl_test (cached) PASSED in 0.1s
//score/time_daemon/src/ipc/core:shared_memory_handler_test (cached) PASSED in 0.1s
//score/time_daemon/src/ipc/svt:common_factory_test (cached) PASSED in 0.1s
//score/time_daemon/src/ipc/svt:svt_time_info_test (cached) PASSED in 0.1s
//score/time_daemon/src/msg_broker:msg_broker_test (cached) PASSED in 0.2s
//score/time_daemon/src/ptp_machine/core:ptp_machine_test (cached) PASSED in 0.6s
//score/time_daemon/src/ptp_machine/shm:gptp_shm_machine_test (cached) PASSED in 0.1s
//score/time_daemon/src/ptp_machine/shm/details:shm_ptp_engine_test (cached) PASSED in 0.1s
//score/time_daemon/src/ptp_machine/stub:gptp_stub_machine_test (cached) PASSED in 0.1s
//score/time_daemon/src/verification_machine/core:verification_machine_test (cached) PASSED in 0.1s
//score/time_daemon/src/verification_machine/svt:svt_verification_machine_test (cached) PASSED in 0.1s
//score/time_daemon/src/verification_machine/svt/validators:synchronization_validator_test (cached) PASSED in 0.1s
//score/time_daemon/src/verification_machine/svt/validators:time_jumps_validator_test (cached) PASSED in 0.1s
//score/time_daemon/src/verification_machine/svt/validators:timeout_validator_test (cached) PASSED in 0.1s
//score/time_slave/src/common:definitions_test (cached) PASSED in 0.0s
//score/time_slave/src/gptp:gptp_engine_test (cached) PASSED in 0.5s
//score/time_slave/src/gptp/details:frame_codec_test (cached) PASSED in 0.1s
//score/time_slave/src/gptp/details:message_parser_test (cached) PASSED in 0.1s
//score/time_slave/src/gptp/details:pdelay_measurer_test (cached) PASSED in 0.0s
//score/time_slave/src/gptp/details:raw_socket_test (cached) PASSED in 0.1s
//score/time_slave/src/gptp/details:sync_state_machine_test (cached) PASSED in 0.1s
//score/time_slave/src/gptp/instrument:probe_test (cached) PASSED in 0.0s
//score/time_slave/src/gptp/record:recorder_test (cached) PASSED in 0.0s
//score/ts_client/src:gptp_ipc_publisher_test (cached) PASSED in 0.1s
//score/ts_client/src:gptp_ipc_receiver_test (cached) PASSED in 0.1s
//score/ts_client/src:gptp_ipc_roundtrip_test (cached) PASSED in 0.1s

Executed 0 out of 49 tests: 49 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.

@BjoernAtBosch BjoernAtBosch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@NEOatNHNG
NEOatNHNG merged commit dcf3ff2 into eclipse-score:main Oct 7, 2026
19 of 22 checks passed
@NEOatNHNG
NEOatNHNG deleted the feature/AZB-101279-Quality-pack-targets branch October 7, 2026 09:00

This branch is waiting to be deployed

1 waiting deployment
workflow-approval — 887c6dd1 Waiting Oct 6, 2026 by GabrielPinheiro7891 via qnx-build (x86_64-qnx) / approval #850
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants