Merge pull request #4778 from Laravel-Backpack/add-more-tests

Add tests

Author: tabacitu

phpunit.xml

@@ -13,6 +13,7 @@
         <testsuite name="Package Test Suite">
             <directory suffix=".php">./tests/</directory>
             <file phpVersion="8.1.0" phpVersionOperator=">=">./tests81/Unit/CrudPanel/CrudPanelUpdateTest.php</file>
+            <file phpVersion="8.1.0" phpVersionOperator=">=">./tests81/Unit/CrudPanel/CrudPanelFieldsTest.php</file>
         </testsuite>
     </testsuites>
     <filter>

src/app/Library/CrudPanel/Traits/FieldsProtectedMethods.php

@@ -242,15 +242,11 @@ protected function inferFieldTypeFromRelationType($relationType)
         switch ($relationType) {
             case 'BelongsTo':
                 return 'select';
-                break;
-
             case 'BelongsToMany':
             case 'MorphToMany':
                 return 'select_multiple';
-
             default:
                 return 'text';
-                break;
         }
     }
 

src/app/Library/CrudPanel/Traits/Relationships.php

@@ -209,7 +209,6 @@ public function guessIfFieldHasMultipleFromRelationType($relation_type)
             case 'MorphOneOrMany':
             case 'MorphToMany':
                 return true;
-
             default:
                 return false;
         }
@@ -228,7 +227,6 @@ public function guessIfFieldHasPivotFromRelationType($relation_type)
             case 'HasManyThrough':
             case 'MorphToMany':
                 return true;
-                break;
             default:
                 return false;
         }

src/helpers.php

@@ -61,6 +61,7 @@ function backpack_form_input()
             // regular fields don't need any aditional parsing
             if (strpos($row['name'], '[') === false) {
                 $result[$row['name']] = $row['value'];
+
                 continue;
             }
 
@@ -87,6 +88,7 @@ function backpack_form_input()
 
             if (isset($repeatableRowKey)) {
                 $result[$parentInputName][$repeatableRowKey][$inputName] = $row['value'];
+
                 continue;
             }
 
@@ -306,6 +308,9 @@ function is_multidimensional_array(array $array)
      */
     function backpack_pro()
     {
+        if (app()->runningUnitTests()) {
+            return true;
+        }
         if (! \Composer\InstalledVersions::isInstalled('backpack/pro')) {
             return false;
         }

tests/BaseTest.php

@@ -23,4 +23,14 @@ protected function getPackageProviders($app)
             BackpackServiceProvider::class,
         ];
     }
+
+    // allow us to run crud panel private/protected methods like `inferFieldTypeFromDbColumnType`
+    public function invokeMethod(&$object, $methodName, array $parameters = [])
+    {
+        $reflection = new \ReflectionClass(get_class($object));
+        $method = $reflection->getMethod($methodName);
+        $method->setAccessible(true);
+
+        return $method->invokeArgs($object, $parameters);
+    }
 }

tests/Unit/CrudPanel/CrudPanelAutoSetTest.php

@@ -868,14 +868,4 @@ public function testSetDoctrineTypesMapping()
         $type = $new_model_db_platform->getDoctrineTypeMapping('enum');
         $this->assertEquals('string', $type);
     }
-
-    // allow us to run crud panel private/protected methods like `inferFieldTypeFromDbColumnType`
-    public function invokeMethod(&$object, $methodName, array $parameters = [])
-    {
-        $reflection = new \ReflectionClass(get_class($object));
-        $method = $reflection->getMethod($methodName);
-        $method->setAccessible(true);
-
-        return $method->invokeArgs($object, $parameters);
-    }
 }

tests/Unit/CrudPanel/CrudPanelAutofocusTest.php

@@ -0,0 +1,18 @@
+<?php
+
+namespace Backpack\CRUD\Tests\Unit\CrudPanel;
+
+/**
+ * @covers Backpack\CRUD\app\Library\CrudPanel\Traits\Autofocus
+ */
+class CrudPanelAutofocusTest extends BaseCrudPanelTest
+{
+    public function testItCanEnableAndDisableAutofocus()
+    {
+        $this->crudPanel->setOperation('create');
+        $this->crudPanel->enableAutoFocus();
+        $this->assertTrue($this->crudPanel->getAutoFocusOnFirstField());
+        $this->crudPanel->disableAutofocus();
+        $this->assertFalse($this->crudPanel->getAutoFocusOnFirstField());
+    }
+}