fankes/moshi
2
0
Fork 0
mirror of https://github.com/fankes/moshi.git synced 2025-10-19 07:59:21 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #507 from square/eric.null-assertion

Browse Source 
Disallow null Type from entering user code.
This commit is contained in:
Jesse Wilson
2018-04-16 22:32:59 -04:00
committed by GitHub
parent b860b6da4f 84745b0537
commit 48698a61ad
2 changed files with 14 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
moshi/src/main/java/com/squareup/moshi/Moshi.java
View File

@@ -92,6 +92,9 @@ public final class Moshi {
@CheckReturnValue
@SuppressWarnings("unchecked") // Factories are required to return only matching JsonAdapters.
public <T> JsonAdapter<T> adapter(Type type, Set<? extends Annotation> annotations) {
if (type == null) {
throw new NullPointerException("type == null");
}
if (annotations == null) {
throw new NullPointerException("annotations == null");
}

11
moshi/src/test/java/com/squareup/moshi/MoshiTest.java
View File

@@ -27,6 +27,7 @@ import java.util.ArrayDeque;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
import java.util.Collections;
import java.util.LinkedHashMap;
import java.util.LinkedHashSet;
import java.util.List;
@@ -725,6 +726,16 @@ public final class MoshiTest {
.isEqualTo("{\"shout\":\"WHAT'S UP\",\"speak\":\"Yo dog\"}");
}
@Test public void adapterLookupDisallowsNullType() {
Moshi moshi = new Moshi.Builder().build();
try {
moshi.adapter(null, Collections.<Annotation>emptySet());
fail();
} catch (NullPointerException expected) {
assertThat(expected).hasMessage("type == null");
}
}
@Test public void adapterLookupDisallowsNullAnnotations() {
Moshi moshi = new Moshi.Builder().build();
try {