devonfw / cobigen

Code-based Incremental Generator
Apache License 2.0
35 stars 72 forks source link

QA on templates #749

Open maybeec opened 6 years ago

maybeec commented 6 years ago

Expected behavior

The current ide setup of devon 2.4.0 delivers templates. I am importing these templates and trying to generate different java application based on different architectures (simple vs complex). It should work in any combination like

Please also check "nested" and spring data increments. Maybe we should cleanup here. Currently, it is not easy to understand what is meant by "nested".

Actual behavior

Steps to reproduce (bug) / Use Case of feature request (enhancement)

  1. Generate different combinations as mentioned above. Different errors will arise.
  2. Please start with a clean state all the time.

Related/Dependent Issues

Implementation Hints

Best would be, if we could provide multiple SWTBot tests, each generating one combination of increments above, which should compile out of the box. Then we could check that, by trying to find compilation errors in the error view. You might want to have a look at the latest commit on cobigen-eclipse, which introduced SWTBot code to access the error log. A prove of validity might be to assert, that there are no errors after generation and rebuild.

jdiazgon commented 6 years ago

In this comment, I will keep updating the errors that I find related to the generation of the different increments.

Using OASP4J archetype version 2.4.0.

Working fine

These are the increments that compile and work fine (although to compile you need Entity infrastructure and TO's increments).

Some compilation errors

jdiazgon commented 6 years ago

The usefulness of nested and spring data increments should be discussed. I haven't received any claim on my side related to compilation errors. This may mean that our developers just use the most common increments like CRUD DAO + CRUD logic (simple) + CRUD service REST + CRUD service SOAP + TO's + Entity infrastructure.

github-actions[bot] commented 5 years ago

Stale issue message