Skip to content

Commit 552eae3

Browse files
authored
Test with actual lock modes (#11465)
1 parent ee4b03a commit 552eae3

File tree

1 file changed

+15
-6
lines changed

1 file changed

+15
-6
lines changed

tests/Tests/ORM/Cache/Persister/Entity/EntityPersisterTestCase.php

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66

77
use Doctrine\Common\Collections\ArrayCollection;
88
use Doctrine\Common\Collections\Criteria;
9+
use Doctrine\DBAL\LockMode;
910
use Doctrine\ORM\Cache\Persister\CachedPersister;
1011
use Doctrine\ORM\Cache\Persister\Entity\AbstractEntityPersister;
1112
use Doctrine\ORM\Cache\Persister\Entity\CachedEntityPersister;
@@ -99,7 +100,7 @@ public function testInvokeGetSelectSQL(): void
99100
->with(
100101
self::identicalTo(['name' => 'Foo']),
101102
self::identicalTo([0]),
102-
self::identicalTo(1),
103+
self::identicalTo(LockMode::OPTIMISTIC),
103104
self::identicalTo(2),
104105
self::identicalTo(3),
105106
self::identicalTo([4])
@@ -109,7 +110,7 @@ public function testInvokeGetSelectSQL(): void
109110
self::assertSame('SELECT * FROM foo WERE name = ?', $persister->getSelectSQL(
110111
['name' => 'Foo'],
111112
[0],
112-
1,
113+
LockMode::OPTIMISTIC,
113114
2,
114115
3,
115116
[4]
@@ -228,13 +229,21 @@ public function testInvokeLoad(): void
228229
self::identicalTo($entity),
229230
self::identicalTo([0]),
230231
self::identicalTo([1]),
231-
self::identicalTo(2),
232+
self::identicalTo(LockMode::PESSIMISTIC_READ),
232233
self::identicalTo(3),
233234
self::identicalTo([4])
234235
)
235236
->willReturn($entity);
236237

237-
self::assertSame($entity, $persister->load(['id' => 1], $entity, [0], [1], 2, 3, [4]));
238+
self::assertSame($entity, $persister->load(
239+
['id' => 1],
240+
$entity,
241+
[0],
242+
[1],
243+
LockMode::PESSIMISTIC_READ,
244+
3,
245+
[4]
246+
));
238247
}
239248

240249
public function testInvokeLoadAll(): void
@@ -391,9 +400,9 @@ public function testInvokeLock(): void
391400

392401
$this->entityPersister->expects(self::once())
393402
->method('lock')
394-
->with(self::identicalTo($identifier), self::identicalTo(1));
403+
->with(self::identicalTo($identifier), self::identicalTo(LockMode::OPTIMISTIC));
395404

396-
$persister->lock($identifier, 1);
405+
$persister->lock($identifier, LockMode::OPTIMISTIC);
397406
}
398407

399408
public function testInvokeExists(): void

0 commit comments

Comments
 (0)