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

Add test for encoding JsonSerializable with pre-processing #43

Merged
merged 2 commits into from Sep 13, 2023

Conversation

Tigrov
Copy link
Member

@Tigrov Tigrov commented Sep 13, 2023

See section About jsonSerialize() at #41

@what-the-diff
Copy link

what-the-diff bot commented Sep 13, 2023

PR Summary

  • Improvements to the JsonTest.php
    The JsonTest.php file has been improved for better functionality. Key modifications are made to test how objects get translated or "encoded" into JSON format.

  • Inclusion of the ArrayObject class
    The ArrayObject class has been included via a use statement. This class allows arrays to behave like objects which helps in advanced operations.

  • Changes to the 'testEncodeObjectProperties' method
    The testEncodeObjectProperties method was fine-tuned to only check ('assert') the result when the object Properties is converted ('encoded') to JSON.

  • Addition of 'testEncodeJsonSerializableWithPreprocessing' method
    A new method called testEncodeJsonSerializableWithPreprocessing was added. This new method checks the result when an array with the elements 1, 2, and 3, put inside a Data object, is converted to JSON format.

@Tigrov Tigrov marked this pull request as ready for review September 13, 2023 07:14
@Tigrov Tigrov requested a review from vjik September 13, 2023 07:18
@vjik vjik merged commit 776be3e into master Sep 13, 2023
32 checks passed
@vjik vjik deleted the add_test_for_encoding_jsonserializable branch September 13, 2023 07:27
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.

None yet

2 participants