dep: update phpunit

PHPUnit 5 is not supported anymore and does not work correctly with PHP 7.2.
PHPUnit 6/7 don't support PHP 5.x anymore, so tests won't work correctly anymore on older PHP versions.

If testable code gets added/changed in the future,
raise the minimum required PHP version to 7.0!

Author: mle86

composer.json

@@ -17,7 +17,7 @@
 	},
 
 	"require-dev": {
-		"phpunit/phpunit": "4.*",
+		"phpunit/phpunit": "^6",
 		"php-coveralls/php-coveralls": "^2.0"
 	},
 

test/01-IncompleteClassTest.php

@@ -2,6 +2,8 @@
 namespace mle86\Value\Tests;
 
 use mle86\Value\AbstractValue;
+use mle86\Value\NotImplementedException;
+use PHPUnit\Framework\TestCase;
 
 
 /**
@@ -11,13 +13,11 @@ class BadTestWrapper  extends AbstractValue { }
 
 
 class IncompleteClassTest
-	extends \PHPUnit_Framework_TestCase
+	extends TestCase
 {
 
-	/**
-	 * @expectedException \mle86\Value\NotImplementedException
-	 */
 	public function testConstructor () {
+	    $this->expectException(NotImplementedException::class);
 		new BadTestWrapper ("1");
 	}
 

test/10-ValueTest.php

@@ -3,6 +3,9 @@
 
 use mle86\Value\AbstractValue;
 use mle86\Value\Value;
+use PHPUnit\Framework\Error\Error;
+use PHPUnit\Framework\TestCase;
+
 require_once 'helpers/TestWrapper4.php';
 require_once 'helpers/TestWrapper9.php';
 require_once 'helpers/ExtTestWrapper4.php';
@@ -14,7 +17,7 @@
  * and their default methods inherited from AbstractValue.
  */
 class ValueTest
-	extends \PHPUnit_Framework_TestCase
+	extends TestCase
 {
 
 	/** @var TestWrapper4 */
@@ -84,9 +87,9 @@ public function testConstructor9 ($initializer) {
 	 *
 	 * @depends testConstructor
 	 * @dataProvider invalidInputs
-	 * @expectedException \InvalidArgumentException
 	 */
 	public function testInvalidInitializer ($initializer) {
+	    $this->expectException(\InvalidArgumentException::class);
 		new TestWrapper4 ($initializer);
 	}
 
@@ -97,9 +100,9 @@ public function testInvalidInitializer ($initializer) {
 	 * @depends testConstructor
 	 * @depends testConstructor9
 	 * @dataProvider validInputs9
-	 * @expectedException \InvalidArgumentException
 	 */
 	public function testCrossInvalidInitializer ($initializer) {
+		$this->expectException(\InvalidArgumentException::class);
 		new TestWrapper4 ($initializer);
 	}
 
@@ -150,9 +153,9 @@ public function testEquals () {
 
 	/**
 	 * @depends testEquals
-	 * @expectedException \PHPUnit_Framework_Error
 	 */
 	public function testBuiltinEqualsZero () {
+		$this->expectException(Error::class);
 		$this->assertFalse((self::$tw1 == 0 || 0 == self::$tw1),
 			"wrapper is considered equal to zero by builtin== !");
 	}
@@ -217,20 +220,20 @@ public function testWrap () {
 	/**
 	 * @depends testWrap
 	 * @dataProvider invalidInputs
-	 * @expectedException \InvalidArgumentException
 	 */
 	public function testWrapInvalid ($initializer) {
 		$v = $initializer;
+		$this->expectException(\InvalidArgumentException::class);
 		TestWrapper4::Wrap($v);
 	}
 
 	/**
 	 * @depends testWrapInvalid
 	 * @dataProvider validInputs9
-	 * @expectedException \InvalidArgumentException
 	 */
 	public function testWrapCrossInvalid ($initializer) {
 		$v = $initializer;
+		$this->expectException(\InvalidArgumentException::class);
 		TestWrapper4::Wrap($v);
 	}
 
@@ -254,9 +257,9 @@ public function testRewrap (TestWrapper9 $tw) {
 	 * Wrapping an instance of a different class must fail.
 	 *
 	 * @depends testRewrap
-	 * @expectedException \InvalidArgumentException
 	 */
 	public function testRewrapInvalid (TestWrapper9 $tx) {
+		$this->expectException(\InvalidArgumentException::class);
 		TestWrapper4::Wrap($tx);
 	}
 
@@ -442,11 +445,11 @@ public function testExtendedBuiltinEquals ($additional_property, ExtTestWrapper4
 	 * @depends testExtendedObject
 	 * @depends testExtendedBuiltinEquals
 	 * @depends testBuiltinEqualsZero
-	 * @expectedException \PHPUnit_Framework_Error
 	 */
 	public function testExtendedBuiltinEqualsZero ($additional_property, ExtTestWrapper4 $ew) {
 		$ew->set_additional_property($additional_property);
 
+		$this->expectException(Error::class);
 		$this->assertFalse(($ew == 0 || 0 == $ew),
 			"Extended wrapper object is considered equal to zero by builtin== !");
 	}

test/15-SerializableValueTest.php

@@ -3,6 +3,9 @@
 
 use mle86\Value\AbstractSerializableValue;
 use mle86\Value\Value;
+use PHPUnit\Framework\Error\Error;
+use PHPUnit\Framework\TestCase;
+
 require_once 'helpers/TestSWrapper6.php';
 
 
@@ -11,7 +14,7 @@
  * and its default methods inherited from AbstractSerializableValue.
  */
 class SerializableValueTest
-	extends \PHPUnit_Framework_TestCase
+	extends TestCase
 {
 
 	const VALID_INPUT   = "61234";
@@ -80,9 +83,9 @@ public function testBuiltinEquals (AbstractSerializableValue $tw) {
 	/**
 	 * @depends testInstance
 	 * @depends testBuiltinEquals
-	 * @expectedException \PHPUnit_Framework_Error
 	 */
 	public function testBuiltinEqualsZero (AbstractSerializableValue $tw) {
+		$this->expectException(Error::class);
 		$this->assertFalse(($tw == 0),
 			"serializable wrapper considered zero as ==equal !");
 	}

test/90-DoubleConstructorCallTest.php

@@ -3,6 +3,8 @@
 
 use mle86\Value\AbstractValue;
 use mle86\Value\Value;
+use PHPUnit\Framework\TestCase;
+
 require_once 'helpers/TestWrapper4.php';
 
 
@@ -11,7 +13,7 @@
  * cannot be used to change the stored value.
  */
 class DoubleConstructorCallTest
-	extends \PHPUnit_Framework_TestCase
+	extends TestCase
 {
 
 	const VALID_VALUE1 = "41111";

test/90-MagicPropertiesTest.php

@@ -3,6 +3,8 @@
 
 use mle86\Value\AbstractValue;
 use mle86\Value\Value;
+use PHPUnit\Framework\TestCase;
+
 require_once 'helpers/TestWrapper4.php';
 
 
@@ -15,7 +17,7 @@
  * but we cannot really prevent that.
  */
 class MagicPropertiesTest
-	extends \PHPUnit_Framework_TestCase
+	extends TestCase
 {
 
 	const VALID_VALUE = "41111";