Skip to content

Commit 934075e

Browse files
committed
Implemented Server GetConfiguration feature.
1 parent 5eb694b commit 934075e

File tree

4 files changed

+23
-3
lines changed

4 files changed

+23
-3
lines changed

ocpp-v1_6-test/src/main/java/eu/chargetime/ocpp/test/FakeCentralSystem.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,4 +171,14 @@ public boolean hasReceivedDataTransferConfirmation() {
171171
public boolean hasHandledDataTransferRequest() {
172172
return receivedRequest instanceof DataTransferRequest;
173173
}
174+
175+
public void sendGetConfigurationRequest(String... key) throws Exception {
176+
GetConfigurationRequest request = new GetConfigurationRequest();
177+
request.setKey(key);
178+
server.send(sessionIndex, request).whenComplete((confirmation, throwable) -> receivedConfirmation = confirmation);
179+
}
180+
181+
public boolean hasReceivedGetConfigurationConfirmation() {
182+
return receivedConfirmation instanceof GetConfigurationConfirmation;
183+
}
174184
}

ocpp-v1_6-test/src/test/groovy/eu/chargetime/ocpp/test/core/GetConfiguration.groovy renamed to ocpp-v1_6-test/src/test/groovy/eu/chargetime/ocpp/test/core/GetConfigurationSpec.groovy

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
package eu.chargetime.ocpp.test.core
22

3+
import eu.chargetime.ocpp.test.FakeCentralSystem
34
import eu.chargetime.ocpp.test.FakeChargePoint
4-
import eu.chargetime.ocpp.test.OldFakeCentralSystem
55
import spock.lang.Shared
66
import spock.lang.Specification
77
import spock.util.concurrent.PollingConditions
88

9-
class GetConfiguration extends Specification
9+
class GetConfigurationSpec extends Specification
1010
{
1111
@Shared
12-
OldFakeCentralSystem centralSystem = OldFakeCentralSystem.getInstance();
12+
FakeCentralSystem centralSystem = FakeCentralSystem.getInstance();
1313
@Shared FakeChargePoint chargePoint = new FakeChargePoint();
1414

1515
def setupSpec() {

ocpp-v1_6/src/main/java/eu/chargetime/ocpp/feature/profile/ServerCoreProfile.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ public ServerCoreProfile(ServerCoreEventHandler handler) {
4848
features.add(new ChangeConfigurationFeature(this));
4949
features.add(new ClearCacheFeature(this));
5050
features.add(new DataTransferFeature(this));
51+
features.add(new GetConfigurationFeature(this));
5152
}
5253

5354
@Override

ocpp-v1_6/src/test/java/eu/chargetime/ocpp/feature/profile/test/ServerCoreProfileTest.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,15 @@ public void handleRequest_aDataTransferRequest_callsHandleDataTransferRequest()
146146
verify(handler, times(1)).handleDataTransferRequest(eq(sessionId), eq(request));
147147
}
148148

149+
@Test
150+
public void getFeatureList_containsGetConfigurationFeature() {
151+
// When
152+
Feature[] features = core.getFeatureList();
153+
154+
// Then
155+
assertThat(findFeature(features, "GetConfiguration"), is(instanceOf(GetConfigurationFeature.class)));
156+
}
157+
149158
private Feature findFeature(Feature[] features, String action) {
150159
Feature output = null;
151160
for (Feature feature : features) {

0 commit comments

Comments
 (0)