Skip to content

Review TT-CONSTRUCTION#120

Closed
aschemmel-tech wants to merge 1 commit intomainfrom
aschemmel-tech-patch-3
Closed

Review TT-CONSTRUCTION#120
aschemmel-tech wants to merge 1 commit intomainfrom
aschemmel-tech-patch-3

Conversation

@aschemmel-tech
Copy link

no impact on nlohman/json

modified assertions and evidences and added SME scores

@github-actions github-actions bot added the M label Nov 5, 2025
@coveralls
Copy link

coveralls commented Nov 5, 2025

Coverage Status

coverage: 99.186%. remained the same
when pulling e7253a8 on aschemmel-tech-patch-3
into 5542b4c on main.

Signed-off-by: aschemmel-tech <aschemmel_job@arcor.de>
@github-actions github-actions bot removed the M label Nov 6, 2025

Evidence:

1. list of reproducible SHAs
Copy link
Collaborator

Choose a reason for hiding this comment

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

nlohmann/json library is just a header file (single_include/nlohmann/json.hpp), so this would be the only thing on the list.

Furthermore, this should already be covered by JLS-14


2. list of non-reproducible elements with:
- explanation and justification
- details of what is not reproducible
Copy link
Collaborator

Choose a reason for hiding this comment

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

nothing is non reproducible

- explanation and justification
- details of what is not reproducible

3. evidence of configuration management for build instructions and infrastructure
Copy link
Collaborator

Choose a reason for hiding this comment

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

TODO:

  • Look into build manifest files
  • Check if there is already some documentation on config management/build instructions in nlohmann/json

Copy link

Choose a reason for hiding this comment

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

Newly created JLS-38


3. evidence of configuration management for build instructions and infrastructure

4. evidence of repeatable builds
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revisit.


Evidence:

1. list of tests
Copy link
Collaborator

Choose a reason for hiding this comment

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

already provided (JLS-16)


aschemmel-tech: we need a split in four statements/evidences (JLS) as defined in TA-TESTS. Here two are accumulated.

For the topic of test list: missing argument why we think tests are complete e.g. structtural coverage, linking to the RFC8259 standard?
Copy link
Collaborator

Choose a reason for hiding this comment

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

Statement on coverage is provided in JLS-27 (recently added)

aschemmel-tech: we need a split in four statements/evidences (JLS) as defined in TA-TESTS. Here two are accumulated.

For the topic of test list: missing argument why we think tests are complete e.g. structtural coverage, linking to the RFC8259 standard?
I also see that nlohman defines fuzz testing (e.g. https://github.com/score-json/json/blob/main/tests/src/fuzzer-parse_bjdata.cpp). We should refer to this but maybe we cannot reproduce? Where are the results for this?
Copy link
Collaborator

@Erikhu1 Erikhu1 Nov 11, 2025

Choose a reason for hiding this comment

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

Fuzz testing is already stated and referred to in JLS-02.

TO-DO: Check if we should/can add another link from TA-TESTS to JLS-02.

Copy link

Choose a reason for hiding this comment

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

JLS-02 fits really well here IMO.

I also see that nlohman defines fuzz testing (e.g. https://github.com/score-json/json/blob/main/tests/src/fuzzer-parse_bjdata.cpp). We should refer to this but maybe we cannot reproduce? Where are the results for this?

For the topic of test execution environment(s) : my understanding: only one environment was used: the github host environment together with Doctest (version ?)
but with a wide range of compilers and versions of these. No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • Check if we can create a list of compiler+compiler version used for each test.
  • Add some reference to github actions page? (to show that we are using github host environment)

Copy link

Choose a reason for hiding this comment

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

added reference for bullet point 2.


All components, dependencies and tools are listed in a manifest.

aschemmel-tech: missing evidence, so impossible to check if the completeness of this statement fulfillment is given.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Already updated (in a separate branch. Will be merged to #115)

A score is calculated based on the number of mirrored and unmirrored things. (TODO) No newline at end of file
A score is calculated based on the number of mirrored and unmirrored things.

aschemmel-lib: This statement should be about "evidence of repeatable builds". Mirroring is one aspect, but also the "internet access" topic, the repeatability on different server and how much the used tools are under our control (i.e. are these available during our product lifecycle). For example do we need CMake, Doctest mirrored? No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

Revisit (discuss internally)

Copy link

Choose a reason for hiding this comment

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

The following evidences are already stated in JLS-10 and JLS-19:

  • source code
  • build instructions
  • tests
  • attestations

tests in JLS-10 will be split up into test code and test results summary to align with the evidence list provided by the trustable doc.

@aschemmel-tech
Copy link
Author

Topics taken into account in eclipse-score#9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants