fix: controller generator

[innoflash]

Description

This PR aims to solve the issue I mentioned on #193
The controller generator seems to be misbehaving and we eventually have a need to go fix the generated code.

Short version is controllers are built with the wrong model.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• Refactor (refactoring a current feature, method, etc...)
• Code Coverage (adding/removing/updating/refactoring tests)
• New feature (non-breaking change which adds functionality)
• Remove feature (non-breaking change which removes functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

I simply added a model option to the inputs.
Updated command is:
php artisan apiato:generate:controller --container Locations --model District
The model can also be requested for if one isnt provided in the command

The generated file has this code:

<?php

namespace App\Containers\AppSection\Locations\UI\API\Controllers;

use Apiato\Core\Exceptions\IncorrectIdException;
use Apiato\Core\Exceptions\InvalidTransformerException;
use App\Containers\AppSection\Locations\Actions\CreateDistrictAction;
use App\Containers\AppSection\Locations\UI\API\Requests\CreateDistrictRequest;
use App\Containers\AppSection\Locations\UI\API\Transformers\DistrictTransformer;
use App\Ship\Exceptions\CreateResourceFailedException;
use App\Ship\Parents\Controllers\ApiController;
use Illuminate\Http\JsonResponse;

class TestFixController extends ApiController
{
    public function __construct(
        private readonly CreateDistrictAction $action,
    ) {
    }

    public function __invoke(CreateDistrictRequest $request): JsonResponse
    {
        $district = $this->action->run($request);

        return $this->created($this->transform($district, DistrictTransformer::class));
    }
}

This now makes use of the desired model in this container

[innoflash]

Works for plural modelling too:

public function __construct(
        private readonly ListDistrictsAction $action
    ) {
    }

    public function __invoke(ListDistrictsRequest $request): array
    {
        $districts = $this->action->run($request);

        return $this->transform($districts, DistrictTransformer::class);
    }

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 4.26%. Comparing base (a9855f1) to head (a194c77).
Report is 10 commits behind head on 8.x.

Files with missing lines	Patch %	Lines
src/Generator/Commands/ContainerApiGenerator.php	0.00%	2 Missing ⚠️
src/Generator/Commands/ContainerWebGenerator.php	0.00%	2 Missing ⚠️
src/Generator/Commands/ControllerGenerator.php	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##               8.x    #194      +/-   ##
==========================================
- Coverage     4.26%   4.26%   -0.01%     
  Complexity     668     668              
==========================================
  Files           92      92              
  Lines         2742    2746       +4     
==========================================
  Hits           117     117              
- Misses        2625    2629       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Mohammad-Alavi]

@innoflash Thank you for this! 💪
Ill review it in the comming days. Sorry for the delay 😭

[innoflash]

@innoflash Thank you for this! 💪 Ill review it in the comming days. Sorry for the delay 😭

Sure, take your time.

I may have found a couple more things I think I can help solve on the framework.
Otherwise Apiato is a marvel to work with @Mohammad-Alavi

[innoflash]

@Mohammad-Alavi where is the test file for the controller generation so I can add a test for this addition?