impl. tests and fix validation status code

This commit is contained in:
Dennis Eichhorn 2020-10-06 21:51:40 +02:00
parent 9d28cb182c
commit 28c751003d
2 changed files with 18 additions and 4 deletions

View File

@ -118,6 +118,7 @@ class ApiControllerTest extends \PHPUnit\Framework\TestCase
$this->module->apiLogUpdate(1, ['id' => 2, 'test' => true], ['id' => 1, 'test' => true], 1, 2, 'Auditor', 'abc', 'def'); $this->module->apiLogUpdate(1, ['id' => 2, 'test' => true], ['id' => 1, 'test' => true], 1, 2, 'Auditor', 'abc', 'def');
$logs = AuditMapper::getAll(); $logs = AuditMapper::getAll();
$found = false;
foreach($logs as $log) { foreach($logs as $log) {
if ($log->getId() > 0 if ($log->getId() > 0
&& $log->getType() === 1 && $log->getType() === 1
@ -128,12 +129,22 @@ class ApiControllerTest extends \PHPUnit\Framework\TestCase
&& $log->getOld() === \json_encode(['id' => 2, 'test' => true], \JSON_PRETTY_PRINT) && $log->getOld() === \json_encode(['id' => 2, 'test' => true], \JSON_PRETTY_PRINT)
&& $log->getNew() === \json_encode(['id' => 1, 'test' => true], \JSON_PRETTY_PRINT) && $log->getNew() === \json_encode(['id' => 1, 'test' => true], \JSON_PRETTY_PRINT)
) { ) {
self::assertTrue(true); $found = true;
return; break;
} }
} }
self::assertTrue(false); self::assertTrue($found);
}
public function testLogUpdateWithoutChange() : void
{
$logs = AuditMapper::getAll();
$this->module->apiLogUpdate(1, ['id' => 2, 'test' => true], ['id' => 2, 'test' => true], 1, 2, 'Auditor', 'abc', 'def');
$logs2 = AuditMapper::getAll();
self::assertGreaterThan(0, \count($logs));
self::assertEquals(\count($logs), \count($logs2));
} }
/** /**

View File

@ -40,6 +40,7 @@ class AuditTest extends \PHPUnit\Framework\TestCase
self::assertNull($audit->getContent()); self::assertNull($audit->getContent());
self::assertNull($audit->getOld()); self::assertNull($audit->getOld());
self::assertNull($audit->getNew()); self::assertNull($audit->getNew());
self::assertEquals(0, $audit->getIp());
self::assertEquals(0, $audit->getCreatedBy()->getId()); self::assertEquals(0, $audit->getCreatedBy()->getId());
self::assertInstanceOf('\DateTimeImmutable', $audit->getCreatedAt()); self::assertInstanceOf('\DateTimeImmutable', $audit->getCreatedAt());
} }
@ -57,7 +58,8 @@ class AuditTest extends \PHPUnit\Framework\TestCase
1, 2, 1, 2,
'3', '3',
'test', 'test',
'content' 'content',
\ip2long('127.0.0.1')
); );
self::assertEquals(1, $audit->getType()); self::assertEquals(1, $audit->getType());
@ -67,6 +69,7 @@ class AuditTest extends \PHPUnit\Framework\TestCase
self::assertEquals('content', $audit->getContent()); self::assertEquals('content', $audit->getContent());
self::assertEquals('old', $audit->getOld()); self::assertEquals('old', $audit->getOld());
self::assertEquals('new', $audit->getNew()); self::assertEquals('new', $audit->getNew());
self::assertEquals(\ip2long('127.0.0.1'), $audit->getIp());
self::assertEquals(0, $audit->getCreatedBy()->getId()); self::assertEquals(0, $audit->getCreatedBy()->getId());
} }
} }