Add Tests for all XX_request methods in test_interpreter.cpp

compiler-research / xeus-cpp

Jupyter kernel for the C++ programming language

BSD 3-Clause "New" or "Revised" License

17 stars 24 forks source link

Add Tests for all XX_request methods in test_interpreter.cpp #114

Closed anutosh491 closed 3 months ago

anutosh491 commented 3 months ago

I think now, for a start we have some implementation for all request methods. We also have tests for some of them and we can finish the others I guess

[x] execute_request
[x] inspect_request
[x] complete_request
[x] is_complete_request
[x] kernel_info_request
[x] shutdown_request

anutosh491 commented 3 months ago

cc @tharun571 you might be interested to cover some of these.

tharun571 commented 3 months ago

Writing tests for other files right now. Is it fine if I get on this after that?

anutosh491 commented 3 months ago

Yeah no issues at all. Just thought of making a note of what we've covered and what we should be addressing.

anutosh491 commented 3 months ago

This issue can now we closed as we've added basic tests for all request methods through #116 and #124