Tests fixes.

w7corp · Jan 18, 2016 · 666113d · 666113d
1 parent 413c25f
commit 666113d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/Payment/LuckyMoney/API.php b/src/Payment/LuckyMoney/API.php
@@ -129,7 +129,7 @@ public function send(array $params, $type = self::TYPE_NORMAL)
     public function sendNormal($params)
     {
         $params['total_num'] = 1;
-        $params['client_ip'] = $params['client_ip'] ?: $_SERVER['HTTP_CLIENT_IP'];
+        $params['client_ip'] = !empty($params['client_ip']) ? $params['client_ip'] : $_SERVER['HTTP_CLIENT_IP'];
 
         return $this->send($params, self::TYPE_NORMAL);
     }
@@ -144,7 +144,7 @@ public function sendNormal($params)
     public function sendGroup($params)
     {
         $params['amt_type'] = 'ALL_RAND';
-        $params['client_ip'] = $params['client_ip'] ?: $_SERVER['HTTP_CLIENT_IP'];
+        $params['client_ip'] = !empty($params['client_ip']) ? $params['client_ip'] : $_SERVER['HTTP_CLIENT_IP'];
 
         return $this->send($params, self::TYPE_GROUP);
     }

diff --git a/tests/Payment/PaymentLuckyMoneyAPITest.php b/tests/Payment/PaymentLuckyMoneyAPITest.php
@@ -16,6 +16,11 @@
 
 class PaymentLuckyMoneyAPITest extends PHPUnit_Framework_TestCase
 {
+    public static function setUpBeforeClass()
+    {
+        $_SERVER['HTTP_CLIENT_IP'] = '127.0.0.1';
+    }
+
     /**
      * Build API instance.
      *