From e227caac514e4fa5b117d59687419e0455a0ffcb Mon Sep 17 00:00:00 2001 From: Andrii Landiak Date: Fri, 5 Jan 2024 14:27:36 +0200 Subject: [PATCH] Provide test for fixed bug --- .../server/edge/DeviceEdgeTest.java | 65 ++++++++++++++++++- 1 file changed, 63 insertions(+), 2 deletions(-) diff --git a/application/src/test/java/org/thingsboard/server/edge/DeviceEdgeTest.java b/application/src/test/java/org/thingsboard/server/edge/DeviceEdgeTest.java index 7bb05cb4d5..c9a7a97c9a 100644 --- a/application/src/test/java/org/thingsboard/server/edge/DeviceEdgeTest.java +++ b/application/src/test/java/org/thingsboard/server/edge/DeviceEdgeTest.java @@ -25,8 +25,10 @@ import io.netty.handler.codec.mqtt.MqttQoS; import org.awaitility.Awaitility; import org.junit.Assert; import org.junit.Test; +import org.springframework.beans.factory.annotation.Autowired; import org.springframework.test.context.TestPropertySource; import org.thingsboard.common.util.JacksonUtil; +import org.thingsboard.server.common.adaptor.JsonConverter; import org.thingsboard.server.common.data.Customer; import org.thingsboard.server.common.data.DataConstants; import org.thingsboard.server.common.data.Device; @@ -44,15 +46,17 @@ import org.thingsboard.server.common.data.edge.EdgeEventActionType; import org.thingsboard.server.common.data.edge.EdgeEventType; import org.thingsboard.server.common.data.id.CustomerId; import org.thingsboard.server.common.data.id.DeviceId; +import org.thingsboard.server.common.data.id.EdgeId; import org.thingsboard.server.common.data.id.EntityId; import org.thingsboard.server.common.data.ota.OtaPackageType; import org.thingsboard.server.common.data.page.PageData; import org.thingsboard.server.common.data.page.PageLink; +import org.thingsboard.server.common.data.page.TimePageLink; import org.thingsboard.server.common.data.security.DeviceCredentials; import org.thingsboard.server.common.data.security.DeviceCredentialsType; import org.thingsboard.server.common.data.tenant.profile.DefaultTenantProfileConfiguration; import org.thingsboard.server.common.msg.session.FeatureType; -import org.thingsboard.server.common.adaptor.JsonConverter; +import org.thingsboard.server.dao.edge.EdgeService; import org.thingsboard.server.dao.service.DaoSqlTest; import org.thingsboard.server.gen.edge.v1.AttributesRequestMsg; import org.thingsboard.server.gen.edge.v1.DeviceCredentialsRequestMsg; @@ -86,6 +90,9 @@ public class DeviceEdgeTest extends AbstractEdgeTest { private static final String DEFAULT_DEVICE_TYPE = "default"; + @Autowired + protected EdgeService edgeService; + @Test public void testDevices() throws Exception { // create device and assign to edge; update device @@ -769,6 +776,45 @@ public class DeviceEdgeTest extends AbstractEdgeTest { } } + @Test + public void testVerifyProcessCorrectEdgeUpdateToDeviceActorOnUnassignFromDifferentEdge() throws Exception { + Device device = saveDeviceOnCloudAndVerifyDeliveryToEdge(); + + // assign device to another edge + Edge tmpEdge = doPost("/api/edge", constructEdge("Test Tmp Edge", "test"), Edge.class); + doPost("/api/edge/" + tmpEdge.getUuidId() + + "/device/" + device.getUuidId(), Device.class); + List relatedEdgeIds = edgeService.findAllRelatedEdgeIds(tenantId, device.getId()); + Assert.assertEquals(2, relatedEdgeIds.size()); + + // unassign device from edge + doDelete("/api/edge/" + edge.getUuidId() + + "/device/" + device.getUuidId(), Device.class); + relatedEdgeIds = edgeService.findAllRelatedEdgeIds(tenantId, device.getId()); + Assert.assertEquals(1, relatedEdgeIds.size()); + Assert.assertEquals(tmpEdge.getId(), relatedEdgeIds.get(0)); + + // clean up stored edge events + edgeEventService.cleanupEvents(1); + + // perform rpc call to verify edgeId in DeviceActorMessageProcessor updated properly + doPostAsync( + "/api/rpc/oneway/" + device.getId().getId().toString(), + JacksonUtil.toString(createDefaultRpc()), + String.class, + status().isOk()); + + PageData result = edgeEventService.findEdgeEvents(tenantId, tmpEdge.getId(), 0L, null, new TimePageLink(1)); + EdgeEvent edgeEvent = result.getData().get(0); + Assert.assertEquals(EdgeEventActionType.RPC_CALL, edgeEvent.getAction()); + Assert.assertEquals(EdgeEventType.DEVICE, edgeEvent.getType()); + Assert.assertEquals(tmpEdge.getId(), edgeEvent.getEdgeId()); + Assert.assertEquals(device.getId().getId(), edgeEvent.getEntityId()); + + // clean up tmp edge + doDelete("/api/edge/" + tmpEdge.getId().getId().toString()).andExpect(status().isOk()); + } + private Device buildDeviceForUplinkMsg(String name, String type) { Device device = new Device(); device.setId(new DeviceId(UUID.randomUUID())); @@ -778,7 +824,6 @@ public class DeviceEdgeTest extends AbstractEdgeTest { return device; } - private DeviceCredentials buildDeviceCredentialsForUplinkMsg(DeviceId deviceId) { DeviceCredentials deviceCredentials = new DeviceCredentials(); deviceCredentials.setDeviceId(deviceId); @@ -786,4 +831,20 @@ public class DeviceEdgeTest extends AbstractEdgeTest { deviceCredentials.setCredentialsType(DeviceCredentialsType.ACCESS_TOKEN); return deviceCredentials; } + + private ObjectNode createDefaultRpc() { + ObjectNode rpc = JacksonUtil.newObjectNode(); + rpc.put("method", "setGpio"); + + ObjectNode params = JacksonUtil.newObjectNode(); + + params.put("pin", 7); + params.put("value", 1); + + rpc.set("params", params); + rpc.put("persistent", true); + rpc.put("timeout", 5000); + + return rpc; + } }