On Tue, 2022-05-03 at 14:20 +0200, Carsten Ziegeler wrote: > Thanks, sounds good to me. > > Would it make sense to also move the tests into the starter, so tests > can be updated/maintained with the starter?
Until we get there, I plan to make at least one incremental improvement, and that is the ability to run the Starter ITs from other projects, with overriden bundle versions. We should be able to make change (for instance) in the Servlets POST bundle and run the Starter ITs with only the Servlets POST bundle built locally. That would IMO make testing PRs much simpler as regressions will be visible without rebuilding the Starter. After that, we can take things in two directions: 1. Move all the test-related classes in the starter project. 2. Move test-related classes into the bundle they are testing, for example [1] into the Servlets POST bundle. At the same time, we should also be running those tests in the Starter module. I see 1. as considerably simplifying builds and reducing the number projects, but not improving the flow of submitting tests with the bundles. On the other hand 2. will complicate a little how we aggregate and launch the tests in the Starter module but will have the advantage of being able to submit integration tests with changes in a single PR. _If_ we decide to go with 2, we will probably have a hybrid approach where some tests get moved in the starter module as they are not really specific to a single module and the rest get moved to bundle modules. I haven't really given it too much though, so feedback is welcome. Thanks, Robert [1]: https://github.com/apache/sling-org-apache-sling-launchpad-integration-tests/tree/master/src/main/java/org/apache/sling/launchpad/webapp/integrationtest/servlets/post > > Regards > Carsten > > Am 29.04.2022 um 16:30 schrieb Robert Munteanu: > > Hi, > > > > We have discussed making testing a bit easier with the Sling > > Starter. > > Right now we have the Starter itself and the launchpad-testing > > module > > that is used for executing the tests. > > > > This can in my opinion be improved by moving the test execution in > > the > > Starter module itself. > > > > I have created two PRs that achieve this goal: > > > > - > > https://github.com/apache/sling-org-apache-sling-launchpad-integration-tests/pull/4 > > - https://github.com/apache/sling-org-apache-sling-starter/pull/66 > > > > After we merge them, with a `mvn clean verify` we can run the full > > set > > of ITs in ~2,5 minutes. > > > > I plan to merge these changes on Wednesday. Please let me know if > > you > > see any reason for this not to proceed. > > > > Thanks, > > Robert >
