Browse Source

changed exception text for mapping based rule nodes

pull/8661/head
ShvaykaD 3 years ago
parent
commit
d49c83b264
  1. 2
      rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbAbstractGetEntityAttrNode.java
  2. 2
      rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbGetOriginatorFieldsNode.java
  3. 2
      rule-engine/rule-engine-components/src/test/java/org/thingsboard/rule/engine/metadata/TbGetCustomerAttributeNodeTest.java
  4. 2
      rule-engine/rule-engine-components/src/test/java/org/thingsboard/rule/engine/metadata/TbGetOriginatorFieldsNodeTest.java
  5. 2
      rule-engine/rule-engine-components/src/test/java/org/thingsboard/rule/engine/metadata/TbGetRelatedAttributeNodeTest.java
  6. 2
      rule-engine/rule-engine-components/src/test/java/org/thingsboard/rule/engine/metadata/TbGetTenantAttributeNodeTest.java

2
rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbAbstractGetEntityAttrNode.java

@ -50,7 +50,7 @@ public abstract class TbAbstractGetEntityAttrNode<T extends EntityId> extends Tb
protected void checkIfMappingIsNotEmptyOrElseThrow(Map<String, String> attrMapping) throws TbNodeException { protected void checkIfMappingIsNotEmptyOrElseThrow(Map<String, String> attrMapping) throws TbNodeException {
if (attrMapping == null || attrMapping.isEmpty()) { if (attrMapping == null || attrMapping.isEmpty()) {
throw new TbNodeException("At least one attribute mapping should be specified!"); throw new TbNodeException("At least one mapping entry should be specified!");
} }
} }

2
rule-engine/rule-engine-components/src/main/java/org/thingsboard/rule/engine/metadata/TbGetOriginatorFieldsNode.java

@ -51,7 +51,7 @@ public class TbGetOriginatorFieldsNode extends TbAbstractNodeWithFetchTo<TbGetOr
protected TbGetOriginatorFieldsConfiguration loadNodeConfiguration(TbNodeConfiguration configuration) throws TbNodeException { protected TbGetOriginatorFieldsConfiguration loadNodeConfiguration(TbNodeConfiguration configuration) throws TbNodeException {
var config = TbNodeUtils.convert(configuration, TbGetOriginatorFieldsConfiguration.class); var config = TbNodeUtils.convert(configuration, TbGetOriginatorFieldsConfiguration.class);
if (config.getFieldsMapping() == null || config.getFieldsMapping().isEmpty()) { if (config.getFieldsMapping() == null || config.getFieldsMapping().isEmpty()) {
throw new TbNodeException("At least one field mapping should be specified!"); throw new TbNodeException("At least one mapping entry should be specified!");
} }
return config; return config;
} }

2
rule-engine/rule-engine-components/src/test/java/org/thingsboard/rule/engine/metadata/TbGetCustomerAttributeNodeTest.java

@ -175,7 +175,7 @@ public class TbGetCustomerAttributeNodeTest {
@Test @Test
public void givenEmptyAttributesMapping_whenInit_thenException() { public void givenEmptyAttributesMapping_whenInit_thenException() {
// GIVEN // GIVEN
var expectedExceptionMessage = "At least one attribute mapping should be specified!"; var expectedExceptionMessage = "At least one mapping entry should be specified!";
config.setAttrMapping(Collections.emptyMap()); config.setAttrMapping(Collections.emptyMap());
nodeConfiguration = new TbNodeConfiguration(JacksonUtil.valueToTree(config)); nodeConfiguration = new TbNodeConfiguration(JacksonUtil.valueToTree(config));

2
rule-engine/rule-engine-components/src/test/java/org/thingsboard/rule/engine/metadata/TbGetOriginatorFieldsNodeTest.java

@ -303,7 +303,7 @@ public class TbGetOriginatorFieldsNodeTest {
var exception = assertThrows(TbNodeException.class, () -> node.init(ctxMock, nodeConfiguration)); var exception = assertThrows(TbNodeException.class, () -> node.init(ctxMock, nodeConfiguration));
// THEN // THEN
assertThat(exception.getMessage()).isEqualTo("At least one field mapping should be specified!"); assertThat(exception.getMessage()).isEqualTo("At least one mapping entry should be specified!");
verify(ctxMock, never()).tellSuccess(any()); verify(ctxMock, never()).tellSuccess(any());
} }

2
rule-engine/rule-engine-components/src/test/java/org/thingsboard/rule/engine/metadata/TbGetRelatedAttributeNodeTest.java

@ -197,7 +197,7 @@ public class TbGetRelatedAttributeNodeTest {
@Test @Test
public void givenEmptyAttributesMapping_whenInit_thenException() { public void givenEmptyAttributesMapping_whenInit_thenException() {
// GIVEN // GIVEN
var expectedExceptionMessage = "At least one attribute mapping should be specified!"; var expectedExceptionMessage = "At least one mapping entry should be specified!";
config.setAttrMapping(Collections.emptyMap()); config.setAttrMapping(Collections.emptyMap());
nodeConfiguration = new TbNodeConfiguration(JacksonUtil.valueToTree(config)); nodeConfiguration = new TbNodeConfiguration(JacksonUtil.valueToTree(config));

2
rule-engine/rule-engine-components/src/test/java/org/thingsboard/rule/engine/metadata/TbGetTenantAttributeNodeTest.java

@ -156,7 +156,7 @@ public class TbGetTenantAttributeNodeTest {
@Test @Test
public void givenEmptyAttributesMapping_whenInit_thenException() { public void givenEmptyAttributesMapping_whenInit_thenException() {
// GIVEN // GIVEN
var expectedExceptionMessage = "At least one attribute mapping should be specified!"; var expectedExceptionMessage = "At least one mapping entry should be specified!";
config.setAttrMapping(Collections.emptyMap()); config.setAttrMapping(Collections.emptyMap());
nodeConfiguration = new TbNodeConfiguration(JacksonUtil.valueToTree(config)); nodeConfiguration = new TbNodeConfiguration(JacksonUtil.valueToTree(config));

Loading…
Cancel
Save