Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhance the serializer tests #597

Merged
merged 4 commits into from
Dec 30, 2023
Merged

enhance the serializer tests #597

merged 4 commits into from
Dec 30, 2023

Conversation

binaryk
Copy link
Collaborator

@binaryk binaryk commented Dec 30, 2023

Fixed

  • enhance the serializer tests

Copy link

what-the-diff bot commented Dec 30, 2023

PR Summary

  • A new configuration file (pint.json) was added, which sets some rules for the project. This provides better control over the project settings.
  • The RoutesDefinition.php, PublishAuthCommand.php, and several other files had some of their function parameters modified to accept not only specific types like string, array or int but also null values. This increases the flexibility of these functions as they can now handle more diverse inputs.
  • The MailTracking.php, PostsShowGetter.php, PostPolicy.php, IntegrationTestCase.php, and Prototypeable.php files underwent similar changes for the same reason: to expand their versatility when handling null values.
  • A test was changed in RepositorySerializerTest.php to check for the existence of data.0.relationships.user and to confirm that the data.0.type value is 'posts'. This change helps to ensure the integrity and correctness of the data structure in our application.

@binaryk binaryk merged commit 7961f44 into 8.x Dec 30, 2023
17 checks passed
@binaryk binaryk deleted the fix/enhance-serializer branch December 30, 2023 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant