Add tests to ModelManager and DateFilter

src/Filter/AbstractDateFilter.php

@@ -49,34 +49,44 @@ public function filter(ProxyQueryInterface $queryBuilder, $alias, $field, $data)
         $data['type'] = !isset($data['type']) || !is_numeric($data['type']) ? DateType::TYPE_EQUAL : $data['type'];
 
         // Some types do not require a value to be set (NULL, NOT NULL).
-        if (!$this->typeRequiresValue($data['type']) && !$data['value']) {
+        if (!($data['value'] ?? false) && !$this->typeRequiresValue($data['type'])) {
             return;
         }
 
         switch ($data['type']) {
             case DateType::TYPE_EQUAL:
+                $this->active = true;
+
                 return $this->applyTypeIsEqual($queryBuilder, $field, $data);
 
             case DateType::TYPE_GREATER_THAN:
                 if (!\array_key_exists('value', $data) || !$data['value']) {
                     return;
                 }
 
+                $this->active = true;
+
                 return $this->applyTypeIsGreaterThan($queryBuilder, $field, $data);
 
             case DateType::TYPE_LESS_EQUAL:
                 if (!\array_key_exists('value', $data) || !$data['value']) {
                     return;
                 }
 
+                $this->active = true;
+
                 return $this->applyTypeIsLessEqual($queryBuilder, $field, $data);
 
             case DateType::TYPE_NULL:
             case DateType::TYPE_NOT_NULL:
+                $this->active = true;
+
                 return $this->applyType($queryBuilder, $this->getOperator($data['type']), $field, null);
 
             case DateType::TYPE_GREATER_EQUAL:
             case DateType::TYPE_LESS_THAN:
+                $this->active = true;
+
                 return $this->applyType($queryBuilder, $this->getOperator($data['type']), $field, $data['value']);
         }
     }
@@ -111,6 +121,30 @@ public function getRenderSettings()
         ]];
     }
 
+    /**
+     * @param string $field
+     * @param array  $data
+     */
+    protected function applyTypeIsLessEqual(ProxyQueryInterface $queryBuilder, $field, $data)
+    {
+    }
+
+    /**
+     * @param string $field
+     * @param array  $data
+     */
+    protected function applyTypeIsGreaterThan(ProxyQueryInterface $queryBuilder, $field, $data)
+    {
+    }
+
+    /**
+     * @param string $field
+     * @param array  $data
+     */
+    protected function applyTypeIsEqual(ProxyQueryInterface $queryBuilder, $field, $data)
+    {
+    }
+
     /**
      * @param string    $operation
      * @param string    $field

src/Filter/DateFilter.php

@@ -14,9 +14,15 @@
 namespace Sonata\DoctrineMongoDBAdminBundle\Filter;
 
 use Sonata\AdminBundle\Datagrid\ProxyQueryInterface;
+use Symfony\Component\Form\Extension\Core\Type\DateType;
 
 class DateFilter extends AbstractDateFilter
 {
+    public function getFieldType()
+    {
+        return $this->getOption('field_type', DateType::class);
+    }
+
     /**
      * @param string $field
      * @param array  $data

src/Filter/DateTimeFilter.php

@@ -14,6 +14,7 @@
 namespace Sonata\DoctrineMongoDBAdminBundle\Filter;
 
 use Sonata\AdminBundle\Datagrid\ProxyQueryInterface;
+use Symfony\Component\Form\Extension\Core\Type\DateTimeType;
 
 class DateTimeFilter extends AbstractDateFilter
 {
@@ -24,6 +25,11 @@ class DateTimeFilter extends AbstractDateFilter
      */
     protected $time = true;
 
+    public function getFieldType()
+    {
+        return $this->getOption('field_type', DateTimeType::class);
+    }
+
     /**
      * @param string $field
      * @param array  $data

src/Model/ModelManager.php

@@ -359,6 +359,17 @@ public function getExportFields($class)
      */
     public function getModelInstance($class)
     {
+        $r = new \ReflectionClass($class);
+        if ($r->isAbstract()) {
+            throw new \InvalidArgumentException(sprintf('Cannot initialize abstract class: %s', $class));
+        }
+
+        $constructor = $r->getConstructor();
+
+        if (null !== $constructor && (!$constructor->isPublic() || $constructor->getNumberOfRequiredParameters() > 0)) {
+            return $r->newInstanceWithoutConstructor();
+        }
+
         return new $class();
     }
 
@@ -443,7 +454,7 @@ public function modelReverseTransform($class, array $array = [])
 
             if ($reflClass->hasMethod($setter)) {
                 if (!$reflClass->getMethod($setter)->isPublic()) {
-                    throw new \RuntimeException(sprintf('Method "%s()" is not public in class "%s"', $setter, $reflClass->getName()));
+                    throw new \BadMethodCallException(sprintf('Method "%s()" is not public in class "%s"', $setter, $reflClass->getName()));
                 }
 
                 $instance->$setter($value);
@@ -452,7 +463,7 @@ public function modelReverseTransform($class, array $array = [])
                 $instance->$property = $value;
             } elseif ($reflClass->hasProperty($property)) {
                 if (!$reflClass->getProperty($property)->isPublic()) {
-                    throw new \RuntimeException(sprintf('Property "%s" is not public in class "%s". Maybe you should create the method "set%s()"?', $property, $reflClass->getName(), ucfirst($property)));
+                    throw new \BadMethodCallException(sprintf('Property "%s" is not public in class "%s". Maybe you should create the method "set%s()"?', $property, $reflClass->getName(), ucfirst($property)));
                 }
 
                 $instance->$property = $value;

tests/Filter/DateTimeFilterTest.php

@@ -0,0 +1,81 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * This file is part of the Sonata Project package.
+ *
+ * (c) Thomas Rabaix <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Sonata\DoctrineMongoDBAdminBundle\Tests\Filter;
+
+use Sonata\AdminBundle\Form\Type\Filter\DateType;
+use Sonata\DoctrineMongoDBAdminBundle\Datagrid\ProxyQuery;
+use Sonata\DoctrineMongoDBAdminBundle\Filter\DateTimeFilter;
+use Symfony\Component\Form\Extension\Core\Type\DateTimeType;
+
+final class DateTimeFilterTest extends FilterWithQueryBuilderTest
+{
+    public function testEmpty(): void
+    {
+        $filter = new DateTimeFilter();
+        $filter->initialize('field_name', ['field_options' => ['class' => 'FooBar']]);
+
+        $builder = new ProxyQuery($this->getQueryBuilder());
+
+        $builder->getQueryBuilder()
+            ->expects($this->never())
+            ->method('field')
+        ;
+
+        $filter->filter($builder, 'alias', 'field', null);
+        $filter->filter($builder, 'alias', 'field', 'all');
+        $filter->filter($builder, 'alias', 'field', []);
+
+        $this->assertFalse($filter->isActive());
+    }
+
+    public function testGetType(): void
+    {
+        $this->assertSame(DateTimeType::class, (new DateTimeFilter())->getFieldType());
+    }
+
+    /**
+     * @dataProvider getExamples
+     */
+    public function testFilter(array $data, string $method): void
+    {
+        $filter = new DateTimeFilter();
+        $filter->initialize('field_name', ['field_options' => ['class' => 'FooBar']]);
+
+        $builder = new ProxyQuery($this->getQueryBuilder());
+
+        $builder->getQueryBuilder()
+            ->expects($this->once())
+            ->method($method)
+            ->with($data['value'] ?? null)
+        ;
+
+        $filter->filter($builder, 'alias', 'field', $data);
+
+        $this->assertTrue($filter->isActive());
+    }
+
+    public function getExamples(): array
+    {
+        return [
+            [['type' => DateType::TYPE_EQUAL, 'value' => new \DateTime('now')], 'range'],
+            [['type' => DateType::TYPE_GREATER_EQUAL, 'value' => new \DateTime('now')], 'gte'],
+            [['type' => DateType::TYPE_GREATER_THAN, 'value' => new \DateTime('now')], 'gt'],
+            [['type' => DateType::TYPE_LESS_EQUAL, 'value' => new \DateTime('now')], 'lte'],
+            [['type' => DateType::TYPE_LESS_THAN, 'value' => new \DateTime('now')], 'lt'],
+            [['type' => DateType::TYPE_NULL], 'equals'],
+            [['type' => DateType::TYPE_NOT_NULL], 'notEqual'],
+            [['value' => new \DateTime('now')], 'range'],
+        ];
+    }
+}

tests/Fixtures/Document/AbstractDocument.php

@@ -0,0 +1,16 @@
+<?php
+
+/*
+ * This file is part of the Sonata Project package.
+ *
+ * (c) Thomas Rabaix <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Sonata\DoctrineMongoDBAdminBundle\Tests\Fixtures\Document;
+
+abstract class AbstractDocument
+{
+}

tests/Fixtures/Document/AssociatedDocument.php

@@ -0,0 +1,29 @@
+<?php
+
+/*
+ * This file is part of the Sonata Project package.
+ *
+ * (c) Thomas Rabaix <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Sonata\DoctrineMongoDBAdminBundle\Tests\Fixtures\Document;
+
+class AssociatedDocument
+{
+    private $plainField;
+    private $embeddedDocument;
+
+    public function __construct(int $plainField, EmbeddedDocument $embeddedDocument)
+    {
+        $this->plainField = $plainField;
+        $this->embeddedDocument = $embeddedDocument;
+    }
+
+    public function getPlainField(): int
+    {
+        return $this->plainField;
+    }
+}

tests/Fixtures/Document/ContainerDocument.php

@@ -0,0 +1,30 @@
+<?php
+
+/*
+ * This file is part of the Sonata Project package.
+ *
+ * (c) Thomas Rabaix <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Sonata\DoctrineMongoDBAdminBundle\Tests\Fixtures\Document;
+
+class ContainerDocument
+{
+    private $plainField;
+    private $associatedDocument;
+    private $embeddedDocument;
+
+    public function __construct(AssociatedDocument $associatedDocument, EmbeddedDocument $embeddedDocument)
+    {
+        $this->associatedDocument = $associatedDocument;
+        $this->embeddedDocument = $embeddedDocument;
+    }
+
+    public function getAssociatedDocument(): AssociatedDocument
+    {
+        return $this->associatedDocument;
+    }
+}

tests/Fixtures/Document/EmbeddedDocument.php

@@ -0,0 +1,20 @@
+<?php
+
+/*
+ * This file is part of the Sonata Project package.
+ *
+ * (c) Thomas Rabaix <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Sonata\DoctrineMongoDBAdminBundle\Tests\Fixtures\Document;
+
+class EmbeddedDocument
+{
+    /**
+     * @var bool
+     */
+    protected $plainField;
+}

tests/Fixtures/Document/ProtectedDocument.php

@@ -0,0 +1,19 @@
+<?php
+
+/*
+ * This file is part of the Sonata Project package.
+ *
+ * (c) Thomas Rabaix <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Sonata\DoctrineMongoDBAdminBundle\Tests\Fixtures\Document;
+
+class ProtectedDocument
+{
+    private function __construct()
+    {
+    }
+}

tests/Fixtures/Document/SimpleDocument.php

@@ -0,0 +1,38 @@
+<?php
+
+/*
+ * This file is part of the Sonata Project package.
+ *
+ * (c) Thomas Rabaix <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Sonata\DoctrineMongoDBAdminBundle\Tests\Fixtures\Document;
+
+class SimpleDocument
+{
+    private $schmeckles;
+    private $multiWordProperty;
+
+    public function getSchmeckles()
+    {
+        return $this->schmeckles;
+    }
+
+    public function setSchmeckles($value): void
+    {
+        $this->schmeckles = $value;
+    }
+
+    public function getMultiWordProperty()
+    {
+        return $this->multiWordProperty;
+    }
+
+    public function setMultiWordProperty($value): void
+    {
+        $this->multiWordProperty = $value;
+    }
+}