-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial draft of a validation testing framework (Tests 6.1.1-6.1.7) #67
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
|
889d59e
to
887cf05
Compare
3b8e1a5
to
159d488
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's sometimes not really straightforward to relate the paths mentioned in the spec to the code but it should match in most cases. Also, some descriptions would help sometimes.
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Extensions.kt
Outdated
Show resolved
Hide resolved
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Extensions.kt
Outdated
Show resolved
Hide resolved
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Extensions.kt
Outdated
Show resolved
Hide resolved
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Extensions.kt
Outdated
Show resolved
Hide resolved
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Extensions.kt
Outdated
Show resolved
Hide resolved
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Tests.kt
Show resolved
Hide resolved
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Tests.kt
Show resolved
Hide resolved
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Tests.kt
Outdated
Show resolved
Hide resolved
csaf-validation/src/main/kotlin/io/github/csaf/sbom/validation/tests/Tests.kt
Show resolved
Hide resolved
This is work in progress, but it tries to provide an API for the implementation of 6.1 mandatory tests for validation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I stumbled on the term "Branche", any chance we can refer to the singular of "Csaf.Branche" as "Branch" instead?
WITHDRAWN: I'm having trouble with the submodules (yes i ran init and update), shouldn't there be a .gitmodules file?
This is work in progress, but it tries to provide an API for the implementation of 6.1 mandatory tests for validation.
It also adds a couple of 6.1.x tests as an example