[INTERNAL] builder: add test for builder JSDoc parameter#592
[INTERNAL] builder: add test for builder JSDoc parameter#592tobiasso85 wants to merge 1 commit intov2from
Conversation
Increase test coverage
|
A verb in the commit message would be nice 🙂 . Something like "Add". 🍒 on top would be some capital letters like "JSDoc" |
There was a problem hiding this comment.
Please be aware that the builder.js tests must write into separate target directories each. Currently, your new test Build library.j with jsdoc flag set uses the same target directory as Build library.j with JSDoc build only.
So it overwrites the build result of the previous test but apparently does not delete any files.
If you change the target to an isolated directory you will find that no JSDoc resources are generated, because the excludedTasks: ["*"] parameter overrules the jsdoc: true flat and excludes all JSDoc tasks.
The jsdoc flag already excludes all unrelated tasks. So there's no need to use the excludedTasks parameter. After all, the UI5 CLI doesn't use it as well.
Increase test coverage