Fix and refactor feature test for creating posts

[zigzagdev]

Description

This PR addresses a correction in the feature test for creating posts, which previously failed due to improper handling of the visibility enum (PostVisibility). The following changes were made:

• Replaced the incorrect visibility value (e.g. 'PUBLIC') with the correct PostVisibility::PUBLIC enum instance
• Ensured that the visibility field passed to the request matches the expected int value (0 for public, 1 for private)
• Cleaned up request payload construction for clarity and type safety
• Added assertion for response content (optional: depending on current API contract)

These adjustments ensure the test now correctly reflects the contract defined by the domain enum and avoids TypeError thrown by PostVisibility::from().

[Copilot]

Pull Request Overview

This PR corrects the handling of the visibility enum in post creation tests and factory methods to prevent TypeError and improve type safety.

• Renamed the test method for clarity
• Cast incoming visibility values to the expected integer enum values
• Refactored entity build logic to normalize string inputs via a match expression

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
src/app/Post/Tests/Post_CreateTest.php	Renamed test method from test_feature_test to test_create_post
src/app/Post/Domain/EntityFactory/PostFromModelEntityFactory.php	Added (int) cast when calling PostVisibilityEnum::from
src/app/Post/Domain/Entity/PostEntity.php	Expanded visibility builder to handle string values with match
src/app/Post/Application/UseCase/CreateUseCase.php	Removed redundant blank line in handle method

[Copilot]

Casting an enum instance to int can be brittle. Use PostVisibilityEnum::PUBLIC->value as the default instead of the enum itself, e.g.: ($request['visibility'] ?? PostVisibilityEnum::PUBLIC->value).

Suggested change

	PostVisibilityEnum::from((int)($request['visibility'] ?? PostVisibilityEnum::PUBLIC))
	PostVisibilityEnum::from($request['visibility'] ?? PostVisibilityEnum::PUBLIC->value)

[Copilot]

The mapping for 'PUBLIC' assigns PostVisibilityEnum::PRIVATE->value, which inverts the intended behavior. It should map to PostVisibilityEnum::PUBLIC->value.

Suggested change

	'PUBLIC' => PostVisibilityEnum::PRIVATE->value,
	'PUBLIC' => PostVisibilityEnum::PUBLIC->value,