Skip to content

Commit 61eefb8

Browse files
committed
Config fixes
1 parent f2574d2 commit 61eefb8

File tree

2 files changed

+13
-8
lines changed

2 files changed

+13
-8
lines changed

modules/hivemq-edge-module-s7/src/main/java/com/hivemq/edge/adapters/s7/S7ProtocolAdapterFactory.java

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
import com.hivemq.adapter.sdk.api.ProtocolAdapterInformation;
2121
import com.hivemq.adapter.sdk.api.config.ProtocolAdapterConfig;
2222
import com.hivemq.adapter.sdk.api.factories.ProtocolAdapterFactory;
23+
import com.hivemq.adapter.sdk.api.factories.ProtocolAdapterFactoryInput;
2324
import com.hivemq.adapter.sdk.api.model.ProtocolAdapterInput;
2425
import com.hivemq.edge.adapters.s7.config.S7AdapterConfig;
2526
import org.jetbrains.annotations.NotNull;
@@ -33,11 +34,7 @@
3334
*/
3435
public class S7ProtocolAdapterFactory implements ProtocolAdapterFactory<S7AdapterConfig> {
3536

36-
final boolean writingEnabled;
37-
38-
public S7ProtocolAdapterFactory(final boolean writingEnabled) {
39-
this.writingEnabled = writingEnabled;
40-
}
37+
public S7ProtocolAdapterFactory(@NotNull final ProtocolAdapterFactoryInput input) {}
4138

4239
@Override
4340
public @NotNull ProtocolAdapterInformation getInformation() {

modules/hivemq-edge-module-s7/src/test/java/com/hivemq/edge/adapters/s7/config/S7AdapterConfigTest.java

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import com.fasterxml.jackson.databind.ObjectMapper;
44
import com.hivemq.adapter.sdk.api.config.MqttUserProperty;
5+
import com.hivemq.adapter.sdk.api.factories.ProtocolAdapterFactoryInput;
56
import com.hivemq.adapter.sdk.api.tag.Tag;
67
import com.hivemq.configuration.entity.HiveMQConfigEntity;
78
import com.hivemq.configuration.reader.ConfigFileReaderWriter;
@@ -23,6 +24,7 @@
2324
import static com.hivemq.protocols.ProtocolAdapterUtils.createProtocolAdapterMapper;
2425
import static org.assertj.core.api.Assertions.assertThat;
2526
import static org.mockito.Mockito.mock;
27+
import static org.mockito.Mockito.when;
2628

2729
class S7AdapterConfigTest {
2830

@@ -36,7 +38,9 @@ public void convertConfigObject_fullConfig_valid() throws Exception {
3638
final HiveMQConfigEntity configEntity = loadConfig(path);
3739
final Map<String, Object> adapters = configEntity.getProtocolAdapterConfig();
3840

39-
final S7ProtocolAdapterFactory s7ProtocolAdapterFactory = new S7ProtocolAdapterFactory(false);
41+
final ProtocolAdapterFactoryInput mockInput = mock(ProtocolAdapterFactoryInput.class);
42+
when(mockInput.isWritingEnabled()).thenReturn(false);
43+
final S7ProtocolAdapterFactory s7ProtocolAdapterFactory = new S7ProtocolAdapterFactory(mockInput);
4044

4145
final AdapterConfigAndTags adapterConfigAndTags =
4246
AdapterConfigAndTags.fromAdapterConfigMap((Map<String, Object>) adapters.get(PROTOCOL_ID),
@@ -90,7 +94,9 @@ public void convertConfigObject_defaults_valid() throws Exception {
9094
final HiveMQConfigEntity configEntity = loadConfig(path);
9195
final Map<String, Object> adapters = configEntity.getProtocolAdapterConfig();
9296

93-
final S7ProtocolAdapterFactory s7ProtocolAdapterFactory = new S7ProtocolAdapterFactory(false);
97+
final ProtocolAdapterFactoryInput mockInput = mock(ProtocolAdapterFactoryInput.class);
98+
when(mockInput.isWritingEnabled()).thenReturn(false);
99+
final S7ProtocolAdapterFactory s7ProtocolAdapterFactory = new S7ProtocolAdapterFactory(mockInput);
94100

95101
final AdapterConfigAndTags adapterConfigAndTags =
96102
AdapterConfigAndTags.fromAdapterConfigMap((Map<String, Object>) adapters.get(PROTOCOL_ID),
@@ -152,7 +158,9 @@ public void unconvertConfigObject_full_valid() {
152158
false,
153159
List.of(pollingContext));
154160

155-
final S7ProtocolAdapterFactory s7ProtocolAdapterFactory = new S7ProtocolAdapterFactory(false);
161+
final ProtocolAdapterFactoryInput mockInput = mock(ProtocolAdapterFactoryInput.class);
162+
when(mockInput.isWritingEnabled()).thenReturn(false);
163+
final S7ProtocolAdapterFactory s7ProtocolAdapterFactory = new S7ProtocolAdapterFactory(mockInput);
156164
final Map<String, Object> config =
157165
s7ProtocolAdapterFactory.unconvertConfigObject(mapper, s7AdapterConfig);
158166

0 commit comments

Comments
 (0)