I've spent some time over the weekend adding more tests to the new test suite.
I've also set up parallel test execution on GitHub. All tests pass and it takes
about 45 minutes to run all tests on GitHub.
There are now 13 test modules with roughly 80 test classes. There are still
some big modules missing (infinispan, messaging). But I think we're now at
a point where it makes sense to think about migrating the existing
test suite to
the new approach.
On Fri, Feb 4, 2022 at 4:51 PM Harald Pehl <hpehl(a)redhat.com> wrote:
I've updated the new test suite:
- remove base test classes (thanks @Jan Kašík for being picky here)
- introduce local mode (using a local browser)
- adjust package names to make it easier to interop with the existing test suite
- update README.md
// Harald
On Fri, Feb 4, 2022 at 12:43 PM Harald Pehl <hpehl(a)redhat.com> wrote:
>
> On Fri, Feb 4, 2022 at 11:44 AM Jan Kašík <jkasik(a)redhat.com> wrote:
> >
> > I observed initialization taking place with each test class.
> >
>
> The WildFly container should be started / stopped before / after each unit test,
> but the browser and HAL container should be started only once.
>
> If your experience is different, let me know, the I should take a closer look.
>
> >
> > IMO it can add up to tens of minutes to the run.
> >
>
> On GitHub Actions the WildFly container starts in under 10 seconds, which is
> ok for me.
>
> >
> > It always starts with a few lines of code and ends up in a pile of mess. :)
I'm really not a fan of inheritance in case of test classes since I spent some time
trying to resolve issues caused by this.
> >
>
> I think it's a trade-off. If you really don't like it, I can try to
> use composition over inheritance.
>
> >
> > There's also another thing which I thought of. The host OS on CI machines
might be RHEL 8 where docker is not available. Do you also plan to document within this
effort how to run tests with podman?
> >
>
> I'll try to find resources on how to run w/o docker. I know there are
> some discussions, issues
> and PRs on the Testcontainers repo.
>