Skip to content

Commit 15b93b3

Browse files
committed
fix unit test
Signed-off-by: Bernd Weymann <bernd.weymann@gmail.com>
1 parent c7d8940 commit 15b93b3

2 files changed

Lines changed: 7 additions & 1 deletion

File tree

bundles/org.openhab.binding.dirigera/src/test/java/org/openhab/binding/dirigera/internal/handler/lights/TestLightSet.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ void testHandleCommandBrightnessRoutesToSetEndpoint() {
170170
Thing thing = handler.getThing();
171171
CallbackMock callback = (CallbackMock) handler.getCallback();
172172
assertNotNull(callback);
173-
DirigeraAPISimu api = (DirigeraAPISimu) handler.gateway().api();
173+
checkLightSetStates(callback);
174174

175175
// Brightness is only dispatched immediately when isPowered() — needs isOn:true AND online:true.
176176
// Simulate a full readDevice-style update (no isReachable key, so LightSetHandler passes it
@@ -181,6 +181,8 @@ void testHandleCommandBrightnessRoutesToSetEndpoint() {
181181
assertNotNull(onOffState);
182182
assertTrue(onOffState instanceof OnOffType);
183183
assertTrue(OnOffType.ON.equals(onOffState), "Power state");
184+
185+
DirigeraAPISimu api = (DirigeraAPISimu) handler.gateway().api();
184186
api.clear();
185187

186188
handler.handleCommand(new ChannelUID(thing.getUID(), CHANNEL_LIGHT_BRIGHTNESS), new PercentType(75));

bundles/org.openhab.binding.dirigera/src/test/java/org/openhab/binding/dirigera/internal/mock/CallbackMock.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,8 @@
4141
import org.openhab.core.types.Command;
4242
import org.openhab.core.types.State;
4343
import org.openhab.core.types.TimeSeries;
44+
import org.slf4j.Logger;
45+
import org.slf4j.LoggerFactory;
4446

4547
/**
4648
* The {@link CallbackMock} basic DeviceHandler for all devices
@@ -49,6 +51,7 @@
4951
*/
5052
@NonNullByDefault
5153
public class CallbackMock implements ThingHandlerCallback {
54+
private final Logger logger = LoggerFactory.getLogger(CallbackMock.class);
5255
private static final int STATUS_DURATION_TIMEOUT_SEC = 5;
5356
private @Nullable Bridge bridge;
5457
private ThingStatusInfo statusInfo = ThingStatusInfoBuilder.create(ThingStatus.UNINITIALIZED).build();
@@ -114,6 +117,7 @@ public void clear() {
114117

115118
@Override
116119
public void stateUpdated(ChannelUID channelUID, State state) {
120+
logger.warn("State updated for channel {}: {}", channelUID, state);
117121
synchronized (stateMap) {
118122
stateMap.put(channelUID.getAsString(), state);
119123
stateMap.notifyAll();

0 commit comments

Comments
 (0)