Skip to content

Commit d5097d2

Browse files
committed
2nd round of addressing CoPilot findings
Signed-off-by: Philipp Ahmann <philipp.ahmann@de.bosch.com>
1 parent b243e61 commit d5097d2

3 files changed

Lines changed: 8 additions & 8 deletions

File tree

process/process_areas/verification/guidance/verification_guideline.rst

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ Automated tests can also be executed locally, as the sources and binaries are av
106106
Failing test cases during re-execution can be reported following the guide :need:`gd_temp__problem_template`.
107107

108108
During a release, for any non-executed test case, the reason for non-execution must be documented in the
109-
:need:`wp__platform_sw_release_note` or :need:`wp__module_sw_release_plan` depending on the level (unit to platform) of the test case.
109+
:need:`wp__platform_sw_release_note` or :need:`wp__module_sw_release_note` depending on the level (unit to platform) of the test case.
110110

111111
Execution of manual test cases
112112
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
@@ -122,18 +122,18 @@ Also manual test cases are written following the :need:`gd_guidl__verification_s
122122
A tag `manual` is suggested to mark the test case as manual, so that it can be filtered out from automated test execution.
123123

124124
In case a manual test case is not executed for a release, the reason for non-execution has to be documented in the
125-
:need:`wp__platform_sw_release_note` or :need:`wp__module_sw_release_plan` depending on the level of the test case.
125+
:need:`wp__platform_sw_release_note` or :need:`wp__module_sw_release_note` depending on the level of the test case.
126126

127127
Reporting of failing test cases
128128
-------------------------------
129129

130130
Any failing test case requires an ISSUE.
131131

132-
The passing rate of safety-critical test cases need to be 100% in order to release the affected component.
132+
The passing rate of safety-critical test cases needs to be 100% in order to release the affected component.
133133
In case of a lower pass rate than 100% for QM level tests, the :need:`rl__project_lead`
134-
can decide, if the platform is in a releasable state. The accepted minimal pass rate is defined
134+
can decide if the platform is in a releasable state. The accepted minimal pass rate is defined
135135
in the :need:`wp__verification_plan`. Due to the high degree of automation, it is recommended
136-
that a pass rate lower 95% is not acceptable. This percentage may increase with the maturity of
136+
that a pass rate lower than 95% is not acceptable. This percentage may increase with the maturity of
137137
the overall platform and the test coverage.
138138

139139
In case an existing test case is failing due to regression in the CI, the PR author in their role as

process/process_areas/verification/guidance/verification_process_reqs.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ Process Requirements
174174

175175
- TestType and DerivationTechnique shall be set
176176
- Description shall not be empty
177-
- In a Platform Integration Test Partially/FullyVerifies shall be set to a Platform Requirement
177+
- In a Platform Integration Test Partially/FullyVerifies shall be set to at least one Platform Requirement
178178
- If Partially/FullyVerifies are set in Feature Integration Test these shall link to at least one Feature Requirement
179179
- If Partially/FullyVerifies are set in Component Integration Test these shall link to at least one Component Requirement
180180
- If Partially/FullyVerifies are set in Unit Test these shall link to at least one Component Requirement

process/process_areas/verification/guidance/verification_specification.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ A test specification contains the following attributes.
5757
- Inputs
5858
- Expected outcome/output/result (e.g. "A success message is displayed." or "Result should be 42.")
5959
- Test environment (e.g. network configuration, clean system state)
60-
- Expected time sequence of events and behavior (where applicable and an expectation)
60+
- Expected time sequence of events and behavior (where applicable and where such an expectation exists)
6161
-
6262
* - TestType
6363
- Examples are:
@@ -85,7 +85,7 @@ The implementation of :need:`wp__verification_plan` defines the full list of all
8585

8686
It is assumed that tests will be written as code (also for manual tests, which are script-based)
8787
and that each test case will have a unique identifier (e.g. its script name, execution call, or function name).
88-
The invocation used to execute the test defines the test case identification, thereby guaranteeing
88+
The invocation used to execute the test defines the test case identification, thereby supporting
8989
proper traceability and reproducibility.
9090

9191
As the tests are stored in a repository close to the implementation code, versioning is handled by the repository's version control.

0 commit comments

Comments
 (0)