result: link tests to requirements - #579
Draft
Nikhil2206 wants to merge 3 commits into
Draft
Nikhil2206 wants to merge 3 commits into
Nikhil2206 wants to merge 3 commits into
Conversation
Nikhil2206
requested review from
4og,
antonkri,
arkjedrz,
fbaeuerle and
pawelrutkaq
as code owners
September 11, 2026 07:54
Nikhil2206
requested a deployment
to
workflow-approval
September 11, 2026 07:54 — with
GitHub Actions
Waiting
Nikhil2206
requested a deployment
to
workflow-approval
September 11, 2026 07:54 — with
GitHub Actions
Waiting
Nikhil2206
requested a deployment
to
workflow-approval
September 11, 2026 07:54 — with
GitHub Actions
Waiting
Nikhil2206
requested a deployment
to
workflow-approval
September 11, 2026 07:54 — with
GitHub Actions
Waiting
Contributor
|
Documentation preview for this pull request is available at: |
Nikhil2206
requested a deployment
to
workflow-approval
September 11, 2026 08:24 — with
GitHub Actions
Waiting
Nikhil2206
requested a deployment
to
workflow-approval
September 11, 2026 08:24 — with
GitHub Actions
Waiting
Nikhil2206
requested a deployment
to
workflow-approval
September 11, 2026 08:24 — with
GitHub Actions
Waiting
Nikhil2206
requested a deployment
to
workflow-approval
September 11, 2026 08:24 — with
GitHub Actions
Waiting
Nikhil2206
marked this pull request as draft
September 11, 2026 08:24
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Link all 256 C++ tests in the
score/resultcomponent to the component requirements using the requiredRecordPropertymetadata.This completes Route 1 traceability authoring for a component that had no pre-existing test requirement metadata.
Files touched
score/result/error_test.cppscore/result/error_msg_mapping_test.cppscore/result/result_test.cppscore/result/details/expected/expected_assignment_test.cppscore/result/details/expected/expected_constructor_test.cppscore/result/details/expected/expected_equality_test.cppscore/result/details/expected/expected_monadic_operator_test.cppscore/result/details/expected/expected_observer_test.cppscore/result/details/expected/expected_type_trait_test.cppscore/result/details/expected/extensions_test.cppscore/result/details/expected/test_type_traits_test.cppscore/result/details/expected/unexpected_test.cppTraceability
Each test now has
PartiallyVerifies,Description,TestType, andDerivationTechniquemetadata.The links cover
error_handling,set_result,domain_error_information,type_safety,std_integration,deterministic_behavior, andexception_free_operation.Observer misuse tests use
fault-injectionandboundary-values; systematic ref-qualified and type-variant test families useequivalence-classes.Validation
bazel test --config=bl-x86_64-linux //score/result/...-> all 7 test targets passed.bazel run //:format.fix -- score/result/details/expected/expected_monadic_operator_test.cpp score/result/details/expected/expected_observer_test.cppcompleted successfully.git diff --checkpassed.FullyVerifies/PartiallyVerifieslinks.comp_reqIDs exist inscore/result/docs/requirements/index.rst.The full docs build remains blocked by the pre-existing root-owned
_build/permission error, so requirement ID validation used the documented grep fallback.Deliberately left unlinked
None. All 256 result tests have traceability metadata.
Flagged requirement-quality issue
extensions_test.cppincludesscore::cpp::expectedandscore::cpp::optionalinteroperability tests in addition tostd::optionalinteroperability. These tests are linked tocomp_req__result__std_integration, whose current wording is narrower and explicitly namesstd::optional. The requirement wording may need clarification in a follow-up change.