-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[mqtt.ruuvigateway] Re-enable itests after core change #15018
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try resolving the runbundles again:
@holgerfriedrich, @lolodomo just FYI, it seems that #14970 caused #15019 to fail? |
Sorry, apparently I'm mistaking mqtt.ruuvigateway and mqtt.espmilighthub. |
280a328
to
4995394
Compare
2e026ee
to
330b72d
Compare
330b72d
to
76c855a
Compare
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
@holgerfriedrich I just pushed a small addition to your branch. Is that ok with you? |
@J-N-K thanks! I could not make it work with the CI, now the test seems fine. |
...ruuvigateway.tests/src/main/java/org/openhab/binding/mqtt/ruuvigateway/RuuviGatewayTest.java
Outdated
Show resolved
Hide resolved
...ruuvigateway.tests/src/main/java/org/openhab/binding/mqtt/ruuvigateway/RuuviGatewayTest.java
Outdated
Show resolved
Hide resolved
mvn spotless:apply -Dspotless.eclipse.version=4.25 Signed-off-by: Wouter Born <github@maindrain.net>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks all!
Also-by: Jan N. Klug <github@klug.nrw> Signed-off-by: Holger Friedrich <mail@holger-friedrich.de> Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Builds for #15010 somehow do not complete. This is a try to do it separately.
Re-enable itests for mqtt.ruuvigateway.
After core change
CoreItemFactory
ctor requires aUnitProvider
.From itests the class
TestUnitProvider
is not visible, so I tried a mockup.Jenkins results need to be reviewed.