Skip to content

feat(rc):Add exposurePercent to ExperimentValues #1201

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
rathovarun1032 wants to merge 17 commits into
base: main
Choose a base branch
from
Open

feat(rc):Add exposurePercent to ExperimentValues #1201

Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
4730f73
Add exposurePercent to ExperimentValues
Mar 24, 2026
9b862b1
fix comments
Mar 24, 2026
3244212
fix build
Mar 25, 2026
cc35f45
Apply suggestions from code review
rathovarun1032 Mar 25, 2026
095b467
fix build
Mar 25, 2026
2110cd1
fix build
Mar 25, 2026
0aed763
fix build
Mar 25, 2026
4ddfa86
fix build
Mar 25, 2026
dca0f0a
fix build
Mar 25, 2026
9661724
fix build
Mar 25, 2026
8b9b722
fix build
Mar 25, 2026
43786bd
resolved comments
Mar 31, 2026
221c248
resolved comments
Apr 2, 2026
1678a12
resolved comments
Apr 2, 2026
610cc75
resolved comments
Apr 2, 2026
f17d637
Update TemplateResponse.java
rathovarun1032 Apr 11, 2026
40e1202
Update Template.java
rathovarun1032 Apr 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
132 changes: 73 additions & 59 deletions src/main/java/com/google/firebase/remoteconfig/ParameterValue.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,14 +26,12 @@
import com.google.firebase.remoteconfig.internal.TemplateResponse.ParameterValueResponse;
import com.google.firebase.remoteconfig.internal.TemplateResponse.PersonalizationValueResponse;
import com.google.firebase.remoteconfig.internal.TemplateResponse.RolloutValueResponse;

import java.util.ArrayList;
import java.util.List;
import java.util.Objects;

/**
* Represents a Remote Config parameter value that can be used in a {@link Template}.
*/
/** Represents a Remote Config parameter value
* that can be used in a
* {@link Template}. */
public abstract class ParameterValue {

/**
Expand Down Expand Up @@ -82,17 +80,18 @@ public static PersonalizationValue ofPersonalization(String personalizationId) {
*
* @param experimentId The experiment ID.
* @param variantValues The list of experiment variant values.
* @param exposurePercent The exposure percentage of the experiment.
* @return A {@link ParameterValue.ExperimentValue} instance.
*/
public static ExperimentValue ofExperiment(String experimentId,
List<ExperimentVariantValue> variantValues) {
return new ExperimentValue(experimentId, variantValues);
public static ExperimentValue ofExperiment(
String experimentId, List<ExperimentVariantValue> variantValues, double exposurePercent) {
return new ExperimentValue(experimentId, variantValues, exposurePercent);
}

abstract ParameterValueResponse toParameterValueResponse();

static ParameterValue fromParameterValueResponse(
@NonNull ParameterValueResponse parameterValueResponse) {
@NonNull ParameterValueResponse parameterValueResponse) {
checkNotNull(parameterValueResponse);
if (parameterValueResponse.isUseInAppDefault()) {
return ParameterValue.inAppDefault();
Expand All @@ -102,7 +101,7 @@ static ParameterValue fromParameterValueResponse(
// Protobuf serialization does not set values for fields on the wire when
// they are equal to the default value for the field type. When deserializing,
// can appear as the value not being set. Explicitly handle default value for
// the percent field since 0 is a valid value.
// the percent field since 0 is a valid value.
double percent = 0;
if (rv.getPercent() != null) {
percent = rv.getPercent();
Expand All @@ -115,18 +114,26 @@ static ParameterValue fromParameterValueResponse(
}
if (parameterValueResponse.getExperimentValue() != null) {
ExperimentValueResponse ev = parameterValueResponse.getExperimentValue();
List<ExperimentVariantValue> variantValues = ev.getExperimentVariantValues().stream()
.map(evv -> new ExperimentVariantValue(
evv.getVariantId(), evv.getValue(), evv.getNoChange()))
.collect(toList());
return ParameterValue.ofExperiment(ev.getExperimentId(), variantValues);
List<ExperimentVariantValue> variantValues =
ev.getExperimentVariantValues().stream()
.map(
evv ->
new ExperimentVariantValue(
evv.getVariantId(), evv.getValue(), evv.getNoChange()))
.collect(toList());
// Handle null exposurePercent by defaulting to 0
double exposurePercent = 0;
if (ev.getExposurePercent() != null) {
exposurePercent = ev.getExposurePercent();
}
return ParameterValue.ofExperiment(
ev.getExperimentId(), variantValues, exposurePercent);
}
return ParameterValue.of(parameterValueResponse.getValue());
}

/**
* Represents an explicit Remote Config parameter value with a value that the
* parameter is set to.
* Represents an explicit Remote Config parameter value with a value that the parameter is set to.
*/
public static final class Explicit extends ParameterValue {

Expand All @@ -147,8 +154,7 @@ public String getValue() {

@Override
ParameterValueResponse toParameterValueResponse() {
return new ParameterValueResponse()
.setValue(this.value);
return new ParameterValueResponse().setValue(this.value);
}

@Override
Expand All @@ -169,9 +175,7 @@ public int hashCode() {
}
}

/**
* Represents an in app default parameter value.
*/
/** Represents an in app default parameter value. */
public static final class InAppDefault extends ParameterValue {

@Override
Expand All @@ -191,9 +195,7 @@ public boolean equals(Object o) {
}
}

/**
* Represents a Rollout value.
*/
/** Represents a Rollout value. */
public static final class RolloutValue extends ParameterValue {
private final String rolloutId;
private final String value;
Expand Down Expand Up @@ -224,8 +226,8 @@ public String getValue() {
}

/**
* Gets the rollout percentage representing the exposure of rollout value
* in the target audience.
* Gets the rollout percentage representing the exposure of rollout value in the target
* audience.
*
* @return Percentage of audience exposed to the rollout
*/
Expand All @@ -235,11 +237,12 @@ public double getPercent() {

@Override
ParameterValueResponse toParameterValueResponse() {
return new ParameterValueResponse().setRolloutValue(
return new ParameterValueResponse()
.setRolloutValue(
new RolloutValueResponse()
.setRolloutId(this.rolloutId)
.setValue(this.value)
.setPercent(this.percent));
.setRolloutId(this.rolloutId)
.setValue(this.value)
.setPercent(this.percent));
}

@Override
Expand All @@ -252,8 +255,8 @@ public boolean equals(Object o) {
}
RolloutValue that = (RolloutValue) o;
return Double.compare(that.percent, percent) == 0
&& Objects.equals(rolloutId, that.rolloutId)
&& Objects.equals(value, that.value);
&& Objects.equals(rolloutId, that.rolloutId)
&& Objects.equals(value, that.value);
}

@Override
Expand All @@ -262,9 +265,7 @@ public int hashCode() {
}
}

/**
* Represents a Personalization value.
*/
/** Represents a Personalization value. */
public static final class PersonalizationValue extends ParameterValue {
private final String personalizationId;

Expand All @@ -283,9 +284,9 @@ public String getPersonalizationId() {

@Override
ParameterValueResponse toParameterValueResponse() {
return new ParameterValueResponse().setPersonalizationValue(
new PersonalizationValueResponse()
.setPersonalizationId(this.personalizationId));
return new ParameterValueResponse()
.setPersonalizationValue(
new PersonalizationValueResponse().setPersonalizationId(this.personalizationId));
}

@Override
Expand All @@ -306,9 +307,7 @@ public int hashCode() {
}
}

/**
* Represents a specific variant within an Experiment.
*/
/** Represents a specific variant within an Experiment. */
public static final class ExperimentVariantValue {
private final String variantId;
private final String value;
Expand Down Expand Up @@ -384,8 +383,8 @@ public boolean equals(Object o) {
}
ExperimentVariantValue that = (ExperimentVariantValue) o;
return noChange == that.noChange
&& Objects.equals(variantId, that.variantId)
&& Objects.equals(value, that.value);
&& Objects.equals(variantId, that.variantId)
&& Objects.equals(value, that.value);
}

@Override
Expand All @@ -394,16 +393,17 @@ public int hashCode() {
}
}

/**
* Represents an Experiment value.
*/
/** Represents an Experiment value. */
public static final class ExperimentValue extends ParameterValue {
private final String experimentId;
private final List<ExperimentVariantValue> variantValues;
private final double exposurePercent;

private ExperimentValue(String experimentId, List<ExperimentVariantValue> variantValues) {
private ExperimentValue(
String experimentId, List<ExperimentVariantValue> variantValues, double exposurePercent) {
this.experimentId = experimentId;
this.variantValues = variantValues;
this.exposurePercent = exposurePercent;
}

/**
Expand All @@ -415,6 +415,15 @@ public String getExperimentId() {
return experimentId;
}

/**
* Gets the exposure percentage of the experiment linked to this value.
*
* @return Exposure percentage of the experiment linked to this value.
*/
public double getExposurePercent() {
return exposurePercent;
}

/**
* Gets a collection of variant values served by the experiment.
*
Expand All @@ -426,16 +435,20 @@ public List<ExperimentVariantValue> getExperimentVariantValues() {

@Override
ParameterValueResponse toParameterValueResponse() {
List<ExperimentVariantValueResponse> variantValueResponses = variantValues.stream()
.map(variantValue -> new ExperimentVariantValueResponse()
.setVariantId(variantValue.getVariantId())
.setValue(variantValue.getValue())
.setNoChange(variantValue.getNoChange()))
.collect(toList());
return new ParameterValueResponse().setExperimentValue(
List<ExperimentVariantValueResponse> variantValueResponses =
variantValues.stream()
.map(
variantValue ->
new ExperimentVariantValueResponse()
.setVariantId(variantValue.getVariantId())
.setValue(variantValue.getValue())
.setNoChange(variantValue.getNoChange()))
.collect(toList());
return new ParameterValueResponse()
.setExperimentValue(
new ExperimentValueResponse()
.setExperimentId(this.experimentId)
.setExperimentVariantValues(variantValueResponses));
.setExperimentId(this.experimentId)
.setExperimentVariantValues(variantValueResponses));
Comment thread
rathovarun1032 marked this conversation as resolved.
Outdated
}

@Override
Expand All @@ -448,12 +461,13 @@ public boolean equals(Object o) {
}
ExperimentValue that = (ExperimentValue) o;
return Objects.equals(experimentId, that.experimentId)
&& Objects.equals(variantValues, that.variantValues);
&& Objects.equals(variantValues, that.variantValues)
&& Double.compare(that.exposurePercent, exposurePercent) == 0;
}

@Override
public int hashCode() {
return Objects.hash(experimentId, variantValues);
return Objects.hash(experimentId, variantValues, exposurePercent);
}
}
}
13 changes: 13 additions & 0 deletions src/main/java/com/google/firebase/remoteconfig/internal/TemplateResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -288,6 +288,9 @@ public static final class ExperimentValueResponse {
@Key("variantValue")
private List<ExperimentVariantValueResponse> experimentVariantValues;

@Key("exposurePercent")
private Double exposurePercent;

public String getExperimentId() {
return experimentId;
}
Expand All @@ -296,6 +299,11 @@ public List<ExperimentVariantValueResponse> getExperimentVariantValues() {
return experimentVariantValues;
}

public Double getExposurePercent() {
return exposurePercent;
}


Comment thread
rathovarun1032 marked this conversation as resolved.
Outdated
public ExperimentValueResponse setExperimentId(String experimentId) {
this.experimentId = experimentId;
return this;
Expand All @@ -306,6 +314,11 @@ public ExperimentValueResponse setExperimentVariantValues(
this.experimentVariantValues = experimentVariantValues;
return this;
}

public ExperimentValueResponse setExposurePercent(Double exposurePercent) {
this.exposurePercent = exposurePercent;
return this;
}
}

/**
Expand Down
17 changes: 10 additions & 7 deletions src/test/java/com/google/firebase/remoteconfig/ParameterValueTest.java
View file
Open in desktop
Comment thread
rathovarun1032 marked this conversation as resolved.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotEquals;
import static org.junit.Assert.assertTrue;

import com.google.common.collect.ImmutableList;
import com.google.firebase.remoteconfig.ParameterValue.ExperimentVariantValue;
Expand Down Expand Up @@ -64,7 +63,11 @@ public void testCreateExperimentValue() {
ParameterValue.ofExperiment("experiment_1", ImmutableList.of(
ExperimentVariantValue.of("variant_1", "value_1"),
ExperimentVariantValue.ofNoChange("variant_2")
));
), 10.0);
Comment thread
rathovarun1032 marked this conversation as resolved.

assertEquals("experiment_1", parameterValue.getExperimentId());
assertEquals(2, parameterValue.getExperimentVariantValues().size());
assertEquals(10.0, parameterValue.getExposurePercent(), 0.0);

assertEquals("experiment_1", parameterValue.getExperimentId());
assertEquals(2, parameterValue.getExperimentVariantValues().size());
Expand All @@ -77,6 +80,7 @@ public void testCreateExperimentValue() {
assertEquals("variant_2", variant2.getVariantId());
assertEquals(null, variant2.getValue());
assertEquals(true, variant2.isNoChange());

}

@Test
Expand Down Expand Up @@ -116,20 +120,19 @@ public void testEquality() {
ParameterValue.ExperimentValue experimentValueOne =
ParameterValue.ofExperiment("experiment_1", ImmutableList.of(
ExperimentVariantValue.of("variant_1", "value_1")
));
), 10.0);
ParameterValue.ExperimentValue experimentValueTwo =
ParameterValue.ofExperiment("experiment_1", ImmutableList.of(
ExperimentVariantValue.of("variant_1", "value_1")
));
), 10.0);
ParameterValue.ExperimentValue experimentValueThree =
ParameterValue.ofExperiment("experiment_2", ImmutableList.of(
ExperimentVariantValue.of("variant_1", "value_1")
));
), 10.0);
ParameterValue.ExperimentValue experimentValueFour =
ParameterValue.ofExperiment("experiment_1", ImmutableList.of(
ExperimentVariantValue.of("variant_2", "value_2")
));

), 20.0);
assertEquals(experimentValueOne, experimentValueTwo);
assertNotEquals(experimentValueOne, experimentValueThree);
assertNotEquals(experimentValueOne, experimentValueFour);
Expand Down
Loading