Skip to content

Commit 02be735

Browse files
committed
updated tests
1 parent 5ca5704 commit 02be735

11 files changed

Lines changed: 29 additions & 21 deletions

File tree

phpstan.neon

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,10 @@ parameters:
77
message: "#^Unable to resolve the template type T in call to method Nette\\\\DI\\\\Container\\:\\:getByType\\(\\)$#"
88
count: 2
99
path: src/Model/ServiceLocator.php
10-
11-
checkGenericClassInNonGenericObjectType: false
12-
checkMissingIterableValueType: false
10+
-
11+
identifier: missingType.iterableValue
12+
-
13+
identifier: missingType.generics
1314

1415
bootstrapFiles:
1516
- %rootDir%/../../../phpstanConstants.php

src/Entity/Collection/AbstractElasticEntityCollection.php

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -215,4 +215,16 @@ public function first(): \Spameri\Elastic\Entity\ElasticEntityInterface|null
215215
return \reset($this->collection) ?: null;
216216
}
217217

218+
public function __serialize(): array
219+
{
220+
return [
221+
'collection' => $this->collection,
222+
];
223+
}
224+
225+
public function __unserialize(array $data): void
226+
{
227+
$this->collection = $data;
228+
}
229+
218230
}

tests/SpameriTests/Elastic/DI/Extension.phpt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ class Extension extends \SpameriTests\Elastic\AbstractTestCase
1717

1818
/** @var \Spameri\Elastic\ClientProvider $clientProvider */
1919
$clientProvider = $this->container->getByType(\Spameri\Elastic\ClientProvider::class);
20-
$connection = $clientProvider->client()->transport->connectionPool->nextConnection();
20+
$connection = $clientProvider->client()->getTransport()->getNodePool()->nextNode()->getUri();
2121

2222
/** @var \Spameri\Elastic\Model\VersionProvider $versionProvider */
2323
$versionProvider = $this->container->getByType(\Spameri\Elastic\Model\VersionProvider::class);

tests/SpameriTests/Elastic/Data/Config/Common.neon

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ parameters:
1616

1717
services:
1818
elastic.person.personMapping:
19-
factory: SpameriTests\Elastic\Data\Model\PersonMapping(%elasticSearch.personIndex%, SpameriTests\Elastic\Data\Entity\Person::class)
19+
factory: SpameriTests\Elastic\Data\Model\PersonMapping(%elasticSearch.personIndex%, [SpameriTests\Elastic\Data\Entity\Person::class])
2020

2121
elastic.person.personFactory:
2222
factory: SpameriTests\Elastic\Data\Model\PersonFactory
@@ -35,7 +35,7 @@ services:
3535
########################################################################################################################
3636

3737
elastic.video.videoMapping:
38-
factory: SpameriTests\Elastic\Data\Model\VideoMapping(%elasticSearch.personIndex%, SpameriTests\Elastic\Data\Entity\Video::class)
38+
factory: SpameriTests\Elastic\Data\Model\VideoMapping(%elasticSearch.personIndex%, [SpameriTests\Elastic\Data\Entity\Video::class])
3939

4040
elastic.video.videoFactory:
4141
factory: SpameriTests\Elastic\Data\Model\VideoFactory

tests/SpameriTests/Elastic/Data/Model/PersonFactory.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ class PersonFactory implements \Spameri\Elastic\Factory\EntityFactoryInterface
1111
public function create(
1212
\Spameri\ElasticQuery\Response\Result\Hit $hit,
1313
string|null $class,
14-
\Spameri\Elastic\EntityManager $entityManager,
14+
\Spameri\Elastic\EntityManager|null $entityManager,
1515
): \Generator
1616
{
1717
yield new \SpameriTests\Elastic\Data\Entity\Person(

tests/SpameriTests/Elastic/Data/Model/VideoFactory.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ public function __construct(
1818
public function create(
1919
\Spameri\ElasticQuery\Response\Result\Hit $hit,
2020
string|null $class,
21-
\Spameri\Elastic\EntityManager $entityManager,
21+
\Spameri\Elastic\EntityManager|null $entityManager,
2222
): \Generator
2323
{
2424
yield new \SpameriTests\Elastic\Data\Entity\Video(

tests/SpameriTests/Elastic/Factory/EntityFactory.phpt

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,10 @@ class EntityFactory extends \SpameriTests\Elastic\AbstractTestCase
1212

1313
public function testCreate()
1414
{
15-
$entityFactory = new \Spameri\Elastic\Factory\EntityFactory();
16-
$prepareEntityArray = new \Spameri\Elastic\Model\Insert\PrepareEntityArray(
17-
\Mockery::mock(\Spameri\Elastic\Model\ServiceLocator::class)
18-
);
15+
$entityManager = $this->container->getByType(\Spameri\Elastic\EntityManager::class);
16+
$entityFactory = $this->container->getByType(\Spameri\Elastic\Factory\EntityFactory::class);
17+
$prepareEntityArray = $this->container->getByType(\Spameri\Elastic\Model\Insert\PrepareEntityArray::class);
18+
1919
$person = new \SpameriTests\Elastic\Data\Entity\Person(
2020
new \Spameri\Elastic\Entity\Property\ElasticId('asd123'),
2121
new \SpameriTests\Elastic\Data\Entity\Video\Identification(
@@ -52,7 +52,8 @@ class EntityFactory extends \SpameriTests\Elastic\AbstractTestCase
5252
);
5353

5454
/** @var \SpameriTests\Elastic\Data\Entity\Person $entity */
55-
$entity = $entityFactory->create($hit, \SpameriTests\Elastic\Data\Entity\Person::class)->current();
55+
$entity = $entityFactory->create($hit, \SpameriTests\Elastic\Data\Entity\Person::class, $entityManager)
56+
->current();
5657

5758
\Tester\Assert::same($person->id->value(), $entity->id->value());
5859
\Tester\Assert::same($person->identification->imdb->value(), $entity->identification->imdb->value());

tests/SpameriTests/Elastic/Model/Indices/CreateTest.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,11 @@ public function testProcess(): void
2121

2222
$create = new \Spameri\Elastic\Model\Indices\Create(
2323
$this->container->getByType(\Spameri\Elastic\ClientProvider::class),
24-
$this->container->getByType(\Spameri\Elastic\Model\VersionProvider::class),
2524
);
2625

2726
$create->execute(
2827
\SpameriTests\Elastic\Config::INDEX_CREATE,
2928
$videoMapping->provide()->toArray(),
30-
\SpameriTests\Elastic\Config::INDEX_CREATE,
3129
);
3230
$response = $getMapping->execute(\SpameriTests\Elastic\Config::INDEX_CREATE);
3331

tests/SpameriTests/Elastic/Model/Insert.phpt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ class Insert extends \SpameriTests\Elastic\AbstractTestCase
1414
$video = new \SpameriTests\Elastic\Data\Entity\Video(
1515
new \Spameri\Elastic\Entity\Property\EmptyElasticId(),
1616
new \SpameriTests\Elastic\Data\Entity\Video\Identification(
17-
new \SpameriTests\Elastic\Data\Entity\Property\ImdbId(4154796)
17+
new \SpameriTests\Elastic\Data\Entity\Property\ImdbId('4154796')
1818
),
1919
new \SpameriTests\Elastic\Data\Entity\Property\Name('Avengers: Endgame'),
2020
new \SpameriTests\Elastic\Data\Entity\Property\Year(2019),

tests/SpameriTests/Elastic/Model/RestoreIndex/Execute.phpt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ class Execute extends \SpameriTests\Elastic\AbstractTestCase
2929
$id = new \Spameri\Elastic\Entity\Property\ElasticId('192461');
3030
/** @var \Spameri\Elastic\Model\Get $get */
3131
$get = $this->container->getByType(\Spameri\Elastic\Model\Get::class);
32-
$result = $get->execute($id, \SpameriTests\Elastic\Config::INDEX_RESTORE, \SpameriTests\Elastic\Config::INDEX_RESTORE);
32+
$result = $get->execute($id, \SpameriTests\Elastic\Config::INDEX_RESTORE);
3333

3434
\Tester\Assert::same($id->value(), $result->hit()->id());
3535
}

0 commit comments

Comments
 (0)