Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Commit

Permalink
Browse files Browse the repository at this point in the history
fix: Remove default value handling (#1353)
This is done in favor of explicit `optional` field handling in the generator.
  • Loading branch information
vam-google committed Apr 30, 2021
1 parent 06a53ac commit ed0fc79
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 19 deletions.
Expand Up @@ -101,9 +101,6 @@ RequestT fromJson(InputStream message, Charset messageCharset, Message.Builder b
* @param fieldValue a field value
*/
public void putPathParam(Map<String, String> fields, String fieldName, Object fieldValue) {
if (isDefaultValue(fieldName, fieldValue)) {
return;
}
fields.put(fieldName, String.valueOf(fieldValue));
}

Expand All @@ -116,11 +113,6 @@ public void putPathParam(Map<String, String> fields, String fieldName, Object fi
* @param fieldValue a field value
*/
public void putQueryParam(Map<String, List<String>> fields, String fieldName, Object fieldValue) {
// Avoids empty query parameter
if (isDefaultValue(fieldName, fieldValue)) {
return;
}

ImmutableList.Builder<String> paramValueList = ImmutableList.builder();
if (fieldValue instanceof List<?>) {
for (Object fieldValueItem : (List<?>) fieldValue) {
Expand All @@ -142,15 +134,4 @@ public void putQueryParam(Map<String, List<String>> fields, String fieldName, Ob
public String toBody(String fieldName, RequestT fieldValue) {
return toJson(fieldValue);
}

private boolean isDefaultValue(String fieldName, Object fieldValue) {
// TODO: Revisit this approach to ensure proper default-value handling as per design.
if (fieldValue instanceof Number) {
return ((Number) fieldValue).longValue() == 0L;
} else if (fieldValue instanceof String) {
return ((String) fieldValue).isEmpty();
}

return false;
}
}
Expand Up @@ -98,6 +98,7 @@ public void getQueryParamNames() {
Map<String, List<String>> queryParamNames = formatter.getQueryParamNames(field);
Map<String, List<String>> expected = new HashMap<>();
expected.put("number", Arrays.asList("2"));
expected.put("typeUrl", Arrays.asList(""));
Truth.assertThat(queryParamNames).isEqualTo(expected);
}

Expand Down
Expand Up @@ -115,7 +115,9 @@ public void putPathParam() {

Map<String, String> expectedFields = new HashMap<>();
expectedFields.put("optName1", "1");
expectedFields.put("optName2", "0");
expectedFields.put("optName3", "three");
expectedFields.put("optName4", "");

Truth.assertThat(fields).isEqualTo(expectedFields);
}
Expand All @@ -131,7 +133,9 @@ public void putQueryParam() {

Map<String, List<String>> expectedFields = new HashMap<>();
expectedFields.put("optName1", Arrays.asList("1"));
expectedFields.put("optName2", Arrays.asList("0"));
expectedFields.put("optName3", Arrays.asList("three"));
expectedFields.put("optName4", Arrays.asList(""));
expectedFields.put("optName5", Arrays.asList("four", "five"));

Truth.assertThat(fields).isEqualTo(expectedFields);
Expand Down

0 comments on commit ed0fc79

Please sign in to comment.