Skip to content
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

UI: add docs to examples #8734

Open
wants to merge 1 commit into
base: release_10
Choose a base branch
from

Conversation

nhaagen
Copy link
Contributor

@nhaagen nhaagen commented Dec 16, 2024

No description provided.

@nhaagen nhaagen force-pushed the 10/UI/exampledocs branch 2 times, most recently from ebc64a2 to 307314e Compare December 16, 2024 15:17
Copy link
Member

@klees klees left a comment

Choose a reason for hiding this comment

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

I only skimmed the changes, but everything I have read sound plausible.

@nhaagen: Could you explain a little how these changes arose?

@nhaagen
Copy link
Contributor Author

nhaagen commented Dec 18, 2024

Hi @klees ,
you personally know all that, but I'll expand for the broader public:

Although the UI is widely covered by unit tests, there is still a large part of manual testing involved.
In order to know about the anticipated behaviour, testers need an informative description of the anticipated behavior,
and/or guidance and instructions on how to operate the component. These instructions are organized in testrail, where every component is assigned two testcases, of which one is the functional testing of the rendered component (the other one is HTML validation).

With 10, we decided to have UI Components generate an import for testrail to setup/update testcases along with changes in components. That transfers the responsibility of updating the testcase to the developer - while she is at it, anyways.

We use the docs in the examples to hold the testcase description, so every developer is asked to add or amend the 'expected output' section of the example docs to fit the changed/introduced behavior of the given example.

Issues within might well be reported in mantis, and I'd also ask @klees, @Amstutz and @thibsy (and whoever will accept contributions to the UI framework) to watch out for missing/erroneous/deficitly rudimentary docs as well.

Best regards,
Nils

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

Successfully merging this pull request may close these issues.

4 participants