Skip to content

Commit 0e951ba

Browse files
committed
TASK: count($result->toArray()) needs to be asserted
count($result) does another count query to elasticsearch
1 parent a8ed645 commit 0e951ba

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

Tests/Functional/Eel/ElasticSearchMultiDimensionQueryTest.php

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
use Exception;
1919
use Flowpack\ElasticSearch\ContentRepositoryAdaptor\Command\NodeIndexCommandController;
2020
use Flowpack\ElasticSearch\ContentRepositoryAdaptor\Eel\ElasticSearchQueryBuilder;
21+
use Flowpack\ElasticSearch\ContentRepositoryAdaptor\Eel\ElasticSearchQueryResult;
2122
use Flowpack\ElasticSearch\ContentRepositoryAdaptor\ElasticSearchClient;
2223
use Flowpack\ElasticSearch\ContentRepositoryAdaptor\Indexer\NodeIndexer;
2324
use Flowpack\ElasticSearch\ContentRepositoryAdaptor\Tests\Functional\Traits\ContentRepositoryMultiDimensionNodeCreationTrait;
@@ -109,8 +110,8 @@ public function countDefaultDimensionNodesTest(): void
109110
->sortDesc('title')
110111
->execute();
111112

112-
// expecting: root, document1, untranslated = 3
113-
static::assertCount(3, $resultDefault);
113+
static::assertCount(3, $resultDefault->toArray());
114+
static::assertNodeNames(['root', 'document1-default', 'document-untranslated'], $resultDefault);
114115
}
115116

116117
/**
@@ -125,8 +126,8 @@ public function countDeDimensionNodesTest(): void
125126
->sortDesc('title')
126127
->execute();
127128

128-
// expecting: root, document1, document2, document3, document4, untranslated (fallback from en_us) = 6
129-
static::assertCount(6, $resultDe);
129+
// expecting: root, document1-default, document2, document3, document4, untranslated (fallback from en_us) = 6
130+
static::assertCount(6, $resultDe->toArray(), 'Found nodes: ' . implode(',', $this->extractNodeNames($resultDe)));
130131
}
131132

132133
/**
@@ -142,7 +143,7 @@ public function countDkDimensionNodesTest(): void
142143
->execute();
143144

144145
// expecting: root, document1, document2, document4 (fallback from de), untranslated (fallback from en_us) = 6
145-
static::assertCount(5, $resultDk);
146+
static::assertCount(5, $resultDk->toArray());
146147
}
147148

148149

0 commit comments

Comments
 (0)