Skip to content

Commit

Permalink
Issue #106: Added getter for optional FeatureDefinition.
Browse files Browse the repository at this point in the history
Signed-off-by: Juergen Fickel <juergen.fickel@bosch.io>
  • Loading branch information
Juergen Fickel committed Dec 9, 2021
1 parent e32db7f commit e2b5217
Show file tree
Hide file tree
Showing 2 changed files with 61 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
import org.eclipse.ditto.protocol.TopicPath;
import org.eclipse.ditto.things.model.Attributes;
import org.eclipse.ditto.things.model.Feature;
import org.eclipse.ditto.things.model.FeatureDefinition;
import org.eclipse.ditto.things.model.Features;
import org.eclipse.ditto.things.model.Thing;
import org.eclipse.ditto.things.model.ThingDefinition;
Expand Down Expand Up @@ -262,6 +263,28 @@ Optional<ThingDefinition> getThingDefinition() {
return result;
}

Optional<FeatureDefinition> getFeatureDefinition() {
final Optional<FeatureDefinition> result;
final Payload payload = adaptable.getPayload();
final Optional<JsonValue> payloadValueOptional = payload.getValue();
if (payloadValueOptional.isPresent()) {
final JsonValue jsonValue = payloadValueOptional.get();
if (jsonValue.isArray()) {
result = Optional.of(ThingsModelFactory.newFeatureDefinition(jsonValue.asArray()));
} else {
throw new IllegalAdaptableException(
MessageFormat.format("Payload value is not a {0} as JSON array but <{1}>.",
FeatureDefinition.class.getSimpleName(),
jsonValue),
adaptable.getDittoHeaders()
);
}
} else {
result = Optional.empty();
}
return result;
}

Optional<PolicyId> getPolicyId() {
final Optional<PolicyId> result;
final Payload payload = adaptable.getPayload();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
import org.eclipse.ditto.protocol.TopicPath;
import org.eclipse.ditto.things.model.Attributes;
import org.eclipse.ditto.things.model.Feature;
import org.eclipse.ditto.things.model.FeatureDefinition;
import org.eclipse.ditto.things.model.FeatureProperties;
import org.eclipse.ditto.things.model.Features;
import org.eclipse.ditto.things.model.Thing;
Expand Down Expand Up @@ -483,6 +484,43 @@ public void getThingDefinitionReturnsEmptyOptionalIfPayloadContainsNoValue() {
assertThat(underTest.getThingDefinition()).isEmpty();
}

@Test
public void getFeatureDefinitionReturnsExpectedThingDefinitionIfContainedInPayload() {
final FeatureDefinition featureDefinition =
ThingsModelFactory.newFeatureDefinition(JsonArray.of(JsonValue.of("example:test:definition")));
final Payload payload = ProtocolFactory.newPayloadBuilder()
.withValue(featureDefinition.toJson())
.build();
Mockito.when(adaptable.getPayload()).thenReturn(payload);
final MappingContext underTest = MappingContext.of(adaptable);

assertThat(underTest.getFeatureDefinition()).contains(featureDefinition);
}

@Test
public void getFeatureDefinitionThrowsExceptionIfPayloadContainsValueThatIsNoJsonString() {
final JsonValue value = JsonValue.of(true);
final Payload payload = ProtocolFactory.newPayloadBuilder()
.withValue(value)
.build();
Mockito.when(adaptable.getPayload()).thenReturn(payload);
final MappingContext underTest = MappingContext.of(adaptable);

assertThatExceptionOfType(IllegalAdaptableException.class)
.isThrownBy(underTest::getFeatureDefinition)
.withMessage("Payload value is not a FeatureDefinition as JSON array but <%s>.", value)
.withNoCause();
}

@Test
public void getFeatureDefinitionReturnsEmptyOptionalIfPayloadContainsNoValue() {
final Payload payload = ProtocolFactory.newPayloadBuilder().build();
Mockito.when(adaptable.getPayload()).thenReturn(payload);
final MappingContext underTest = MappingContext.of(adaptable);

assertThat(underTest.getFeatureDefinition()).isEmpty();
}

@Test
public void getPolicyIdReturnsPolicyIfContainedInPayload() {
final PolicyId policyId = PolicyId.inNamespaceWithRandomName("org.ditto");
Expand Down

0 comments on commit e2b5217

Please sign in to comment.