Added rector/rector

Author: tdwesten

composer.json

@@ -20,7 +20,7 @@
     },
     "scripts": {
         "test": [
-            "./vendor/bin/pest"
+            "./vendor/bin/pest && ./vendor/bin/rector"
         ],
         "generate-field": [
             "php field-generator/Generator.php"
@@ -38,7 +38,8 @@
         "orchestra/testbench": "^9.0",
         "spatie/laravel-ray": "^1.39",
         "pestphp/pest-plugin-laravel": "^3",
-        "statamic/eloquent-driver": "^4.16"
+        "statamic/eloquent-driver": "^4.16",
+        "rector/rector": "^2.0"
     },
     "config": {
         "allow-plugins": {

composer.lock

@@ -2,7 +2,7 @@
 
 use Tdwesten\StatamicBuilder\Enums\VisibilityOption;
 
-it('can render to a array', function () {
+it('can render to a array', function (): void {
     $field = new \Tdwesten\StatamicBuilder\FieldTypes\__Field__('title');
     $field->displayName('Display Name')
         ->instructions('Enter the title')

field-generator/FieldTestTemplate.php

@@ -0,0 +1,19 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+
+return RectorConfig::configure()
+    ->withPaths([
+        __DIR__ . '/config',
+        __DIR__ . '/field-generator',
+        __DIR__ . '/resources',
+        __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ])
+    // uncomment to reach your current PHP version
+    // ->withPhpSets()
+    ->withTypeCoverageLevel(0)
+    ->withDeadCodeLevel(0)
+    ->withCodeQualityLevel(0);

rector.php

@@ -33,10 +33,10 @@ private function exportBlueprints()
     {
         $namespaces = collect(config('statamic.builder.blueprints', []));
 
-        $namespaces->each(function ($blueprints, $namespace) {
+        $namespaces->each(function ($blueprints, $namespace): void {
             $blueprints = collect($blueprints);
 
-            $blueprints->each(function ($blueprint, $handle) use ($namespace) {
+            $blueprints->each(function ($blueprint, $handle) use ($namespace): void {
                 $this->exportBlueprint($namespace, $handle, $blueprint);
             });
         });
@@ -64,7 +64,7 @@ private function exportFieldSets()
     {
         $namespaces = collect(config('statamic.builder.fieldsets', []));
 
-        $namespaces->each(function ($fieldset) {
+        $namespaces->each(function ($fieldset): void {
             $this->exportFieldset($fieldset);
         });
     }

src/Console/Export.php

@@ -28,7 +28,7 @@ private function initializeCollections()
 
         $this->collections = collect();
 
-        $collections->each(function (string $collection) {
+        $collections->each(function (string $collection): void {
             if (class_exists($collection, true)) {
                 $this->collections->put($collection::handle(), $collection);
             }

src/Repositories/CollectionRepository.php

@@ -52,7 +52,7 @@ private function initializeGlobals()
 
         $this->globals = collect();
 
-        $globals->each(function (string $set) {
+        $globals->each(function (string $set): void {
             if (class_exists($set, true)) {
                 $this->globals->put($set::handle(), $set);
             }

src/Repositories/GlobalRepository.php

@@ -28,7 +28,7 @@ private function initializeTaxonomies()
 
         $this->taxonomies = collect();
 
-        $taxonomies->each(function (string $taxonomy) {
+        $taxonomies->each(function (string $taxonomy): void {
             if (class_exists($taxonomy, true)) {
                 $this->taxonomies->put($taxonomy::handle(), $taxonomy);
             }

src/Repositories/TaxonomyRepository.php

@@ -5,22 +5,22 @@
 use Tdwesten\StatamicBuilder\FieldTypes\Text;
 use Tests\Helpers\TestBlueprint;
 
-test('Has a title', function () {
+test('Has a title', function (): void {
     $blueprint = TestBlueprint::make('test_blueprint');
 
     expect($blueprint->toArray()['title'])->toBe(
         'Test Blueprint'
     );
 });
 
-it('can be set to hidden', function () {
+it('can be set to hidden', function (): void {
     $blueprint = TestBlueprint::make('test_blueprint');
     $blueprint->hidden(true);
 
     expect($blueprint->toArray()['hide'])->toBe(true);
 });
 
-test('Tabs are renderd', function () {
+test('Tabs are renderd', function (): void {
     $blueprint = TestBlueprint::make('test_blueprint');
 
     $expected = [
@@ -58,7 +58,7 @@
     expect($blueprint->toArray())->toBe($expected);
 });
 
-it('throws an exception when adding a field to a tab', function () {
+it('throws an exception when adding a field to a tab', function (): void {
     $blueprint = TestBlueprint::make('test_blueprint');
     $blueprint
         ->title('School')
@@ -69,14 +69,14 @@
     $blueprint->toArray();
 })->throws(BlueprintRenderException::class, 'Only sections are allowed in tabs');
 
-test('you can set a title', function () {
+test('you can set a title', function (): void {
     $blueprint = TestBlueprint::make('test_blueprint');
     $blueprint->title('School');
 
     expect($blueprint->toArray()['title'])->toBe('School');
 });
 
-test('you can get the handle', function () {
+test('you can get the handle', function (): void {
     $blueprint = TestBlueprint::make('test_blueprint');
 
     expect($blueprint->getHandle())->toBe('test_blueprint');

tests/Unit/BlueprintTest.php

@@ -3,31 +3,31 @@
 use Tests\Helpers\MultisiteTestCollection;
 use Tests\Helpers\TestCollection;
 
-test('Has a title', function () {
+test('Has a title', function (): void {
     $collection = new TestCollection;
 
     expect($collection->title())->toBe(
         'Shows'
     );
 });
 
-test('Has a handle', function () {
+test('Has a handle', function (): void {
     $collection = new TestCollection;
 
     expect($collection->handle())->toBe(
         'shows'
     );
 });
 
-test('Has a route', function () {
+test('Has a route', function (): void {
     $collection = new TestCollection;
 
     expect($collection->route())->toBe(
         '/shows/{slug}'
     );
 });
 
-test('Has multiple route for multisites', function () {
+test('Has multiple route for multisites', function (): void {
     $collection = new MultisiteTestCollection;
 
     expect($collection->route())->toBe(
@@ -38,15 +38,15 @@
     );
 });
 
-test('Can have multiple sites', function () {
+test('Can have multiple sites', function (): void {
     $collection = new MultisiteTestCollection;
 
     expect($collection->sites())->toBe(
         ['en', 'de']
     );
 });
 
-test('Has slugs', function () {
+test('Has slugs', function (): void {
     $collection = new TestCollection;
 
     expect($collection->slugs())->toBeTrue();