Merge dcec4a5 into 46dd458

CHANGELOG.md

@@ -30,12 +30,14 @@ _TBD_
 - [x] Added new property to `PaymentCardToken`: `isUsed`
 - [x] Added new blacklist types to `Blacklist`: `email-domain`, `bank-account`, `address`
 - [x] Added new `taxCategory` to `Product`
+- [x] Added new property to `Blacklist`: `expirationTime`.
 
 ### Changed
 - [x] **Upgraded minimum PHP version to 7.1**
 
 ### Deprecated
 - [x] Deprecated `Customer` entity methods: `getFirstName`, `setFirstName`, `getLastName`, `setLastName`, `getEmail`, `setEmail`
+- [x] Deprecated `Blacklist` `expiredTime` property.
 
 ## [2.3.0] 2018-10-08
 

src/Entities/Blacklist.php

@@ -103,44 +103,68 @@ public function setValue($value)
 
     /**
      * @deprecated The method is deprecated and will be removed in next version.
-     * @see Blacklist::getExpiredTime()
+     * @see Blacklist::getExpirationTime()
      *
      * @return mixed
      */
     public function getExpireTime()
     {
-        return $this->getAttribute('expireTime');
+        return $this->getExpirationTime();
     }
 
     /**
      * @deprecated The method is deprecated and will be removed in next version.
-     * @see Blacklist::setExpiredTime()
+     * @see Blacklist::setExpirationTime()
      *
      * @param mixed $value
      *
      * @return $this
      */
     public function setExpireTime($value)
     {
-        return $this->setAttribute('expireTime', $value);
+        return $this->setExpirationTime($value);
     }
 
     /**
+     * @deprecated The method is deprecated and will be removed in next version.
+     * @see Blacklist::getExpirationTime()
+     *
      * @return mixed
      */
     public function getExpiredTime()
     {
-        return $this->getAttribute('expiredTime');
+        return $this->getExpirationTime();
     }
 
     /**
+     * @deprecated The method is deprecated and will be removed in next version.
+     * @see Blacklist::setExpirationTime()
+     *
      * @param mixed $value
      *
      * @return $this
      */
     public function setExpiredTime($value)
     {
-        return $this->setAttribute('expiredTime', $value);
+        return $this->setExpirationTime($value);
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getExpirationTime()
+    {
+        return $this->getAttribute('expirationTime');
+    }
+
+    /**
+     * @param mixed $value
+     *
+     * @return $this
+     */
+    public function setExpirationTime($value)
+    {
+        return $this->setAttribute('expirationTime', $value);
     }
 
     /**

tests/Api/ApiTest.php

@@ -29,8 +29,9 @@ class ApiTest extends TestCase
      *
      * @param string $class
      * @param string $id
+     * @param array $exclude
      */
-    public function buildJson($class, $id = 'id')
+    public function buildJson($class, $id = 'id', $exclude = [])
     {
         $getters = [];
         $setters = [];
@@ -51,6 +52,10 @@ public function buildJson($class, $id = 'id')
             $prefix = substr($method, 0, 3);
             $attribute = lcfirst(substr($method, 3));
 
+            if (in_array($attribute, $exclude, true)) {
+                continue;
+            }
+
             if ($prefix === 'get') {
                 $getters[$attribute] = $method;
             } elseif ($prefix === 'set') {
@@ -131,7 +136,7 @@ public function provideEntityClasses()
             [Entities\Attachment::class],
             [Entities\AuthenticationOptions::class, null],
             [Entities\AuthenticationToken::class, 'token'],
-            [Entities\Blacklist::class],
+            [Entities\Blacklist::class, 'id', ['expireTime', 'expiredTime']],
             [Entities\Contact::class],
             [Entities\Contact\Email::class],
             [Entities\Contact\PhoneNumber::class],

tests/TestCase.php

@@ -105,7 +105,7 @@ protected function getFakeValue($attribute, $class)
                 return $faker->uuid;
             case 'dueTime':
             case 'expiredTime':
-            case 'expireTime': // TODO inconsistent name
+            case 'expirationTime':
             case 'periodStartTime':
             case 'renewalTime':
             case 'periodEndTime':