[
https://issues.apache.org/jira/browse/SLING-10747?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Bertrand Delacretaz resolved SLING-10747.
-----------------------------------------
Resolution: Fixed
Implemented in commit
https://github.com/apache/sling-org-apache-sling-repoinit-parser/commit/0128ee6bdc284a060f442258da59883b2dcab828
> Remove test-99 and better sync docs page with repoinit parser tests
> -------------------------------------------------------------------
>
> Key: SLING-10747
> URL: https://issues.apache.org/jira/browse/SLING-10747
> Project: Sling
> Issue Type: Bug
> Components: Repoinit
> Affects Versions: Repoinit Parser 1.6.10
> Reporter: Bertrand Delacretaz
> Assignee: Bertrand Delacretaz
> Priority: Minor
> Fix For: Repoinit Parser 1.6.12
>
>
> The docs page at
> [https://sling.apache.org/documentation/bundles/repository-initialization.html]
> is supposed to show examples of all repoinit statements, but it's hard to
> keep in sync manually and is often slightly out of sync.
> The {{src/test/resources/testcases/test-99.txt}} is supposed to also expose
> all the repoinit syntax, is also maintained manually and is not 100% in sync
> with that docs page.
> To make sure the docs stay in sync with minimal effort, we should:
> * Remove the {{test-99.txt}} and adapt the other tests scenarios to make
> sure the test coverage remains the same or better. Currently there's a slight
> difference in "missed branches" in the {{-P jacoco-report}} output for the
> parser.impl package if I remove it.
> * Create a script that aggregates all the
> {{src/test/resources/testcases/test-*.txt}} scenarios, ordered by name, and
> use that output in the docs page.
> * Verify that the result is at least as good as the current docs page in
> terms of examples, comments and notes on required versions, and adapt the
> test scenarios if not.
> * Add information to the docs page on how to keep it up to date.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
