Command migration for JSON.ARRTRIM, JSON.ARRINSERT. JSON.OBJKEYS

[ayushsatyam146]

Closes #1027

Command migration for JSON.ARRTRIM, JSON.ARRINSERT. JSON.OBJKEYS

• Migrated the evalXXX function with the latest definition
• Update or add unit tests for the new implementation.
• All unit tests pass successfully.
• Ensure all integration tests pass successfully.
• Ensure integration tests are added for the migrated command on multi-threaded resp server.
• Move Integration tests for the respective commands under the RESP integration tests directory from Async directory
• Please validate that the documentation for the respective commands is up to date. If not then consider adding them.
• Add websocket integration tests

[soumya-codes]

@ayushsatyam146 we need to migrate the integration tests to the multi-threaded RESP server.

In the PR description, I have added the checklist of tasks required to be completed for the migration of commands to multi-threaded RESP server. Please follow the same.

[ayushsatyam146]

@soumya-codes I am yet to add integration tests. I raised the PR just to show activity and give updates. I'll soon add integration tests

[soumya-codes]

Ahh, thanks for the update. In that case can you please mark the PR status as draft?

[lucifercr07]

@ayushsatyam146 can we please add websocket tests also?
Also if you can follow sample doc template and update for these commands accordingly it'd be helpful.

[AshwinKul28]

HI @ayushsatyam146 Thanks for the commendable efforts. Most of the things look great. I see the documents added for each commands are not exactly aligning with the sample documentation. Can you please make required changes to make it align so that we don't need to revisit this in the future. Thanks again.

[ayushsatyam146]

HI @ayushsatyam146 Thanks for the commendable efforts. Most of the things look great. I see the documents added for each commands are not exactly aligning with the sample documentation. Can you please make required changes to make it align so that we don't need to revisit this in the future. Thanks again.

I'll change the docs asap

[AshwinKul28]

Hey @ayushsatyam146, can you please look at the conflicts and rebase them with the master? Then, we will immediately merge the PR.

[ayushsatyam146]

Hey @ayushsatyam146, can you please look at the conflicts and rebase them with the master? Then, we will immediately merge the PR.

Hi @AshwinKul28 I fixed the conflicts. Now it is good to merge. Thanks!