Merge pull request #41 from clementbirkle/main

Fix phpType Handling for Big Integer Columns

.phpunit.cache/test-results

@@ -0,0 +1 @@
+{"version":"pest_2.34.8","defects":[],"times":{"P\\Tests\\RelationTest::__pest_evaluable_it_will_find_no_relations_on_a_model_that_has_none":0,"P\\Tests\\RelationTest::__pest_evaluable_it_can_get_the_model_info_of_the_related_model":0.001,"P\\Tests\\RelationTest::__pest_evaluable_it_can_find_the_relations_on_a_model":0.001,"P\\Tests\\ModelFinderTest::__pest_evaluable_it_can_discover_all_models_in_a_directory":0.001,"P\\Tests\\AttributeTest::__pest_evaluable_it_can_get_extended_column_types_for_a_model":0.002,"P\\Tests\\AttributeTest::__pest_evaluable_it_can_get_the_attributes_of_a_model":0.002,"P\\Tests\\AttributeTest::__pest_evaluable_it_can_get_the_accessor_attributes_of_a_model":0.001,"P\\Tests\\AttributeTest::__pest_evaluable_it_can_get_the_mutator_attributes_of_a_model":0.001,"P\\Tests\\AttributeTest::__pest_evaluable_it_can_handle_virtual_attributes_of_a_model":0.001,"P\\Tests\\AttributeTest::__pest_evaluable_it_can_handle_accessor_mutator_combinations":0.002,"P\\Tests\\ModelInfoTest::__pest_evaluable_it_can_get_meta_information_about_all_models":0.037,"P\\Tests\\ModelInfoTest::__pest_evaluable_it_can_get_traits_from_a_model":0.001,"P\\Tests\\ModelInfoTest::__pest_evaluable_it_it_will_return_null_when_getting_a_non_existing_attribute":0.001,"P\\Tests\\ModelInfoTest::__pest_evaluable_it_can_get_a_specific_relation":0.002,"P\\Tests\\ModelInfoTest::__pest_evaluable_it_can_get_extra_info_from_a_model":0.001,"P\\Tests\\ModelInfoTest::__pest_evaluable_it_can_get_a_specific_attribute":0.001,"P\\Tests\\ModelInfoTest::__pest_evaluable_it_it_will_return_null_when_getting_a_non_existing_relation":0.001,"P\\Tests\\ModelInfoTest::__pest_evaluable_it_can_get_meta_information_about_a_model":0.013}}

src/Attributes/AttributeFinder.php

@@ -68,7 +68,7 @@ protected function getPhpType(array $column): string
         };
 
         $type ??= match ($column['type_name']) {
-            'tinyint', 'integer', 'int', 'int4', 'smallint', 'int2', 'mediumint' => 'int',
+            'tinyint', 'integer', 'int', 'int4', 'smallint', 'int2', 'mediumint', 'bigint', 'int8' => 'int',
             'float', 'real', 'float4', 'double', 'float8' => 'float',
             'binary', 'varbinary', 'bytea', 'image', 'blob', 'tinyblob', 'mediumblob', 'longblob' => 'resource',
             'boolean', 'bool' => 'bool',
@@ -77,7 +77,6 @@ protected function getPhpType(array $column): string
             // 'char', 'bpchar', 'nchar',
             // 'varchar', 'nvarchar',
             // 'text', 'tinytext', 'longtext', 'mediumtext', 'ntext',
-            // 'bigint', 'int8',
             // 'decimal', 'numeric',
             // 'money', 'smallmoney',
             // 'uuid', 'uniqueidentifier',

tests/AttributeTest.php

@@ -10,7 +10,7 @@
 it('can get the attributes of a model', function () {
     $attributes = AttributeFinder::forModel(new TestModel());
 
-    expect($attributes)->toHaveCount(11);
+    expect($attributes)->toHaveCount(12);
 
     matchesAttributesSnapshot($attributes);
 });

tests/TestSupport/TestCase.php

@@ -19,6 +19,7 @@ public function getEnvironmentSetUp($app)
             $table->id();
             $table->string('name');
             $table->string('password');
+            $table->bigInteger('bigInteger');
             $table->timestamps();
         });
 

tests/__snapshots__/AttributeTest__it_can_get_the_attributes_of_a_model__1.yml

@@ -40,6 +40,20 @@
     cast: null
     virtual: false
     hidden: true
+-
+    name: bigInteger
+    phpType: int
+    type: integer
+    increments: false
+    nullable: false
+    default: null
+    primary: false
+    unique: false
+    fillable: false
+    appended: null
+    cast: null
+    virtual: false
+    hidden: false
 -
     name: created_at
     phpType: DateTime