Skip to content

process: adjust testing levels and traceability #823

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pahmann
Copy link
Contributor

@pahmann pahmann commented Mar 31, 2025

see #821

Copy link

License Check Results

🚀 The license check preparation job ran successfully.

Status: ⚠️ Needs Review

Click to expand output
2025/03/31 12:32:57 Downloading https://releases.bazel.build/7.4.0/release/bazel-7.4.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: docs
Loading: 0 packages loaded
    currently loading: docs
Analyzing: target //docs:license.check.python (1 packages loaded, 0 targets configured)
Analyzing: target //docs:license.check.python (1 packages loaded, 0 targets configured)

Analyzing: target //docs:license.check.python (96 packages loaded, 10 targets configured)

Analyzing: target //docs:license.check.python (124 packages loaded, 379 targets configured)

Analyzing: target //docs:license.check.python (131 packages loaded, 840 targets configured)

Analyzing: target //docs:license.check.python (144 packages loaded, 2465 targets configured)

Analyzing: target //docs:license.check.python (144 packages loaded, 2465 targets configured)

Analyzing: target //docs:license.check.python (147 packages loaded, 4487 targets configured)

Analyzing: target //docs:license.check.python (148 packages loaded, 4611 targets configured)

Analyzing: target //docs:license.check.python (148 packages loaded, 4611 targets configured)

INFO: Analyzed target //docs:license.check.python (149 packages loaded, 4736 targets configured).
[9 / 13] JavaToolchainCompileClasses external/rules_java~/toolchains/platformclasspath_classes; 0s processwrapper-sandbox ... (2 actions running)
[11 / 13] JavaToolchainCompileBootClasspath external/rules_java~/toolchains/platformclasspath.jar; 0s processwrapper-sandbox
INFO: Found 1 target...
Target //docs:license.check.python up-to-date:
  bazel-bin/docs/license.check.python
  bazel-bin/docs/license.check.python.jar
[13 / 13] no actions running
INFO: Elapsed time: 19.588s, Critical Path: 2.39s
INFO: 13 processes: 9 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/docs/license.check.python docs/formatted.txt -review -project automotive.score -repo https://github.com/eclipse-score/score -token otyhZ4eaRYK1tKLNNF-Y
[main] INFO Querying Eclipse Foundation for license data for 69 items.
[main] INFO Found 45 items.
[main] INFO Querying ClearlyDefined for license data for 25 items.
[main] INFO Found 25 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO pypi/pypi/-/docutils/0.21.2
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.
[main] INFO A review is required for pypi/pypi/-/docutils/0.21.2.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/19880 .

@pahmann
Copy link
Contributor Author

pahmann commented Mar 31, 2025

@aschemmel-tech this is the draft PR to rework the testing level and to unify component and component integration testing as discussed in the process WG meeting last week.

Copy link

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

Copy link
Contributor

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

Additional to inline comments: I really think we need to provide a complete example for a "unit test with gtest" and a "feature test with ITF".

:output: wp__verification__comp_int_test
:contains: gd_req__link_tests, gd_guidl__verification_specification
:has: doc_concept__verification__process, doc_getstrt__verification__process

Component Integration test cases are based on component architecture and detailed design of implementation.
Component Integration test cases are based on component architecture and component requirements.
They may optionally cover the detailed design of implementation as an optional element where units are seen complex.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would not say so. The detailed design describes the the units forming a component and their interactions - see https://eclipse-score.github.io/score/main/process/process_areas/implementation/_assets/detailed_design_example.html - so the interfaces and interactions between units can be used as a basis for testing. Maybe rather make the integration testing of component architecture optional (in case a component has no (sub-)components.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think what is still unclear: are the component integration tests 6-9 or 6-10? In the SW verification plan it seems to say that "level 1" is 6-9. Is this what you want to say? And I also think we need to explain better what a "Unit Tester" has to do in the guideline: 1. read the Detailed Desig, create a test for every interface of the unit showing at least every flow in dynamic diagrams, 2. follow the DD to the component requirements - test these 3. fill attributes including test description 4. link test to ??? (not component requirements)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

From WG working meeting 6-10 goes down to green "Component".
6-9 is for grey part.
Unclear is the handling of requirements.
Start with the simple use case (without Component architecture due to unit simplicity)
Detailed design is directly related to component requirements and implicitly covered.

Copy link
Contributor

Choose a reason for hiding this comment

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

I was thinking if we not just also add a link from Unit Test to the Component Requirements. Reasoning is that this would be also easier to demonstrate a full requirement coverage by testing. But also we should keep the DD link possibility for unit test because we do want also the possibility to test DD instead have a Component requirement for everything - maybe best to say you have to have one or the other.

@masc2023 masc2023 moved this to In Progress in Process Community Apr 7, 2025
@pahmann pahmann linked an issue Apr 22, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Improvement: Simplify component testing structure
2 participants