Skip to content

Add documentation for testing guidelines#2554

Open
Rahul-Sutariya wants to merge 1 commit intoeclipse-score:mainfrom
Rahul-Sutariya:rs_testing_guidelines
Open

Add documentation for testing guidelines#2554
Rahul-Sutariya wants to merge 1 commit intoeclipse-score:mainfrom
Rahul-Sutariya:rs_testing_guidelines

Conversation

@Rahul-Sutariya
Copy link
Contributor

@Rahul-Sutariya Rahul-Sutariya commented Feb 6, 2026

This pull request adds documentation for testing guidelines to the SCORE repository. It introduces a new set of written instructions and best practices on how to write, organize, and run tests for the project. These guidelines aim to help contributors and maintainers consistently implement high-quality tests across the codebase. The PR includes documentation files and updates relevant to the project’s docs-as-code framework.

@Rahul-Sutariya Rahul-Sutariya requested a review from a team as a code owner February 6, 2026 08:46
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

⚠️ Docs-as-Code version mismatch detected
Please check the CI build logs for details and align the documentation version with the Bazel dependency.

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

The created documentation from the pull request is available at: docu-html

Copy link
Contributor

@PiotrKorkus PiotrKorkus left a comment

Choose a reason for hiding this comment

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

This document is quite specific and oriented only for C++ tests implementation. I find score repo a place for general rules so it shouldn't be too strict.
Other languages are omitted and they have their own rules. How about a general Guideline and then C++/Rust/Python specific ones?
Also some parts of this document are already mentioned in process description - did you check them?
e.g.:

Those should be in harmony, not repeat or conflict with each other.

:security: YES
:realizes: wp__sw_development_plan

These guidelines are based on discussions within the score com team. They're not set in stone and can be changed in the future.
Copy link
Contributor

Choose a reason for hiding this comment

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

If that's specific to communication, maybe it should be a part of com specific documentaiton

:status: valid
:safety: ASIL_B
:security: YES
:realizes: wp__sw_development_plan
Copy link
Contributor

Choose a reason for hiding this comment

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

This work product is already here, https://eclipse-score.github.io/score/main/platform_management_plan/software_development.html, you may add more content to that here, our Verification Plan https://eclipse-score.github.io/score/main/platform_management_plan/software_verification.html, so get in touch with @pahmann , where these content fits best

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.

3 participants