search

hal / testsuite.next

Testsuite for HAL
Apache License 2.0
2 stars 26 forks source link

HAL-1779 Rework broadcast and discovery group tests #171

Closed TomasHofman closed 2 years ago

TomasHofman commented 2 years ago

to reflect resources split into jgroups- and socket- variants

https://issues.redhat.com/browse/HAL-1779 https://issues.redhat.com/browse/EAP7-1886

@hpehl this is testsuite change related to HAL-1779. I developed it against current HAL main branch, it's going to fail against HAL 3.4.1 which is currently in the pom.

All the broadcast & discovery group tests should pass, but there are other tests failing, which are not relevant to changes in HAL-1779.

hpehl commented 2 years ago

Thanks @TomasHofman

jbliznak commented 2 years ago

this probably shouldn't have been merged (yet) - the TS can't compile now ..

TomasHofman commented 2 years ago

I did try to build with mvn clean install -DskipTests -Pbasic and it does compile now.

hpehl commented 2 years ago

@jbliznak you're right. I merged this too soon. Sorry for that.

I've "fixed" the test suite by commenting on some code that uses constants from HAL upstream. I'll undo that as soon as HAL 3.5.13.Final has been released.