Skip to content

Refactor tests and improve component structure - #173

Merged
masc2023 merged 2 commits into
mainfrom
cleanup2
Sep 11, 2026
Merged

masc2023 merged 2 commits into
mainfrom
cleanup2

Conversation

@AlexanderLanin

@AlexanderLanin AlexanderLanin commented Sep 11, 2026 •

Copy link
Copy Markdown
Member

Reorganize tests and enhance the structure of the component example. Introduce unit and integration tests for better coverage and clarity. Update documentation references and remove unnecessary files.

Resolves #130

- Removed unnecessary test directories and files.
- Updated test names for clarity and consistency.
- Added unit tests for `make_hello()` function.
- Enhanced `make_hello_message()` function to utilize `make_hello()`.
- Introduced new integration tests for module-level functionality.
- Updated documentation references in the BUILD files.
@github-actions

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]
Extracting Bazel installation...
Starting local Bazel server (8.7.0) and connecting to it...
INFO: Invocation ID: 7d5700af-6774-433b-b963-48616232882f
Computing main repo mapping: 
WARNING: For repository 'aspect_rules_lint', the root module requires module version aspect_rules_lint@1.10.2, but got aspect_rules_lint@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: 
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: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

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

Analyzing: target //:license-check (146 packages loaded, 776 targets configured)

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

Analyzing: target //:license-check (164 packages loaded, 6664 targets configured)

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

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

Analyzing: target //:license-check (172 packages loaded, 8459 targets configured)

Analyzing: target //:license-check (175 packages loaded, 10344 targets configured)

Analyzing: target //:license-check (176 packages loaded, 10464 targets configured)

Analyzing: target //:license-check (176 packages loaded, 10464 targets configured)

INFO: Analyzed target //:license-check (177 packages loaded, 12043 targets configured).
[7 / 15] [Prepa] Creating source manifest for @@score_tooling+//dash/tool/formatters:dash_format_converter [for tool] ... (2 actions, 0 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] [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: 194.303s, Critical Path: 2.74s
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
Contributor

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

Comment thread docs/template/index.rst
├── docs/ # Global documentation of the module
│ ├── features/ # Feature documentation and architecture
│ │ └── <feature_name>/ # Folder for each feature parts which should be in module documentation
│ │ └── <feature_name>/ # Folder for each feature parts which should be in module documentation. Omit in single feature case.

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.

Omit subfolder in single feature case.

@masc2023
masc2023 merged commit 03c00dd into main Sep 11, 2026
7 checks passed
@masc2023
masc2023 deleted the cleanup2 branch September 11, 2026 07:17
aryansingh0012 added a commit to aryansingh0012/module_template that referenced this pull request Oct 14, 2026
:
:wq:

:

docs: describe onboarding module

chore(github-actions): bump AndreasAugustin/actions-template-sync (eclipse-score#171)

Bumps the github-actions group with 1 update: [AndreasAugustin/actions-template-sync](https://github.com/andreasaugustin/actions-template-sync).

Updates `AndreasAugustin/actions-template-sync` from 2.5.3 to 2.6.0
- [Release notes](https://github.com/andreasaugustin/actions-template-sync/releases)
- [Changelog](https://github.com/AndreasAugustin/actions-template-sync/blob/main/CHANGELOG.md)
- [Commits](AndreasAugustin/actions-template-sync@v2.5.3...v2.6.0)

---
updated-dependencies:
- dependency-name: AndreasAugustin/actions-template-sync
  dependency-version: 2.6.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Update dependency score_docs_as_code to v8.1.2 (eclipse-score#172)

Refactor tests and improve component structure (eclipse-score#173)

* rename

* refactor: reorganize tests and improve component example structure

- Removed unnecessary test directories and files.
- Updated test names for clarity and consistency.
- Added unit tests for `make_hello()` function.
- Enhanced `make_hello_message()` function to utilize `make_hello()`.
- Introduced new integration tests for module-level functionality.
- Updated documentation references in the BUILD files.

add example for auto generated module verification report (eclipse-score#166)

* add example for auto generated module verifiaction report

* udpate rules lint version

* Update docs/verification_report/module_verification_report.rst

Co-authored-by: Alexander Lanin <Alexander.Lanin@etas.com>
Signed-off-by: RolandJentschETAS <135332348+RolandJentschETAS@users.noreply.github.com>

* fix link name

* add link to report from process

---------

Signed-off-by: RolandJentschETAS <135332348+RolandJentschETAS@users.noreply.github.com>
Co-authored-by: Alexander Lanin <Alexander.Lanin@etas.com>
Co-authored-by: Markus Schu <markus.schu@accenture.com>

chore(bazel): align development-only dependencies (eclipse-score#175)

Co-authored-by: eclipse-score-bot <187756813+eclipse-score-bot@users.noreply.github.com>

chore: satisfy onboarding pre-commit checks

style: apply yamlfmt formatting

Update onboarding.agent.yaml

Signed-off-by: aryansingh0012 <aryan.s@ltts.com>

trimmed toml

removed tools name

removed handoff recom

fix module directory

Delete tools/onboarding/tools/onboarding/agent/onboarding.agent.yaml

Signed-off-by: aryansingh0012 <aryan.s@ltts.com>

Fix onboarding BUILD data reference and ignore .onboarding/

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RqoW1gXiXg4iXXTahrwNkg

improvement of module verification report (eclipse-score#178)

chore(github-actions): bump eclipse-score/cicd-workflows/.github/workflows/daily.yml (eclipse-score#179)

Bumps the github-actions group with 1 update: [eclipse-score/cicd-workflows/.github/workflows/daily.yml](https://github.com/eclipse-score/cicd-workflows).

Updates `eclipse-score/cicd-workflows/.github/workflows/daily.yml` from 37eaadadfefc263d94a858d25665735659e916fa to 8c48c65fc9e8927a06214192e6ebce520853087d
- [Release notes](https://github.com/eclipse-score/cicd-workflows/releases)
- [Commits](eclipse-score/cicd-workflows@37eaada...8c48c65)

---
updated-dependencies:
- dependency-name: eclipse-score/cicd-workflows/.github/workflows/daily.yml
  dependency-version: 8c48c65fc9e8927a06214192e6ebce520853087d
  dependency-type: direct:production
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Update dependency score_devcontainer to v1.11.1 (eclipse-score#181)

Update dependency score_docs_as_code to v8.2.0 (eclipse-score#182)

chore(github-actions): bump eclipse-score/cicd-workflows/.github/workflows/daily.yml (eclipse-score#183)

Bumps the github-actions group with 1 update: [eclipse-score/cicd-workflows/.github/workflows/daily.yml](https://github.com/eclipse-score/cicd-workflows).

Updates `eclipse-score/cicd-workflows/.github/workflows/daily.yml` from 8c48c65fc9e8927a06214192e6ebce520853087d to 145c53eb965228c981ba0073bbf8adb2b2ab0ebf
- [Release notes](https://github.com/eclipse-score/cicd-workflows/releases)
- [Commits](eclipse-score/cicd-workflows@8c48c65...145c53e)

---
updated-dependencies:
- dependency-name: eclipse-score/cicd-workflows/.github/workflows/daily.yml
  dependency-version: 145c53eb965228c981ba0073bbf8adb2b2ab0ebf
  dependency-type: direct:production
  dependency-group: github-actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Update dependency score_docs_as_code to v8.3.0 (eclipse-score#184)

Update dependency score_bazel_cpp_toolchains to v1.0.4 (eclipse-score#186)

feat(toolchain): centralize S-CORE toolchain configurations and compiler warning policies (eclipse-score#180)

Co-authored-by: Alexander Lanin <Alexander.Lanin@etas.com>
Co-authored-by: Srinivasu K <srinivasu.kandukuri@in.bosch.com>
Signed-off-by: Srinivasu K <srinivasu.kandukuri@in.bosch.com>

chore(github-actions): bump eclipse-score/cicd-workflows/.github/workflows/daily.yml (eclipse-score#187)

Signed-off-by: dependabot[bot] <support@github.com>

chore(docs): update score_docs_as_code gitignore configuration (eclipse-score#188)

Co-authored-by: eclipse-score-bot <187756813+eclipse-score-bot@users.noreply.github.com>

fix of safety analysis templates (eclipse-score#189)

add fixes

new changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

add code and test to example component

3 participants