Skip to content

Extend switch_run_target test to also cover feat_req__lifecycle__process_ordering and comp_req__launch_man__launch_manager_shutdown - #442

Merged
MaciejKaszynski merged 11 commits into
eclipse-score:mainfrom
etas-contrib:feature/extend_switch_run_target_test
Aug 11, 2026
Merged

Extend switch_run_target test to also cover feat_req__lifecycle__process_ordering and comp_req__launch_man__launch_manager_shutdown#442
MaciejKaszynski merged 11 commits into
eclipse-score:mainfrom
etas-contrib:feature/extend_switch_run_target_test

Conversation

@TimoSteuerwaldETAS

@TimoSteuerwaldETAS TimoSteuerwaldETAS commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Part of #317
Extends text to verify correct shutdown order.

@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown

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]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 24cf5fa7-24cf-47bb-a3ac-cae81179f7a4
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

Analyzing: target //:license-check (101 packages loaded, 22 targets configured)

Analyzing: target //:license-check (149 packages loaded, 3098 targets configured)

Analyzing: target //:license-check (162 packages loaded, 6249 targets configured)

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

Analyzing: target //:license-check (170 packages loaded, 7160 targets configured)

Analyzing: target //:license-check (174 packages loaded, 11409 targets configured)

Analyzing: target //:license-check (174 packages loaded, 11409 targets configured)

INFO: Analyzed target //:license-check (175 packages loaded, 11535 targets configured).
[14 / 16] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
[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: 25.751s, 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>]

@github-actions

Copy link
Copy Markdown

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

@WilliamRoebuck WilliamRoebuck 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.

I think the test already satisfied the feature requirement and, if you agree with me on the component one, we'd just need to change the control client to wait for SIGTERM rather than requesting the "off" target

Comment thread tests/integration/switch_run_target/control_client_mock.cpp Outdated
Comment thread tests/integration/switch_run_target/control_client_mock.cpp Outdated
Comment thread tests/integration/switch_run_target/switch_run_target.py

@WilliamRoebuck WilliamRoebuck 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.

Looks good, just one minor fix

Comment thread tests/integration/switch_run_target/switch_run_target.py Outdated
TimoSteuerwaldETAS and others added 4 commits August 11, 2026 11:54
Co-authored-by: WilliamRoebuck <244554584+WilliamRoebuck@users.noreply.github.com>
Signed-off-by: Timo Steuerwald <142218678+TimoSteuerwaldETAS@users.noreply.github.com>
@TimoSteuerwaldETAS
TimoSteuerwaldETAS deployed to workflow-approval August 11, 2026 11:04 — with GitHub Actions Active
@TimoSteuerwaldETAS
TimoSteuerwaldETAS deployed to workflow-approval August 11, 2026 11:04 — with GitHub Actions Active
@MaciejKaszynski
MaciejKaszynski merged commit 3374789 into eclipse-score:main Aug 11, 2026
20 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in LCM - Lifecycle & Health FT Aug 11, 2026
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.

3 participants