review the code. fix some spelling issue

Author: xiezefan

src/cn/jpush/api/JPushClient.java

@@ -68,7 +68,7 @@ public PushResult sendPush(PushPayload pushPayload) throws APIConnectionExceptio
 	 * 
 	 * Attention: globally settings cannot be affect this type of Push.
      * 
-     * @param pushPayload payload of a push. 
+     * @param  payloadString payload of a push.
      * @return PushResult. Can be printed to a JSON.
 	 * @throws APIConnectionException
 	 * @throws APIRequestException

src/cn/jpush/api/common/IHttpClient.java

@@ -22,8 +22,8 @@ public enum RequestMethod {
         DELETE
     }
 
-    public static final String IO_ERROR_MESSAGE = "Conneciton IO error. \n"
-            + "Chould not connect to JPush Server. "
+    public static final String IO_ERROR_MESSAGE = "Connection IO error. \n"
+            + "Can not connect to JPush Server. "
             + "Please ensure your internet connection is ok. \n"
             + "If the problem persists, please let us know at support@jpush.cn.";
 

src/cn/jpush/api/push/PushClient.java

@@ -19,7 +19,7 @@
  * Can be used directly.
  */
 public class PushClient {
-    public static String HOST_NAME_SSL = "https://api.jpush.cn";
+    public static final String HOST_NAME_SSL = "https://api.jpush.cn";
     public static final String PUSH_PATH = "/v3/push";
 
     private final NativeHttpClient _httpClient;
@@ -58,17 +58,17 @@ public PushClient(String masterSecret, String appKey) {
 	 * 
 	 * @param masterSecret  API access secret of the appKey.
 	 * @param appKey The KEY of one application on JPush.
-	 * @param masxRetryTimes
+	 * @param maxRetryTimes max retry times
 	 */
-	public PushClient(String masterSecret, String appKey, int masxRetryTimes) {
+	public PushClient(String masterSecret, String appKey, int maxRetryTimes) {
         this._masterSecret = masterSecret;
         this._appKey = appKey;
 
         ServiceHelper.checkBasic(appKey, masterSecret);
 
         this._authCode = ServiceHelper.getAuthorizationBase64(_appKey, _masterSecret);
         this._baseUrl = HOST_NAME_SSL + PUSH_PATH;
-        this._httpClient = new NativeHttpClient(masxRetryTimes);
+        this._httpClient = new NativeHttpClient(maxRetryTimes);
 	}
 
 	/**

src/cn/jpush/api/push/model/Message.java

@@ -132,7 +132,7 @@ public Builder addExtra(String key, Boolean value) {
 
         public Message build() {
             Preconditions.checkArgument(! (null == msgContent), 
-                    "msgConent should be set");
+                    "msgContent should be set");
             return new Message(title, msgContent, contentType, 
                     (null == extrasBuilder) ? null : extrasBuilder.build(), 
                     (null == numberExtrasBuilder) ? null : numberExtrasBuilder.build(),

src/cn/jpush/api/utils/StringUtils.java

@@ -48,11 +48,11 @@ public static String encodeParam(String param) {
 		return encodeParam;
 	}
 
-    public static String arrayToString(String[] vlaues) {
-        StringBuffer buffer = new StringBuffer(vlaues.length);
-        if (vlaues != null) {
-            for (int i = 0; i < vlaues.length; i++) {
-                buffer.append(vlaues[i]).append(",");
+    public static String arrayToString(String[] values) {
+        StringBuffer buffer = new StringBuffer(values.length);
+        if (values != null) {
+            for (int i = 0; i < values.length; i++) {
+                buffer.append(values[i]).append(",");
             }
         }
         if (buffer.length() > 0) {

test/cn/jpush/api/report/ReportFunctionTests.java

@@ -2,6 +2,7 @@
 
 import static org.junit.Assert.*;
 
+import cn.jpush.api.common.TimeUnit;
 import org.junit.Before;
 import org.junit.Test;
 
@@ -33,5 +34,42 @@ public void getReceivedsFixed2() throws Exception {
         assertTrue(result.received_list.size() > 0);
     }
 
+    @Test
+    public void getMessagesTest() throws Exception {
+        MessagesResult result = jpushClient.getReportMessages("1613113584");
+        assertTrue(result.isResultOK());
+        assertTrue(result.messages.size() > 0);
+    }
+
+    @Test
+    public void getMessagesTest2() throws Exception {
+        MessagesResult result = jpushClient.getReportMessages("1613113584,   ,1229760629,  ");
+        assertTrue(result.isResultOK());
+        assertTrue(result.messages.size() > 0);
+    }
+
+    @Test
+    public void getUsersTest() throws Exception {
+        UsersResult result = jpushClient.getReportUsers(TimeUnit.MONTH, "2014-05", 1);
+        assertTrue(result.isResultOK());
+        assertTrue(result.items.size() > 0);
+    }
+
+    @Test
+    public void getUserTest2() throws  Exception {
+        UsersResult result = jpushClient.getReportUsers(TimeUnit.DAY, "2014-05-10", 5);
+        assertTrue(result.isResultOK());
+        assertTrue(result.items.size() > 0);
+    }
+
+    @Test
+    public void getUserTest3() throws Exception {
+        UsersResult result = jpushClient.getReportUsers(TimeUnit.HOUR, "2014-05-10 06", 10);
+        assertTrue(result.isResultOK());
+        assertTrue(result.items.size() > 0);
+    }
+
+
+
 }