Dependent on #727

Can I Take this up once it is implemented?

@karandixit10 assigned, thanks for contributing. Please let me know if any other details required. Also you can collaborate with assignee of other issue.

@karandixit10 please let us know if you require any help with this.

@karandixit10 please let us know if you require any help with this.

I'll try to add this by EOD

Hi @lucifercr07 Do I have to create a mock DiceDB client to isolate the HTTP handler for Unit tests?

And will the folder structure for integration_tests be same as DiceDB tests?

@karandixit10 we can keep the same.

Hi @lucifercr07 for integration test I have to run DiceDB server in the terminal so will it fail in the CI when pushed?

Hello @karandixit10,

There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.

We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.

Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.

Thanks again.

Hi @lucifercr07 ,

I’ve reviewed the requirements for enhancing integration tests for the Playground CLI requests. I’m familiar with the dependencies and test structure for DiceDB. I’d like to take this up and ensure comprehensive validation, unit tests, and integration tests for the /cli API.

Please assign this issue to me so I can get started.

Hey @lucifercr07 @JyotinderSingh , I've a doubt regarding this issue, for writing the tests, those have to be written for playground-mono/internal/server/http.go, am I correct?

Hey @lucifercr07 @JyotinderSingh , I've a doubt regarding this issue, for writing the tests, those have to be written for playground-mono/internal/server/http.go, am I correct?

Yes, just ensure all edge cases and happy paths are being tested

Thanks! I'll create a PR for this soon 👍

Hey @JyotinderSingh , I've a doubt in here, that while I was trying to test the CliHandler(...) func, my proposed workflow is as follows:

1. Setup diceCmd as created in http.go at this line.
2. I've then check that if err != nil, then I've to return an err.
3. Then, on this line, as mentioned here, it executes the diceCmd, similarly this command must be executed in the test environment and any error should be returned(if present)?

Could you please have a look at my workflow and guide that if it's correct?