diff --git a/Tests/ClientTest.php b/Tests/ClientTest.php index 63b5c9a..6e98998 100644 --- a/Tests/ClientTest.php +++ b/Tests/ClientTest.php @@ -255,4 +255,20 @@ class ClientTest extends ApiTestCase 'NOK status' => [false], ]; } + + /** + * @test + * @return void + * @throws ReflectionException + */ + public function testGetLoggerHandler() + { + $this->assertInstanceOf( + LoggerHandler::class, + $this->callMethod( + $this->api, + 'getLoggerHandler' + ) + ); + } } diff --git a/Tests/Request/AbstractRequest.php b/Tests/Request/AbstractRequest.php index a669164..47055f9 100644 --- a/Tests/Request/AbstractRequest.php +++ b/Tests/Request/AbstractRequest.php @@ -620,4 +620,31 @@ abstract class AbstractRequest extends ApiTestCase $this->checkGetterSetter($clientMock, 'setClient', 'getClient'); } + + /** + * @test + * @return void + * @throws ReflectionException + */ + public function testGetResponseClass() + { + $className = $this->callMethod( + $this->request, + 'getResponseClass' + ); + + $this->assertIsString( + $className + ); + + /** @var \D3\LinkmobilityClient\Response\ResponseInterface|MockObject $responseMock */ + $responseMock = $this->getMockBuilder($className) + ->disableOriginalConstructor() + ->getMock(); + + $this->assertInstanceOf( + \D3\LinkmobilityClient\Response\ResponseInterface::class, + $responseMock + ); + } } diff --git a/Tests/Response/AbstractResponse.php b/Tests/Response/AbstractResponse.php index b4c2028..0f8d21b 100644 --- a/Tests/Response/AbstractResponse.php +++ b/Tests/Response/AbstractResponse.php @@ -117,6 +117,15 @@ abstract class AbstractResponse extends ApiTestCase $this->checkProperties('statusMessage', 'statusMessage', 'getStatusMessage'); } + /** + * @test + * @throws ReflectionException + */ + public function testGetClientMessageId() + { + $this->checkProperties('clientMessageId', 'clientMessageId', 'getClientMessageId'); + } + /** * @test * @throws ReflectionException diff --git a/Tests/ValueObject/RecipientTest.php b/Tests/ValueObject/RecipientTest.php index ffe8ddc..9edfab5 100644 --- a/Tests/ValueObject/RecipientTest.php +++ b/Tests/ValueObject/RecipientTest.php @@ -143,9 +143,13 @@ class RecipientTest extends ApiTestCase */ public function constructInvalidDataProvider(): array { + $phoneUtil = PhoneNumberUtil::getInstance(); + $example = $phoneUtil->getExampleNumberForType($this->phoneCountryFixture, PhoneNumberType::MOBILE); + $phoneNumberFixture = $phoneUtil->format($example, PhoneNumberFormat::NATIONAL); + return [ 'empty number' => ['', 'DE', true, InvalidArgumentException::class], - 'invalid country code' => [$this->phoneNumberFixture, 'DEX', true, InvalidArgumentException::class], + 'invalid country code' => [$phoneNumberFixture, 'DEX', true, InvalidArgumentException::class], 'unparsable' => ['abc', 'DE', true, NumberParseException::class], 'invalid number' => ['abc', 'DE', false, NumberParseException::class] ]; diff --git a/Tests/ValueObject/SenderTest.php b/Tests/ValueObject/SenderTest.php index 4d702fb..6a81f1e 100644 --- a/Tests/ValueObject/SenderTest.php +++ b/Tests/ValueObject/SenderTest.php @@ -50,7 +50,6 @@ class SenderTest extends ApiTestCase $this->sender = new Sender($this->phoneNumberFixture, $this->phoneCountryFixture); - } /** @@ -140,9 +139,13 @@ class SenderTest extends ApiTestCase */ public function constructInvalidDataProvider(): array { + $phoneUtil = PhoneNumberUtil::getInstance(); + $example = $phoneUtil->getExampleNumberForType($this->phoneCountryFixture, PhoneNumberType::MOBILE); + $phoneNumberFixture = $phoneUtil->format($example, PhoneNumberFormat::NATIONAL); + return [ 'empty number' => ['', 'DE', true, InvalidArgumentException::class], - 'invalid country code' => [$this->phoneNumberFixture, 'DEX', true, InvalidArgumentException::class], + 'invalid country code' => [$phoneNumberFixture, 'DEX', true, InvalidArgumentException::class], 'unparsable' => ['abc', 'DE', true, NumberParseException::class], 'invalid number' => ['abc', 'DE', false, NumberParseException::class] ];