Fix prophecy to use trait, update maintainer details (#300)

* Big rewrite to extend off a new base class so the prophecy trait can be used across the suite

* Update change of ownership within Vonage

* remove support for PHP7.2 + PHP7.3

* remove support for PHP7.2 + PHP7.3. Build support for 8.1

* split support for older php versions as a temporary measure until we make production catch up

* Update changelog

Author: SecondeJK

.github/workflows/build.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        php: [ '7.2', '7.3', '7.4', '8.0' ]
+        php: [ '7.4', '8.0', '8.1' ]
     name: PHP ${{ matrix.php }} Test
 
     steps:

CHANGELOG.md

@@ -1,3 +1,14 @@
+# 2.10.0
+
+### Fixed
+
+* PHPUnit tests now no longer throw `prophesize()` depreciation notices
+
+### Changed
+
+* Maintainer and Contribution documents changed to reflect current ownership
+* All test cases now extend off a the new `VonageTestCase` class that implements the `ProphesizeTrait`
+
 # 2.9.3
 
 ### Fixed

CONTRIBUTORS.md

@@ -1,13 +1,14 @@
 ## Maintainers
 
 If you have any problems with our library, feel free to reach out to our
-maintainers.
+maintainer.
 
-* [Chris Tankersley](https://github.com/dragonmantank)
-* [Lorna Jane Mitchell](https://github.com/lornajane)
+* [James Seconde](https://github.com/secondejk)
 
 ### Past Maintainers
 
+* [Chris Tankersley](https://github.com/dragonmantank)
+* [Lorna Jane Mitchell](https://github.com/lornajane)
 * [Tim Lytle](https://github.com/tjlytle)
 * [Micheal Heap](https://github.com/mheap)
 

composer.json

@@ -6,14 +6,9 @@
   "license": "Apache-2.0",
   "authors": [
     {
-      "name": "Chris Tankersley",
-      "email": "[email protected]",
-      "role": "Developer"
-    },
-    {
-      "name": "Lorna Mitchell",
-      "email": "[email protected]",
-      "role": "Developer"
+      "name": "James Seconde",
+      "email": "[email protected]",
+      "role": "PHP Developer Advocate"
     }
   ],
   "require": {
@@ -29,14 +24,16 @@
     "psr/log": "^1.1"
   },
   "require-dev": {
+    "php": ">=7.4",
     "guzzlehttp/guzzle": ">=6",
     "helmich/phpunit-json-assert": "^3.3",
     "php-http/mock-client": "^1.4",
     "phpstan/phpstan": "^0.12",
     "phpunit/phpunit": "^8.5|^9.4",
     "roave/security-advisories": "dev-latest",
     "squizlabs/php_codesniffer": "^3.5",
-    "softcreatr/jsonpath": "^0.6.4"
+    "softcreatr/jsonpath": "^0.6.4",
+    "phpspec/prophecy-phpunit": "^2.0"
   },
   "config": {
     "optimize-autoloader": true,
@@ -62,6 +59,6 @@
     "email": "[email protected]",
     "issues": "https://github.com/Vonage/vonage-php-sdk-core/issues",
     "source": "https://github.com/Vonage/vonage-php-sdk-core",
-    "docs": "https://developer.nexmo.com"
+    "docs": "https://developer.vonage.com"
   }
 }

test/Account/BalanceTest.php

@@ -11,11 +11,11 @@
 
 namespace VonageTest\Account;
 
-use PHPUnit\Framework\TestCase;
+use VonageTest\VonageTestCase;
 use Vonage\Account\Balance;
 use Vonage\Client\Exception\Exception as ClientException;
 
-class BalanceTest extends TestCase
+class BalanceTest extends VonageTestCase
 {
     /**
      * @var Balance

test/Account/ClientFactoryTest.php

@@ -11,13 +11,13 @@
 
 namespace VonageTest\Account;
 
-use PHPUnit\Framework\TestCase;
+use VonageTest\VonageTestCase;
 use Vonage\Account\ClientFactory;
 use Vonage\Client;
 use Vonage\Client\APIResource;
 use Vonage\Client\Factory\MapFactory;
 
-class ClientFactoryTest extends TestCase
+class ClientFactoryTest extends VonageTestCase
 {
     /**
      * @var MapFactory

test/Account/ClientTest.php

@@ -12,8 +12,9 @@
 namespace VonageTest\Account;
 
 use Laminas\Diactoros\Response;
-use PHPUnit\Framework\TestCase;
+use VonageTest\VonageTestCase;
 use Prophecy\Argument;
+use Prophecy\PhpUnit\ProphecyTrait;
 use Psr\Http\Client\ClientExceptionInterface;
 use Psr\Http\Message\RequestInterface;
 use Vonage\Account\Client as AccountClient;
@@ -29,10 +30,11 @@
 
 use function fopen;
 
-class ClientTest extends TestCase
+class ClientTest extends VonageTestCase
 {
     use Psr7AssertionTrait;
 
+
     protected $vonageClient;
 
     /**

test/Account/ConfigTest.php

@@ -11,10 +11,10 @@
 
 namespace VonageTest\Account;
 
-use PHPUnit\Framework\TestCase;
+use VonageTest\VonageTestCase;
 use Vonage\Account\Config;
 
-class ConfigTest extends TestCase
+class ConfigTest extends VonageTestCase
 {
     /**
      * @var Config

test/Account/PrefixPriceTest.php

@@ -12,10 +12,10 @@
 namespace VonageTest\Account;
 
 use Exception;
-use PHPUnit\Framework\TestCase;
+use VonageTest\VonageTestCase;
 use Vonage\Account\PrefixPrice;
 
-class PrefixPriceTest extends TestCase
+class PrefixPriceTest extends VonageTestCase
 {
     /**
      * @dataProvider prefixPriceProvider

test/Account/SecretCollectionTest.php

@@ -11,14 +11,14 @@
 
 namespace VonageTest\Account;
 
-use PHPUnit\Framework\TestCase;
+use VonageTest\VonageTestCase;
 use Vonage\Account\Secret;
 use Vonage\Account\SecretCollection;
 use Vonage\InvalidResponseException;
 
 use function array_map;
 
-class SecretCollectionTest extends TestCase
+class SecretCollectionTest extends VonageTestCase
 {
     /**
      * @var array[]