Skip to content

Commit 4136805

Browse files
authored
Bug/76 fix featuretoggle (#77)
* Added JsonIgnore for proper serialization. * Added assertion for feature toggle test
1 parent e0a237f commit 4136805

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

azd/src/main/java/org/azd/core/types/ProjectFeature.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
package org.azd.core.types;
22

3+
import com.fasterxml.jackson.annotation.JsonIgnore;
34
import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
45
import com.fasterxml.jackson.annotation.JsonProperty;
56
import org.azd.abstractions.serializer.SerializableEntity;
@@ -44,6 +45,7 @@ public void setState(String state) {
4445
this.state = state;
4546
}
4647

48+
@JsonIgnore
4749
public Optional<Boolean> getStateAsBoolean() {
4850
if (state == null) {
4951
return Optional.empty();

azd/src/test/java/org/azd/unittests/CoreRequestBuilderTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,8 @@ public void shouldReadProjectFeatures() throws AzDException {
199199
public void shouldToggleFeature() throws AzDException {
200200
var project = client.core().projects().get(testConfiguration.properties.core.projectName);
201201
Assume.assumeNotNull(project);
202-
client.helpers().featureManagement()
202+
var resp = client.helpers().featureManagement()
203203
.featureToggle(project.getId(), FeatureManagement.TEST_PLANS, false);
204+
assert !resp.getStateAsBoolean().get();
204205
}
205206
}

0 commit comments

Comments
 (0)