Skip to content

Commit 2d4d90c

Browse files
committed
Merge pull request #63 from jpush/dev
merge version 3.2.9
2 parents 2d9481b + b32306a commit 2d4d90c

File tree

2 files changed

+15
-2
lines changed

2 files changed

+15
-2
lines changed

src/main/java/cn/jpush/api/device/DeviceClient.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,6 @@ public DefaultResult updateDeviceTagAlias(String registrationId, String alias,
144144

145145
public DefaultResult bindMobile(String registrationId, String mobile)
146146
throws APIConnectionException, APIRequestException {
147-
Preconditions.checkNotNull(mobile, "The mobile must not be null.");
148-
149147
if ( StringUtils.isEmpty(mobile) ) {
150148
// delete bind while mobile is empty.
151149
mobile = "";

src/test/java/cn/jpush/api/device/DeviceNormalRemoteTest.java

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,4 +181,19 @@ public void testBindMobile() throws APIConnectionException, APIRequestException
181181
DefaultResult result = jpushClient.bindMobile(REGISTRATION_ID1, "13000000000");
182182
assertTrue(result.isResultOK());
183183
}
184+
185+
@Test
186+
@TestOrder(order = 361)
187+
public void testBindMobile_null() throws APIConnectionException, APIRequestException {
188+
DefaultResult result = jpushClient.bindMobile(REGISTRATION_ID1, null);
189+
assertTrue(result.isResultOK());
190+
}
191+
192+
@Test
193+
@TestOrder(order = 362)
194+
public void testBindMobile_empty() throws APIConnectionException, APIRequestException {
195+
DefaultResult result = jpushClient.bindMobile(REGISTRATION_ID1, "");
196+
assertTrue(result.isResultOK());
197+
}
198+
184199
}

0 commit comments

Comments
 (0)