Performance bug (#3123)

* floating point number bug fix

* formatting
This commit is contained in:
Russell Wheatley
2020-01-27 10:11:56 +00:00
committed by GitHub
parent e8e7becc06
commit 1e567214e9
2 changed files with 16 additions and 3 deletions

View File

@@ -117,15 +117,15 @@ public class UniversalFirebasePerfModule extends UniversalFirebaseModule {
HttpMetric httpMetric = httpMetrics.get(id);
if (httpMetricConfig.containsKey("httpResponseCode")) {
httpMetric.setHttpResponseCode(httpMetricConfig.getInt("httpResponseCode"));
httpMetric.setHttpResponseCode((int) httpMetricConfig.getDouble("httpResponseCode"));
}
if (httpMetricConfig.containsKey("requestPayloadSize")) {
httpMetric.setRequestPayloadSize(httpMetricConfig.getInt("requestPayloadSize"));
httpMetric.setRequestPayloadSize((int) httpMetricConfig.getDouble("requestPayloadSize"));
}
if (httpMetricConfig.containsKey("responsePayloadSize")) {
httpMetric.setResponsePayloadSize(httpMetricConfig.getInt("responsePayloadSize"));
httpMetric.setResponsePayloadSize((int) httpMetricConfig.getDouble("responsePayloadSize"));
}
if (httpMetricConfig.containsKey("responseContentType")) {

View File

@@ -62,6 +62,19 @@ android.describe('perf()', () => {
await httpMetric.stop();
should.equal(await httpMetric.stop(), null);
});
it('handles floating point numbers', async () => {
const httpMetric = firebase.perf().newHttpMetric(aCoolUrl, 'POST');
await httpMetric.start();
const floatingPoint = 500.447553;
httpMetric.setHttpResponseCode(floatingPoint);
httpMetric.setResponsePayloadSize(floatingPoint);
httpMetric.setRequestPayloadSize(floatingPoint);
await Utils.sleep(100);
await httpMetric.stop();
});
});
// describe('removeAttribute()', async () => {