aboutsummaryrefslogtreecommitdiffstats
path: root/tests/lib/Files/Search
diff options
context:
space:
mode:
Diffstat (limited to 'tests/lib/Files/Search')
-rw-r--r--tests/lib/Files/Search/QueryOptimizer/CombinedTests.php114
-rw-r--r--tests/lib/Files/Search/QueryOptimizer/FlattenNestedBoolTest.php12
-rw-r--r--tests/lib/Files/Search/QueryOptimizer/MergeDistributiveOperationsTest.php78
-rw-r--r--tests/lib/Files/Search/QueryOptimizer/OrEqualsToInTest.php50
-rw-r--r--tests/lib/Files/Search/SearchIntegrationTest.php12
5 files changed, 143 insertions, 123 deletions
diff --git a/tests/lib/Files/Search/QueryOptimizer/CombinedTests.php b/tests/lib/Files/Search/QueryOptimizer/CombinedTests.php
index c5abd5603da..665224cb63e 100644
--- a/tests/lib/Files/Search/QueryOptimizer/CombinedTests.php
+++ b/tests/lib/Files/Search/QueryOptimizer/CombinedTests.php
@@ -1,5 +1,9 @@
<?php
+/**
+ * SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
namespace Test\Files\Search\QueryOptimizer;
use OC\Files\Search\QueryOptimizer\QueryOptimizer;
@@ -18,21 +22,21 @@ class CombinedTests extends TestCase {
$this->optimizer = new QueryOptimizer();
}
- public function testBasicOrOfAnds() {
+ public function testBasicOrOfAnds(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
])
]
);
@@ -43,35 +47,35 @@ class CombinedTests extends TestCase {
$this->assertEquals('(storage eq 1 and path in ["foo","bar","asd"])', $operator->__toString());
}
- public function testComplexSearchPattern1() {
+ public function testComplexSearchPattern1(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 2),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 2),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "201"),
- new SearchComparison(ISearchComparison::COMPARE_LIKE, "path", "201/%"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '201'),
+ new SearchComparison(ISearchComparison::COMPARE_LIKE, 'path', '201/%'),
]),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "301"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '301'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 4),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "401"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 4),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '401'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "302"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '302'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 4),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "402"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 4),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '402'),
]),
]
);
@@ -82,33 +86,33 @@ class CombinedTests extends TestCase {
$this->assertEquals('(storage eq 1 or (storage eq 2 and (path eq "201" or path like "201\/%")) or (storage eq 3 and path in ["301","302"]) or (storage eq 4 and path in ["401","402"]))', $operator->__toString());
}
- public function testComplexSearchPattern2() {
+ public function testComplexSearchPattern2(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 2),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 2),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "201"),
- new SearchComparison(ISearchComparison::COMPARE_LIKE, "path", "201/%"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '201'),
+ new SearchComparison(ISearchComparison::COMPARE_LIKE, 'path', '201/%'),
]),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "301"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '301'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 4),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "401"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 4),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '401'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "302"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '302'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 4),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "402"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 4),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', '402'),
]),
]
);
@@ -119,30 +123,30 @@ class CombinedTests extends TestCase {
$this->assertEquals('(storage eq 1 or (storage eq 2 and (path eq "201" or path like "201\/%")) or (storage eq 3 and path in ["301","302"]) or (storage eq 4 and path in ["401","402"]))', $operator->__toString());
}
- public function testApplySearchConstraints1() {
+ public function testApplySearchConstraints1(): void {
$operator = new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "mimetype", "image/png"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "mimetype", "image/jpeg"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', 'image/png'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', 'image/jpeg'),
]),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "files"),
- new SearchComparison(ISearchComparison::COMPARE_LIKE, "path", "files/%"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'files'),
+ new SearchComparison(ISearchComparison::COMPARE_LIKE, 'path', 'files/%'),
]),
]),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 2),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 2),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "files/301"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'files/301'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "files/302"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'files/302'),
]),
]),
]);
@@ -153,32 +157,32 @@ class CombinedTests extends TestCase {
$this->assertEquals('(mimetype in ["image\/png","image\/jpeg"] and ((storage eq 1 and (path eq "files" or path like "files\/%")) or storage eq 2 or (storage eq 3 and path in ["files\/301","files\/302"])))', $operator->__toString());
}
- public function testApplySearchConstraints2() {
+ public function testApplySearchConstraints2(): void {
$operator = new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "mimetype", "image/png"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "mimetype", "image/jpeg"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', 'image/png'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', 'image/jpeg'),
]),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "files"),
- new SearchComparison(ISearchComparison::COMPARE_LIKE, "path", "files/%"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'files'),
+ new SearchComparison(ISearchComparison::COMPARE_LIKE, 'path', 'files/%'),
]),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 2),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 2),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "files/301"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'files/301'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "files/302"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'files/302'),
]),
]),
]);
diff --git a/tests/lib/Files/Search/QueryOptimizer/FlattenNestedBoolTest.php b/tests/lib/Files/Search/QueryOptimizer/FlattenNestedBoolTest.php
index a21f2a19b90..1d43541a5a0 100644
--- a/tests/lib/Files/Search/QueryOptimizer/FlattenNestedBoolTest.php
+++ b/tests/lib/Files/Search/QueryOptimizer/FlattenNestedBoolTest.php
@@ -1,5 +1,9 @@
<?php
+/**
+ * SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
namespace Test\Files\Search\QueryOptimizer;
use OC\Files\Search\QueryOptimizer\FlattenNestedBool;
@@ -21,14 +25,14 @@ class FlattenNestedBoolTest extends TestCase {
$this->simplifier = new FlattenSingleArgumentBinaryOperation();
}
- public function testOrs() {
+ public function testOrs(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
])
]
);
diff --git a/tests/lib/Files/Search/QueryOptimizer/MergeDistributiveOperationsTest.php b/tests/lib/Files/Search/QueryOptimizer/MergeDistributiveOperationsTest.php
index 4c7ecc9d46f..9aaa7030aac 100644
--- a/tests/lib/Files/Search/QueryOptimizer/MergeDistributiveOperationsTest.php
+++ b/tests/lib/Files/Search/QueryOptimizer/MergeDistributiveOperationsTest.php
@@ -1,5 +1,9 @@
<?php
+/**
+ * SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
namespace Test\Files\Search\QueryOptimizer;
use OC\Files\Search\QueryOptimizer\FlattenSingleArgumentBinaryOperation;
@@ -21,21 +25,21 @@ class MergeDistributiveOperationsTest extends TestCase {
$this->simplifier = new FlattenSingleArgumentBinaryOperation();
}
- public function testBasicOrOfAnds() {
+ public function testBasicOrOfAnds(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
])
]
);
@@ -47,21 +51,21 @@ class MergeDistributiveOperationsTest extends TestCase {
$this->assertEquals('(storage eq 1 and (path eq "foo" or path eq "bar" or path eq "asd"))', $operator->__toString());
}
- public function testDontTouchIfNotSame() {
+ public function testDontTouchIfNotSame(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 2),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 2),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 3),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 3),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
])
]
);
@@ -73,21 +77,21 @@ class MergeDistributiveOperationsTest extends TestCase {
$this->assertEquals('((storage eq 1 and path eq "foo") or (storage eq 2 and path eq "bar") or (storage eq 3 and path eq "asd"))', $operator->__toString());
}
- public function testMergePartial() {
+ public function testMergePartial(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 2),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 2),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
])
]
);
@@ -99,7 +103,7 @@ class MergeDistributiveOperationsTest extends TestCase {
$this->assertEquals('((storage eq 1 and (path eq "foo" or path eq "bar")) or (storage eq 2 and path eq "asd"))', $operator->__toString());
}
- public function testOptimizeInside() {
+ public function testOptimizeInside(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_AND,
[
@@ -107,20 +111,20 @@ class MergeDistributiveOperationsTest extends TestCase {
ISearchBinaryOperator::OPERATOR_OR,
[
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
])
]
),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "mimetype", "text")
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', 'text')
]
);
$this->assertEquals('(((storage eq 1 and path eq "foo") or (storage eq 1 and path eq "bar") or (storage eq 1 and path eq "asd")) and mimetype eq "text")', $operator->__toString());
@@ -131,22 +135,22 @@ class MergeDistributiveOperationsTest extends TestCase {
$this->assertEquals('((storage eq 1 and (path eq "foo" or path eq "bar" or path eq "asd")) and mimetype eq "text")', $operator->__toString());
}
- public function testMoveInnerOperations() {
+ public function testMoveInnerOperations(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
]),
new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_AND, [
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "storage", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
- new SearchComparison(ISearchComparison::COMPARE_GREATER_THAN, "size", "100"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'storage', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
+ new SearchComparison(ISearchComparison::COMPARE_GREATER_THAN, 'size', '100'),
])
]
);
diff --git a/tests/lib/Files/Search/QueryOptimizer/OrEqualsToInTest.php b/tests/lib/Files/Search/QueryOptimizer/OrEqualsToInTest.php
index 3d3160079cd..d51fe3d0d16 100644
--- a/tests/lib/Files/Search/QueryOptimizer/OrEqualsToInTest.php
+++ b/tests/lib/Files/Search/QueryOptimizer/OrEqualsToInTest.php
@@ -1,5 +1,9 @@
<?php
+/**
+ * SPDX-FileCopyrightText: 2023 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
namespace Test\Files\Search\QueryOptimizer;
use OC\Files\Search\QueryOptimizer\FlattenSingleArgumentBinaryOperation;
@@ -21,13 +25,13 @@ class OrEqualsToInTest extends TestCase {
$this->simplifier = new FlattenSingleArgumentBinaryOperation();
}
- public function testOrs() {
+ public function testOrs(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
]
);
$this->assertEquals('(path eq "foo" or path eq "bar" or path eq "asd")', $operator->__toString());
@@ -38,15 +42,15 @@ class OrEqualsToInTest extends TestCase {
$this->assertEquals('path in ["foo","bar","asd"]', $operator->__toString());
}
- public function testOrsMultipleFields() {
+ public function testOrsMultipleFields(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "fileid", 1),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "fileid", 2),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "mimetype", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'fileid', 1),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'fileid', 2),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', 'asd'),
]
);
$this->assertEquals('(path eq "foo" or path eq "bar" or fileid eq 1 or fileid eq 2 or mimetype eq "asd")', $operator->__toString());
@@ -57,13 +61,13 @@ class OrEqualsToInTest extends TestCase {
$this->assertEquals('(path in ["foo","bar"] or fileid in [1,2] or mimetype eq "asd")', $operator->__toString());
}
- public function testPreserveHints() {
+ public function testPreserveHints(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
]
);
foreach ($operator->getArguments() as $argument) {
@@ -78,13 +82,13 @@ class OrEqualsToInTest extends TestCase {
$this->assertEquals(false, $operator->getQueryHint(ISearchComparison::HINT_PATH_EQ_HASH, true));
}
- public function testOrSomeEq() {
+ public function testOrSomeEq(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
- new SearchComparison(ISearchComparison::COMPARE_LIKE, "path", "foo%"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
+ new SearchComparison(ISearchComparison::COMPARE_LIKE, 'path', 'foo%'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
]
);
$this->assertEquals('(path eq "foo" or path like "foo%" or path eq "bar")', $operator->__toString());
@@ -95,17 +99,17 @@ class OrEqualsToInTest extends TestCase {
$this->assertEquals('(path in ["foo","bar"] or path like "foo%")', $operator->__toString());
}
- public function testOrsInside() {
+ public function testOrsInside(): void {
$operator = new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_AND,
[
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "mimetype", "text"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'mimetype', 'text'),
new SearchBinaryOperator(
ISearchBinaryOperator::OPERATOR_OR,
[
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "foo"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "bar"),
- new SearchComparison(ISearchComparison::COMPARE_EQUAL, "path", "asd"),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'foo'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'bar'),
+ new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'path', 'asd'),
]
)
]
diff --git a/tests/lib/Files/Search/SearchIntegrationTest.php b/tests/lib/Files/Search/SearchIntegrationTest.php
index 74018a597d9..e3e99bbfadd 100644
--- a/tests/lib/Files/Search/SearchIntegrationTest.php
+++ b/tests/lib/Files/Search/SearchIntegrationTest.php
@@ -1,5 +1,9 @@
<?php
+/**
+ * SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
namespace Test\Files\Search;
use OC\Files\Search\SearchBinaryOperator;
@@ -26,12 +30,12 @@ class SearchIntegrationTest extends TestCase {
}
- public function testThousandAndOneFilters() {
- $id = $this->cache->put("file10", ['size' => 1, 'mtime' => 50, 'mimetype' => 'foo/folder']);
+ public function testThousandAndOneFilters(): void {
+ $id = $this->cache->put('file10', ['size' => 1, 'mtime' => 50, 'mimetype' => 'foo/folder']);
$comparisons = [];
- for($i = 1; $i <= 1001; $i++) {
- $comparisons[] = new SearchComparison(ISearchComparison::COMPARE_EQUAL, "name", "file$i");
+ for ($i = 1; $i <= 1001; $i++) {
+ $comparisons[] = new SearchComparison(ISearchComparison::COMPARE_EQUAL, 'name', "file$i");
}
$operator = new SearchBinaryOperator(ISearchBinaryOperator::OPERATOR_OR, $comparisons);
$query = new SearchQuery($operator, 10, 0, []);