Add tests for xmagics, os, xutils, xinterpreter

[tharun571]

No description provided.

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[mcbarton]

sys/wait.h is for unix systems only. You will need a different header for Windows.

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[mcbarton]

unistd.h is also unix only.

[tharun571]

@mcbarton the handler function which has backtrace and backtrace_symbols_fd are also not available in windows. Shall I change that particular test for unix only?

[mcbarton]

@tharun571 Maybe you could try one of the suggestions from this stackoverflow page where they talk about Windows equivalents https://stackoverflow.com/questions/341817/is-there-a-replacement-for-unistd-h-for-windows-visual-c .

[tharun571]

Also is there a way to check windows build in local? or should I push the code everytime to check?

[mcbarton]

Also is there a way to check windows build in local? or should I push the code everytime to check?

If you have access to a Windows machine you could run the commands in the ci. There are currently no Windows instructions in the documentation. I will add at some point.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.08%. Comparing base (6692a45) to head (2f37664).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
+ Coverage   70.09%   75.08%   +4.98%     
==========================================
  Files          17       17              
  Lines         602      602              
  Branches       59       59              
==========================================
+ Hits          422      452      +30     
+ Misses        180      150      -30     

see 3 files with indirect coverage changes

see 3 files with indirect coverage changes

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[anutosh491]

Converting to draft, let us know when the PR is ready for review (after maybe squashing the commits too)

[tharun571]

@anutosh491 this covers most of #114. Having some trouble with complete_request. Will push that later.

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[github-actions]

clang-tidy review says "All clean, LGTM! 👍"

[anutosh491]

Looks good. Let's wait for maybe one more review before we merge this.
cc @mcbarton