6545986262

Committed 17 Oct 2023 10:35AM UTC coverage: 37.305% (+0.02%) from 37.285%

Build # 6545986262

Build Type

push

github

Committed by

web-flow

Commit Message

feat(elasticsearch): filtering on nested fields (#5835)

* feat(elasticsearch): filtering on nested fields

* Add additional checks and function doc for less confusion

* Fixing tests

Run Details

56 of 56 new or added lines in 4 files covered. (100.0%)

10266 of 27519 relevant lines covered (37.31%)

19.99 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/src/Elasticsearch/Tests/Util/FieldDatatypeTraitTest.php

<?php

/*
 * This file is part of the API Platform project.
 *
 * (c) Kévin Dunglas <dunglas@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace ApiPlatform\Elasticsearch\Tests\Util;

use ApiPlatform\Elasticsearch\Tests\Fixtures\Foo;
use ApiPlatform\Elasticsearch\Util\FieldDatatypeTrait;
use ApiPlatform\Metadata\ApiProperty;
use ApiPlatform\Metadata\Exception\PropertyNotFoundException;
use ApiPlatform\Metadata\Property\Factory\PropertyMetadataFactoryInterface;
use ApiPlatform\Metadata\ResourceClassResolverInterface;
use PHPUnit\Framework\TestCase;
use Prophecy\PhpUnit\ProphecyTrait;
use Symfony\Component\PropertyInfo\Type;

class FieldDatatypeTraitTest extends TestCase
{
    use ProphecyTrait;

    public function testGetNestedFieldPath(): void
    {
        $fieldDatatype = $this->getValidFieldDatatype();

        self::assertSame('foo.bar', $fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar.baz'));
        self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.baz'));
        self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'baz'));
    }

    public function testGetNestedFieldInNestedCollection(): void
    {
        $fieldDatatype = $this->getValidFieldDatatype();

        self::assertSame('foo.foo.bar', $fieldDatatype->getNestedFieldPath(Foo::class, 'foo.foo.bar.baz'));
        self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.foo.baz'));
        self::assertSame('foo.bar', $fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar.foo.baz'));
        self::assertSame('bar', $fieldDatatype->getNestedFieldPath(Foo::class, 'bar.foo'));
        self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'baz'));
    }

    public function testGetNestedFieldPathWithPropertyNotFound(): void
    {
        $propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);
        $propertyMetadataFactoryProphecy->create(Foo::class, 'foo')->willThrow(new PropertyNotFoundException())->shouldBeCalled();

        $fieldDatatype = self::createFieldDatatypeInstance($propertyMetadataFactoryProphecy->reveal(), $this->prophesize(ResourceClassResolverInterface::class)->reveal());

        self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar'));
    }

    public function testGetNestedFieldPathWithPropertyWithoutType(): void
    {
        $propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);
        $propertyMetadataFactoryProphecy->create(Foo::class, 'foo')->willReturn(new ApiProperty())->shouldBeCalled();

        $fieldDatatype = self::createFieldDatatypeInstance($propertyMetadataFactoryProphecy->reveal(),
            $this->prophesize(ResourceClassResolverInterface::class)->reveal());

        self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar'));
    }

    public function testGetNestedFieldPathWithInvalidCollectionType(): void
    {
        $propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);
        $propertyMetadataFactoryProphecy->create(Foo::class, 'foo')->willReturn((new ApiProperty())->withBuiltinTypes([new Type(Type::BUILTIN_TYPE_STRING)]))->shouldBeCalled();

        $fieldDatatype = self::createFieldDatatypeInstance($propertyMetadataFactoryProphecy->reveal(),
            $this->prophesize(ResourceClassResolverInterface::class)->reveal());

        self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar'));
    }

    public function testIsNestedField(): void
    {
        $fieldDatatype = $this->getValidFieldDatatype();

        self::assertTrue($fieldDatatype->isNestedField(Foo::class, 'foo.bar.baz'));
        self::assertFalse($fieldDatatype->isNestedField(Foo::class, 'foo.baz'));
        self::assertFalse($fieldDatatype->isNestedField(Foo::class, 'baz'));
    }

    private function getValidFieldDatatype()
    {
        $fooType = new Type(Type::BUILTIN_TYPE_OBJECT, false, Foo::class);
        $barType = new Type(Type::BUILTIN_TYPE_ARRAY, false, Foo::class, true, new Type(Type::BUILTIN_TYPE_INT), new Type(Type::BUILTIN_TYPE_OBJECT, false, Foo::class));
        $bazType = new Type(Type::BUILTIN_TYPE_STRING, false, Foo::class);

        $propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);
        $propertyMetadataFactoryProphecy->create(Foo::class, 'foo')->willReturn((new ApiProperty())->withBuiltinTypes([$fooType]))->shouldBeCalled();
        $propertyMetadataFactoryProphecy->create(Foo::class, 'bar')->willReturn((new ApiProperty())->withBuiltinTypes([$barType]))->shouldBeCalled();
        $propertyMetadataFactoryProphecy->create(Foo::class, 'baz')->willReturn((new ApiProperty())->withBuiltinTypes([$bazType]));

        $resourceClassResolverProphecy = $this->prophesize(ResourceClassResolverInterface::class);
        $resourceClassResolverProphecy->isResourceClass(Foo::class)->willReturn(true)->shouldBeCalled();

        return self::createFieldDatatypeInstance($propertyMetadataFactoryProphecy->reveal(),
            $resourceClassResolverProphecy->reveal());
    }

    private static function createFieldDatatypeInstance(PropertyMetadataFactoryInterface $propertyMetadataFactory, ResourceClassResolverInterface $resourceClassResolver)
    {
        return new class($propertyMetadataFactory, $resourceClassResolver) {
            use FieldDatatypeTrait {
                isNestedField as public;
                getNestedFieldPath as public;
            }

            public function __construct(PropertyMetadataFactoryInterface $propertyMetadataFactory, ResourceClassResolverInterface $resourceClassResolver)
            {
                $this->propertyMetadataFactory = $propertyMetadataFactory;
                $this->resourceClassResolver = $resourceClassResolver;
            }
        };
    }
}

1	<?php
2
3	/*
4	* This file is part of the API Platform project.
5	*
6	* (c) Kévin Dunglas <dunglas@gmail.com>
7	*
8	* For the full copyright and license information, please view the LICENSE
9	* file that was distributed with this source code.
10	*/
11
12	declare(strict_types=1);
13
14	namespace ApiPlatform\Elasticsearch\Tests\Util;
15
16	use ApiPlatform\Elasticsearch\Tests\Fixtures\Foo;
17	use ApiPlatform\Elasticsearch\Util\FieldDatatypeTrait;
18	use ApiPlatform\Metadata\ApiProperty;
19	use ApiPlatform\Metadata\Exception\PropertyNotFoundException;
20	use ApiPlatform\Metadata\Property\Factory\PropertyMetadataFactoryInterface;
21	use ApiPlatform\Metadata\ResourceClassResolverInterface;
22	use PHPUnit\Framework\TestCase;
23	use Prophecy\PhpUnit\ProphecyTrait;
24	use Symfony\Component\PropertyInfo\Type;
25
26	class FieldDatatypeTraitTest extends TestCase
27	{
28	use ProphecyTrait;
29
30	public function testGetNestedFieldPath(): void
31	{
32	$fieldDatatype = $this->getValidFieldDatatype();	×
33
34	self::assertSame('foo.bar', $fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar.baz'));	×
35	self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.baz'));	×
36	self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'baz'));	×
37	}
38
39	public function testGetNestedFieldInNestedCollection(): void
40	{
41	$fieldDatatype = $this->getValidFieldDatatype();	×
42
43	self::assertSame('foo.foo.bar', $fieldDatatype->getNestedFieldPath(Foo::class, 'foo.foo.bar.baz'));	×
44	self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.foo.baz'));	×
45	self::assertSame('foo.bar', $fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar.foo.baz'));	×
46	self::assertSame('bar', $fieldDatatype->getNestedFieldPath(Foo::class, 'bar.foo'));	×
47	self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'baz'));	×
48	}
49
50	public function testGetNestedFieldPathWithPropertyNotFound(): void
51	{
52	$propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);	×
53	$propertyMetadataFactoryProphecy->create(Foo::class, 'foo')->willThrow(new PropertyNotFoundException())->shouldBeCalled();	×
54
55	$fieldDatatype = self::createFieldDatatypeInstance($propertyMetadataFactoryProphecy->reveal(), $this->prophesize(ResourceClassResolverInterface::class)->reveal());	×
56
57	self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar'));	×
58	}
59
60	public function testGetNestedFieldPathWithPropertyWithoutType(): void
61	{
62	$propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);	×
63	$propertyMetadataFactoryProphecy->create(Foo::class, 'foo')->willReturn(new ApiProperty())->shouldBeCalled();	×
64
65	$fieldDatatype = self::createFieldDatatypeInstance($propertyMetadataFactoryProphecy->reveal(),	×
66	$this->prophesize(ResourceClassResolverInterface::class)->reveal());	×
67
68	self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar'));	×
69	}
70
71	public function testGetNestedFieldPathWithInvalidCollectionType(): void
72	{
73	$propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);	×
74	$propertyMetadataFactoryProphecy->create(Foo::class, 'foo')->willReturn((new ApiProperty())->withBuiltinTypes([new Type(Type::BUILTIN_TYPE_STRING)]))->shouldBeCalled();	×
75
76	$fieldDatatype = self::createFieldDatatypeInstance($propertyMetadataFactoryProphecy->reveal(),	×
77	$this->prophesize(ResourceClassResolverInterface::class)->reveal());	×
78
79	self::assertNull($fieldDatatype->getNestedFieldPath(Foo::class, 'foo.bar'));	×
80	}
81
82	public function testIsNestedField(): void
83	{
84	$fieldDatatype = $this->getValidFieldDatatype();	×
85
86	self::assertTrue($fieldDatatype->isNestedField(Foo::class, 'foo.bar.baz'));	×
87	self::assertFalse($fieldDatatype->isNestedField(Foo::class, 'foo.baz'));	×
88	self::assertFalse($fieldDatatype->isNestedField(Foo::class, 'baz'));	×
89	}
90
91	private function getValidFieldDatatype()
92	{
93	$fooType = new Type(Type::BUILTIN_TYPE_OBJECT, false, Foo::class);	×
94	$barType = new Type(Type::BUILTIN_TYPE_ARRAY, false, Foo::class, true, new Type(Type::BUILTIN_TYPE_INT), new Type(Type::BUILTIN_TYPE_OBJECT, false, Foo::class));	×
95	$bazType = new Type(Type::BUILTIN_TYPE_STRING, false, Foo::class);	×
96
97	$propertyMetadataFactoryProphecy = $this->prophesize(PropertyMetadataFactoryInterface::class);	×
98	$propertyMetadataFactoryProphecy->create(Foo::class, 'foo')->willReturn((new ApiProperty())->withBuiltinTypes([$fooType]))->shouldBeCalled();	×
99	$propertyMetadataFactoryProphecy->create(Foo::class, 'bar')->willReturn((new ApiProperty())->withBuiltinTypes([$barType]))->shouldBeCalled();	×
100	$propertyMetadataFactoryProphecy->create(Foo::class, 'baz')->willReturn((new ApiProperty())->withBuiltinTypes([$bazType]));	×
101
102	$resourceClassResolverProphecy = $this->prophesize(ResourceClassResolverInterface::class);	×
103	$resourceClassResolverProphecy->isResourceClass(Foo::class)->willReturn(true)->shouldBeCalled();	×
104
105	return self::createFieldDatatypeInstance($propertyMetadataFactoryProphecy->reveal(),	×
106	$resourceClassResolverProphecy->reveal());	×
107	}
108
109	private static function createFieldDatatypeInstance(PropertyMetadataFactoryInterface $propertyMetadataFactory, ResourceClassResolverInterface $resourceClassResolver)
110	{
111	return new class($propertyMetadataFactory, $resourceClassResolver) {	×
112	use FieldDatatypeTrait {
113	isNestedField as public;
114	getNestedFieldPath as public;
115	}
116
117	public function __construct(PropertyMetadataFactoryInterface $propertyMetadataFactory, ResourceClassResolverInterface $resourceClassResolver)
118	{
119	$this->propertyMetadataFactory = $propertyMetadataFactory;	×
120	$this->resourceClassResolver = $resourceClassResolver;	×
121	}
122	};	×
123	}
124	}

api-platform / core / 6545986262

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous