qspin / qtaste

[NEW RELEASE VERSION 3.0.0] The QTaste is an open-source functional and non-functional test environment offering services to developers and testers. It has been implemented following a data driven testing philosophy.
http://www.qtaste.org
GNU Lesser General Public License v3.0
6 stars 11 forks source link

QTaste doesn't generate all Test API documentation anymore #72

Closed dergo closed 8 years ago

dergo commented 10 years ago

In QTaste 2.1.0 and current 2.2.0-SNAPSHOT, the Test API documentation generated in /target/TestAPI-doc/ doesn't include the documentation of the other Test API on which the Test API depends anymore.

E.g. at BMS level, I had the documentation of BMS, BDS, BSS, ICOMP, QTaste-Toolbox, etc..., now I only have BMS documentation.

There is a maven inheritance configuration issue in qtaste-testapi-parent: the configuration of maven-dependency-plugin is not inherited anymore!

Also even if I override the configuration myself, JavaGUI is not included because the package name doesn't match the includes: <includes>*/qtaste/testapi/api/.java</includes> must be changed to: <includes>*/qtaste//testapi/api/.java</includes>

nfac commented 9 years ago

changed testapi-parent pom file.

Issue to be validated.

lvboque commented 9 years ago

Validated OK: Utility documentation is generated! and wildcard are accepted!

dergo commented 9 years ago

Fix is not correct, configuration is still not correctly inherited from testapi-parent (you applied a workaround in demo/testapi to make it work, and the configuration is not even completely correct). You can use "mvn help:effective-pom" to check the effective pom in demo/testapi for example.

The problem seems to come from the <inherited>${plugins-inheritance}</inherited> tag in the testapi-parent, if I remove it the effective pom is correct, but I don't know why it has been added and if there are side-effects of removing it.

dergo commented 9 years ago

Actually I don't think the "inherited" tag should ever be used in a "pluginManagement" section, but only in "plugin" sections. "pluginManagement" is there expressly to define a default plugin configuration, so disabling inheritance would anyway be strange.

dergo commented 9 years ago

Fixed TestAPI documentation generation. To be validated.

The inherited tag has been removed, so if it causes an issue when doing a release, an other way to fix it will have to be found.

lvboque commented 8 years ago

Could you check if it was fixed in 2.3.0 and simply regenerate the release notes? Thks

dergo commented 8 years ago

Fixed in 2.3.0.