Update tags definition, add tests for xmagics, xinspect

Author: tharun571

CMakeLists.txt

@@ -455,6 +455,9 @@ else()
     add_definitions(-DXCPP_TAGCONFS_DIR="${CMAKE_INSTALL_PREFIX}/${XEUS_CPP_CONF_DIR}/tags.d")
 endif()
 
+configure_file("${CMAKE_CURRENT_SOURCE_DIR}/include/xeus-cpp/xeus_cpp_tags_config.hpp.in" 
+                "${CMAKE_CURRENT_BINARY_DIR}/include/xeus-cpp/xeus_cpp_tags_config.hpp")
+
 # Configure 'xeus-cppConfig.cmake.in for an install tree
 set(XEUS_CPP_CONFIG_CODE "")
 configure_package_config_file(${PROJECT_NAME}Config.cmake.in

include/xeus-cpp/xeus_cpp_tags_config.hpp.in

@@ -0,0 +1,15 @@
+/************************************************************************************
+ * Copyright (c) 2023, xeus-cpp contributors                                        *
+ *                                                                                  *
+ * Distributed under the terms of the BSD 3-Clause License.                         *
+ *                                                                                  *
+ * The full license is in the file LICENSE, distributed with this software.         *
+ ************************************************************************************/
+
+#ifndef XEUS_CPP_TAGS_CONFIG_HPP
+#define XEUS_CPP_TAGS_CONFIG_HPP
+
+#define XCPP_TAGFILES_DIR "@XCPP_TAGFILES_DIR@"
+#define XCPP_TAGCONFS_DIR "@XCPP_TAGCONFS_DIR@"
+
+#endif

test/test_interpreter.cpp

@@ -12,11 +12,14 @@
 #include "xeus-cpp/xmanager.hpp"
 #include "xeus-cpp/xutils.hpp"
 #include "xeus-cpp/xoptions.hpp"
+#include "xeus-cpp/xeus_cpp_tags_config.hpp"
 
 #include "../src/xparser.hpp"
 #include "../src/xsystem.hpp"
 #include "../src/xmagics/os.hpp"
+#include "../src/xinspect.hpp"
 
+#include <pugixml.hpp>
 #include <fstream>
 #if defined(__GNUC__) && !defined(XEUS_CPP_EMSCRIPTEN_WASM_BUILD)
     #include <sys/wait.h>
@@ -626,6 +629,22 @@ TEST_SUITE("xmagics_apply"){
         manager.apply("%dummy", kernel_res);
         REQUIRE(kernel_res["status"] == "error");
     } 
+
+    TEST_CASE("cell magic with empty cell body") {
+        xcpp::xmagics_manager manager;
+
+        std::stringstream ss;
+        auto cerr_buff = std::cerr.rdbuf();
+        std::cerr.rdbuf(ss.rdbuf());
+
+        manager.apply("test", "line", "");
+
+        std::cerr.rdbuf(cerr_buff);
+        REQUIRE(ss.str() == "UsageError: %%test is a cell magic, but the cell body is empty.\n"
+                      "If you only intend to display %%test help, please use a double line break to fill in the cell body.\n");
+
+    }
+
 }
 
 #if defined(__GNUC__) && !defined(XEUS_CPP_EMSCRIPTEN_WASM_BUILD)
@@ -673,4 +692,58 @@ TEST_SUITE("complete_request")
         }
         REQUIRE(found == 2);
     }
+}
+
+TEST_SUITE("xinspect"){
+    TEST_CASE("class_member_predicate_get_filename"){
+        xcpp::class_member_predicate cmp;
+        cmp.class_name = "TestClass";
+        cmp.kind = "public";
+        cmp.child_value = "testMethod";
+
+        pugi::xml_document doc;
+        pugi::xml_node node = doc.append_child("node");
+        node.append_attribute("kind") = "class";
+        pugi::xml_node name = node.append_child("name");
+        name.append_child(pugi::node_pcdata).set_value("TestClass");
+        pugi::xml_node child = node.append_child("node");
+        child.append_attribute("kind") = "public";
+        pugi::xml_node child_name = child.append_child("name");
+        child_name.append_child(pugi::node_pcdata).set_value("testMethod");
+        pugi::xml_node anchorfile = child.append_child("anchorfile");
+        anchorfile.append_child(pugi::node_pcdata).set_value("testfile.cpp");
+
+        REQUIRE(cmp.get_filename(node) == "testfile.cpp");
+    
+        cmp.child_value = "nonexistentMethod";
+        REQUIRE(cmp.get_filename(node) == "");
+    }
+
+    TEST_CASE("class_member_predicate_operator"){
+        xcpp::class_member_predicate cmp;
+        cmp.class_name = "TestClass";
+        cmp.kind = "public";
+        cmp.child_value = "testMethod";
+
+        pugi::xml_document doc;
+        pugi::xml_node node = doc.append_child("node");
+        node.append_attribute("kind") = "class";
+        pugi::xml_node name = node.append_child("name");
+        name.append_child(pugi::node_pcdata).set_value("TestClass");
+        pugi::xml_node child = node.append_child("node");
+        child.append_attribute("kind") = "public";
+        pugi::xml_node child_name = child.append_child("name");
+        child_name.append_child(pugi::node_pcdata).set_value("testMethod");
+
+    
+        REQUIRE(cmp(node) == true);
+        node.attribute("kind").set_value("struct");
+        REQUIRE(cmp(node) == true);
+        cmp.class_name = "NonexistentClass";
+        REQUIRE(cmp(node) == false);
+        cmp.kind = "private";
+        REQUIRE(cmp(node) == false);
+        cmp.child_value = "nonexistentMethod";
+        REQUIRE(cmp(node) == false);
+    }
 }