Browse Source

fixed test

pull/12721/head
dashevchenko 2 years ago
parent
commit
46005a957a
  1. 2
      application/src/main/java/org/thingsboard/server/service/edqs/EdqsSyncService.java
  2. 5
      common/data/src/main/java/org/thingsboard/server/common/data/edqs/fields/TenantFields.java
  3. 6
      common/edqs/src/main/java/org/thingsboard/server/edqs/util/RepositoryUtils.java
  4. 4
      edqs/src/test/java/org/thingsboard/server/edqs/repo/RepositoryUtilsTest.java

2
application/src/main/java/org/thingsboard/server/service/edqs/EdqsSyncService.java

@ -166,7 +166,7 @@ public abstract class EdqsSyncService {
if (entityIdInfo != null) {
process(entityIdInfo.tenantId(), RELATION, relation.toData());
} else {
log.info("Relation from entity not found: " + relation.getFromId());
log.info("Relation from id not found: {} ", relation);
}
}
}

5
common/data/src/main/java/org/thingsboard/server/common/data/edqs/fields/TenantFields.java

@ -59,4 +59,9 @@ public class TenantFields extends AbstractEntityFields {
public TenantFields(UUID id, Long version) {
super(id, 0L, null, version);
}
@Override
public UUID getTenantId() {
return getId();
}
}

6
common/edqs/src/main/java/org/thingsboard/server/edqs/util/RepositoryUtils.java

@ -207,11 +207,13 @@ public class RepositoryUtils {
default -> throw new IllegalStateException();
};
}
DataPoint dp = entity.getDataPoint(keyFilter.key(), null);
DataKey dataKey = keyFilter.key();
DataPoint dp = entity.getDataPoint(dataKey, null);
boolean checkResult = switch (valueType) {
case STRING -> {
String str = dp != null ? dp.valueToString() : null;
yield str != null && checkKeyFilter(str, keyFilter.predicate());
yield (dataKey.type() == EntityKeyType.ENTITY_FIELD) ? (str == null || checkKeyFilter(str, keyFilter.predicate())) :
(str != null && checkKeyFilter(str, keyFilter.predicate()));
}
case BOOLEAN -> {
Boolean booleanValue = dp != null ? dp.getBool() : null;

4
edqs/src/test/java/org/thingsboard/server/edqs/repo/RepositoryUtilsTest.java

@ -54,7 +54,7 @@ import static org.assertj.core.api.Assertions.assertThat;
public class RepositoryUtilsTest {
private static Stream<Arguments> deviceNameFilters() {
return Stream.of(Arguments.of(null, getNameFilter(StringOperation.STARTS_WITH, "lora"), false),
return Stream.of(Arguments.of(null, getNameFilter(StringOperation.STARTS_WITH, "lora"), true),
Arguments.of("loranet device 123", getNameFilter(StringOperation.STARTS_WITH, "lora"), true),
Arguments.of("loranet 123", getNameFilter(StringOperation.STARTS_WITH, "ra"), false),
Arguments.of("loranet 123", getNameFilter(StringOperation.ENDS_WITH, "123"), true),
@ -132,7 +132,7 @@ public class RepositoryUtilsTest {
}
private static Stream<Arguments> deviceNameComplexFilters() {
return Stream.of(Arguments.of(null, List.of(getComplexComplexDeviceNameFilter(StringOperation.STARTS_WITH, "lo", ComplexOperation.AND, StringOperation.ENDS_WITH, "123")), false),
return Stream.of(Arguments.of(null, List.of(getComplexComplexDeviceNameFilter(StringOperation.STARTS_WITH, "lo", ComplexOperation.AND, StringOperation.ENDS_WITH, "123")), true),
Arguments.of("loranet 123", List.of(getComplexComplexDeviceNameFilter(StringOperation.STARTS_WITH, "lo", ComplexOperation.AND, StringOperation.ENDS_WITH, "123")), true),
Arguments.of("loranet 123", List.of(getComplexComplexDeviceNameFilter(StringOperation.STARTS_WITH, "lo", ComplexOperation.AND, StringOperation.ENDS_WITH, "124")), false),
Arguments.of("loranet 123", List.of(getComplexComplexDeviceNameFilter(StringOperation.STARTS_WITH, "lo", ComplexOperation.OR, StringOperation.STARTS_WITH, "net")), true),

Loading…
Cancel
Save