[Bug] remove compatible models from brick instance list endpoint

[mirkoCrobu]

Motivation

closes #114

The v1/apps/{id}/bricks response must not include compatible_models fields.

Change description

Decouple the v1/apps/{id}/bricks and v1/apps/{id}/bricks/{brick_id} response structures by creating a dedicated struct for the v1/apps/{id}/bricks endpoint.

Additional Notes

Reviewer checklist

• PR addresses a single concern.
• PR title and description are properly filled.
• Changes will be merged in main.
• Changes are covered by tests.
• Logging is meaningful in case of troubleshooting.

[dido18]

Don't we have tests for the endpoint or bricks ?
I see no modification in *_test.go files

[mirkoCrobu]

Don't we have tests for the endpoint or bricks? I see no modification in *_test.go files

We already have both unit and e2e tests for TestBricksDetails, added in the previous issues handled by Giulio and me.
Those tests cover all the expected fields, and I didn’t need to update them since this PR is removing a field that shouldn’t be there.
The field likely appeared due to a bad rebase or merge(?).

[dido18]

I think the require_model must be kept in the response, isn't it ?

[mirkoCrobu]

You are right. I preferred to restart the PR here: #120
Ensuring to have an up-to-date repo and integrating unit and e2e tests.

[mirkoCrobu]

moved #120