Add message for DoesNotExistException
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
This commit is contained in:
parent
72cc8270cd
commit
5ce9e73bad
2 changed files with 19 additions and 2 deletions
|
@ -56,8 +56,8 @@ class ChangesCheck {
|
|||
$version = $this->normalizeVersion($version);
|
||||
$changesInfo = $this->mapper->getChanges($version);
|
||||
$changesData = json_decode($changesInfo->getData(), true);
|
||||
if(empty($changesData)) {
|
||||
throw new DoesNotExistException();
|
||||
if (empty($changesData)) {
|
||||
throw new DoesNotExistException('Unable to decode changes info');
|
||||
}
|
||||
return $changesData;
|
||||
}
|
||||
|
|
|
@ -380,4 +380,21 @@ class ChangesCheckTest extends TestCase {
|
|||
$this->assertTrue(isset($data['whatsNew']['en']['regular']));
|
||||
$this->assertTrue(isset($data['changelogURL']));
|
||||
}
|
||||
|
||||
public function testGetChangesForVersionEmptyData() {
|
||||
$entry = $this->createMock(ChangesResult::class);
|
||||
$entry->expects($this->once())
|
||||
->method('__call')
|
||||
->with('getData')
|
||||
->willReturn('');
|
||||
|
||||
$this->mapper->expects($this->once())
|
||||
->method('getChanges')
|
||||
->with('13.0.7')
|
||||
->willReturn($entry);
|
||||
|
||||
$this->expectException(DoesNotExistException::class);
|
||||
/** @noinspection PhpUnhandledExceptionInspection */
|
||||
$this->checker->getChangesForVersion('13.0.7');
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue