fix crash

This commit is contained in:
Glavo 2023-02-08 20:21:53 +08:00 committed by Haowei Wen
parent 7e522ff136
commit 1ac1ff3c95
2 changed files with 13 additions and 0 deletions

View File

@ -66,10 +66,12 @@ public final class VersionRange {
} }
public boolean contains(String versionNumber) { public boolean contains(String versionNumber) {
if (versionNumber == null) return false;
return contains(VersionNumber.asVersion(versionNumber)); return contains(VersionNumber.asVersion(versionNumber));
} }
public boolean contains(VersionNumber versionNumber) { public boolean contains(VersionNumber versionNumber) {
if (versionNumber == null) return false;
if (isEmpty()) return false; if (isEmpty()) return false;
if (isAll()) return true; if (isAll()) return true;

View File

@ -32,6 +32,17 @@ public class VersionRangeTest {
assertTrue(all().contains("0")); assertTrue(all().contains("0"));
assertTrue(all().contains("10")); assertTrue(all().contains("10"));
assertFalse(all().contains((String) null));
assertFalse(empty().contains((String) null));
assertFalse(between("0", "10").contains((String) null));
assertFalse(atLeast("10").contains((String) null));
assertFalse(atMost("10").contains((String) null));
assertFalse(all().contains((VersionNumber) null));
assertFalse(empty().contains((VersionNumber) null));
assertFalse(between("0", "10").contains((VersionNumber) null));
assertFalse(atLeast("10").contains((VersionNumber) null));
assertFalse(atMost("10").contains((VersionNumber) null));
} }
private static void assertIsOverlappedBy(boolean value, VersionRange range1, VersionRange range2) { private static void assertIsOverlappedBy(boolean value, VersionRange range1, VersionRange range2) {