Added tests for xinspect#278
Conversation
|
clang-tidy review says "All clean, LGTM! 👍" |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #278 +/- ##
==========================================
+ Coverage 80.87% 82.15% +1.27%
==========================================
Files 20 20
Lines 957 958 +1
Branches 88 88
==========================================
+ Hits 774 787 +13
+ Misses 183 171 -12
🚀 New features to boost your workflow:
|
|
clang-tidy review says "All clean, LGTM! 👍" |
|
@anutosh491 This is not something that I can change. The format of I think this is how it is supposed to work. PR is complete from my side. |
92a2397 to
6dc6e14
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
anutosh491
left a comment
There was a problem hiding this comment.
LGTM !
Coverage goes up. Works with lite too so no complaints. Thanks.
| xcpp::interpreter interpreter((int)Args.size(), Args.data()); | ||
|
|
||
| std::string code = "?std::vector.push_back"; | ||
| std::string inspect_result = "https://en.cppreference.com/w/cpp/container/vector/push_back"; |
There was a problem hiding this comment.
Do we require internet connection for this test to work?
Description
fetch_documentation_of_member_or_parameterandis_inspect_request.inspectfunction. (Previously, one can't fetch documentation of member functions and variables.)Fixes #276