Merge branch 'main' into wasm

Author: tharun571

docs/environment.yml

@@ -5,5 +5,6 @@ channels:
 
 dependencies:
   - breathe
+  - graphviz
   - pip:
-    - sphinx_rtd_theme
+    - sphinx_rtd_theme

environment-wasm-host.yml

@@ -3,7 +3,7 @@ channels:
   - https://repo.prefix.dev/emscripten-forge-dev
   - https://repo.mamba.pm/conda-forge
 dependencies:
-  - nlohmann_json
+  - nlohmann_json=3.11.3
   - xeus-lite
   - xeus
   - CppInterOp

src/xinspect.cpp

@@ -132,6 +132,7 @@ namespace xcpp
         if (std::regex_search(to_inspect, method, std::regex(R"((.*)\.(\w*)$)")))
         {
             std::string type_name = find_type_slow(method[1]);
+            type_name = (type_name.empty()) ? method[1] : type_name;
 
             if (!type_name.empty())
             {

test/test_interpreter.cpp

@@ -134,6 +134,43 @@ TEST_SUITE("execute_request")
         REQUIRE(result["status"] == "ok");
     }
 
+    TEST_CASE("fetch_documentation_of_member_or_parameter")
+    {
+        std::vector<const char*> Args = {/*"-v", "resource-dir", "....."*/};
+        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";
+        nl::json user_expressions = nl::json::object();
+        xeus::execute_request_config config;
+        config.silent = false;
+        config.store_history = false;
+        config.allow_stdin = false;
+        nl::json header = nl::json::object();
+        xeus::xrequest_context::guid_list id = {};
+        xeus::xrequest_context context(header, id);
+
+        std::promise<nl::json> promise;
+        std::future<nl::json> future = promise.get_future();
+        auto callback = [&promise](nl::json result) {
+            promise.set_value(result);
+        };
+
+        interpreter.execute_request(
+            std::move(context),
+            std::move(callback),
+            code,
+            std::move(config),
+            user_expressions
+        );
+        nl::json result = future.get();
+        REQUIRE(result["payload"][0]["data"]["text/plain"] == inspect_result);
+        REQUIRE(result["user_expressions"] == nl::json::object());
+        REQUIRE(result["found"] == true);
+        REQUIRE(result["status"] == "ok");
+    }
+
+
     TEST_CASE("bad_status")
     {
         std::vector<const char*> Args = {"resource-dir"};
@@ -861,6 +898,14 @@ TEST_SUITE("xinspect"){
         cmp.child_value = "nonexistentMethod";
         REQUIRE(cmp(node) == false);
     }
+
+    TEST_CASE("is_inspect_request"){ 
+        std::string code = "vector";
+        std::regex re_expression(R"(non_matching_pattern)");
+        std::pair<bool, std::smatch> result = xcpp::is_inspect_request(code, re_expression);
+        REQUIRE(result.first == false);
+    }
+
 }
 
 #if !defined(XEUS_CPP_EMSCRIPTEN_WASM_BUILD)