fix(CSAF2.1): #197 fix mandatoryTest_6_1_27.8 - #648
Open
bendo-eXX wants to merge 4 commits into
Open
Conversation
bendo-eXX
force-pushed
the
fix/csaf-2.1_mandatory_test_6.1.27.8
branch
from
June 18, 2026 09:39
ea490a4 to
a3db3e1
Compare
|
Coverage after merging fix/csaf-2.1_mandatory_test_6.1.27.8 into main
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
# Conflicts: # tests/csaf_2_1/oasis.js
|
Coverage after merging fix/csaf-2.1_mandatory_test_6.1.27.8 into main will be
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
# Conflicts: # csaf_2_1/mandatoryTests.js
|
Coverage after merging fix/csaf-2.1_mandatory_test_6.1.27.8 into main will be
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
rainer-exxcellent
requested changes
Jul 27, 2026
| const groupProductMap = new Map() | ||
| /** @type {ProductGroup[] | undefined} */ | ||
| const productGroups = doc.product_tree?.product_groups | ||
| if (Array.isArray(productGroups)) { |
Contributor
There was a problem hiding this comment.
I think most of the isArray statements are not necessary. This should be done in the Validation
| const vulnerabilities = doc.vulnerabilities | ||
| if (Array.isArray(vulnerabilities)) { | ||
| vulnerabilities.forEach((vulnerability, vulnerabilityIndex) => { | ||
| if (['ids', 'cve'].every((p) => vulnerability[p] === undefined)) { |
Contributor
There was a problem hiding this comment.
Please use meaningful variable names, not p
| } | ||
| }) | ||
| } | ||
| return hadErrors |
Contributor
There was a problem hiding this comment.
The result in oasis_csaf_tc-csaf_2_1-2024-6-1-27-08-02.result.json does not match to the result of the function
[
{
"instancePath": "/vulnerabilities/0/product_status/known_not_affected/0",
"message": "product id `CSAFPID-9080700` does not have a vulnerability id assigned nor a CVE or general vulnerability id is given"
},
{
"instancePath": "/vulnerabilities/0/product_status/known_not_affected/1",
"message": "product id `CSAFPID-9080701` does not have a vulnerability id assigned nor a CVE or general vulnerability id is given"
},
{
"instancePath": "/vulnerabilities/1/product_status/known_not_affected/3",
"message": "product id `CSAFPID-9080703` does not have a vulnerability id assigned nor a CVE or general vulnerability id is given"
},
{
"instancePath": "/vulnerabilities/2/product_status/known_not_affected/3",
"message": "product id `CSAFPID-9080703` does not have a vulnerability id assigned nor a CVE or general vulnerability id is given"
}
]
Contributor
Author
There was a problem hiding this comment.
Possible inconsistency between test case and expected result
I may be overlooking something or misunderstanding the intended behavior, but it seems that the test case oasis_csaf_tc-csaf_2_1-2024-6-1-27-08-02.json and the corresponding oasis_csaf_tc-csaf_2_1-2024-6-1-27-08-02.result.json do not match.
For the first vulnerability (/vulnerabilities/0/), the scenario is:
"product_ids": ["CSAFPID-9080702"]Validation against:
known_not_affected = [
"CSAFPID-9080700",
"CSAFPID-9080701",
"CSAFPID-9080702"
]Results in:
CSAFPID-9080700→ not covered → validation error at index 0CSAFPID-9080701→ not covered → validation error at index 1CSAFPID-9080702→ covered → no validation error
This produces 2 validation errors (indices 0 and 1).
However, the expected result.json only contains a validation error at index 2.
Possible solution
If the expected result is correct, the test case could instead reference the product group:
"group_ids": ["CSAFGID-1020300"]where CSAFGID-1020300 resolves to:
[
"CSAFPID-9080700",
"CSAFPID-9080701"
]Validation against the same known_not_affected list would then produce:
CSAFPID-9080700→ covered → OKCSAFPID-9080701→ covered → OKCSAFPID-9080702→ not covered → validation error at index 2
This results in 1 validation error (index 2), which matches the expected result.json.
Expected behavior
If the expected outcome is the one defined in result.json, it seems that the affected products should be resolved via the referenced product group rather than by referencing the single product directly.
Could you confirm whether this is an inconsistency in the test case or if I am missing some aspect of the intended validation logic?
|
Coverage after merging fix/csaf-2.1_mandatory_test_6.1.27.8 into main will be
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
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
2 participants
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.
No description provided.