Skip to content

Commit daba1fa

Browse files
authored
Add comments and asserts that didn't merge with PR 5059 (#5063)
* Add different defaults for when RC fetch failed * Add changelog entry * Format * Add throttled case to isLastFetchFailed check * Add rc fetch failed defaults for cpu and memory capturing frequency * Address comments
1 parent dd04244 commit daba1fa

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

firebase-perf/src/main/java/com/google/firebase/perf/config/ConfigurationConstants.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -232,6 +232,7 @@ protected Long getDefault() {
232232

233233
@Override
234234
protected Long getDefaultOnRcFetchFail() {
235+
// Increase the typical default by factor of 3.
235236
return getDefault() * 3;
236237
}
237238

@@ -313,6 +314,7 @@ protected Long getDefault() {
313314

314315
@Override
315316
protected Long getDefaultOnRcFetchFail() {
317+
// Increase the typical default by factor of 3.
316318
return getDefault() * 3;
317319
}
318320

firebase-perf/src/test/java/com/google/firebase/perf/config/ConfigurationConstantsTest.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,7 @@ public void getInstance_TraceSamplingRate_validateConstants() {
155155
TraceSamplingRate configFlag = TraceSamplingRate.getInstance();
156156

157157
assertThat(configFlag.getDefault()).isEqualTo(1.00);
158+
assertThat(configFlag.getDefaultOnRcFetchFail()).isEqualTo(0.01);
158159
assertThat(configFlag.getDeviceCacheFlag())
159160
.isEqualTo("com.google.firebase.perf.TraceSamplingRate");
160161
assertThat(configFlag.getRemoteConfigFlag()).isEqualTo("fpr_vc_trace_sampling_rate");
@@ -166,6 +167,7 @@ public void getInstance_NetworkRequestSamplingRate_validateConstants() {
166167
NetworkRequestSamplingRate configFlag = NetworkRequestSamplingRate.getInstance();
167168

168169
assertThat(configFlag.getDefault()).isEqualTo(1.00);
170+
assertThat(configFlag.getDefaultOnRcFetchFail()).isEqualTo(0.01);
169171
assertThat(configFlag.getDeviceCacheFlag())
170172
.isEqualTo("com.google.firebase.perf.NetworkRequestSamplingRate");
171173
assertThat(configFlag.getRemoteConfigFlag()).isEqualTo("fpr_vc_network_request_sampling_rate");
@@ -177,6 +179,7 @@ public void getInstance_SessionsSamplingRate_validateConstants() {
177179
SessionsSamplingRate configFlag = SessionsSamplingRate.getInstance();
178180

179181
assertThat(configFlag.getDefault()).isEqualTo(0.01);
182+
assertThat(configFlag.getDefaultOnRcFetchFail()).isEqualTo(0.0001);
180183
assertThat(configFlag.getDeviceCacheFlag())
181184
.isEqualTo("com.google.firebase.perf.SessionSamplingRate");
182185
assertThat(configFlag.getRemoteConfigFlag()).isEqualTo("fpr_vc_session_sampling_rate");

0 commit comments

Comments
 (0)