trial 4

Author: anutosh491

test/test_interpreter.cpp

@@ -99,50 +99,16 @@ TEST_SUITE("execute_request")
         REQUIRE(result["status"] == "ok");
     }
 
-// #if defined(XEUS_CPP_EMSCRIPTEN_WASM_BUILD)
-//     TEST_CASE("headers found in sysroot/include/compat")
-//     {
-//         std::vector<const char*> Args = {
-//             "-v",
-//             "-Xclang", "-iwithsysroot/include/compat"
-//         };
-
-//         xcpp::interpreter interpreter((int)Args.size(), Args.data());
-//         std::string code = "#include <xlocale.h>";
-//         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["status"] == "ok");
-//     }
-// #endif
-
-    TEST_CASE("fetch_documentation")
+#if defined(XEUS_CPP_EMSCRIPTEN_WASM_BUILD)
+    TEST_CASE("headers found in sysroot/include/compat")
     {
-        std::vector<const char*> Args = {/*"-v", "resource-dir", "....."*/};
-        xcpp::interpreter interpreter((int)Args.size(), Args.data());
+        std::vector<const char*> Args = {
+            "-v",
+            "-Xclang", "-iwithsysroot/include/compat"
+        };
 
-        std::string code = "?std::vector";
-        std::string inspect_result = "https://en.cppreference.com/w/cpp/container/vector";
+        xcpp::interpreter interpreter((int)Args.size(), Args.data());
+        std::string code = "#include <xlocale.h>";
         nl::json user_expressions = nl::json::object();
         xeus::execute_request_config config;
         config.silent = false;
@@ -166,19 +132,17 @@ TEST_SUITE("execute_request")
             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");
     }
+#endif
 
-    TEST_CASE("fetch_documentation_of_member_or_parameter")
+    TEST_CASE("fetch_documentation")
     {
         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";
+        std::string code = "?std::vector";
+        std::string inspect_result = "https://en.cppreference.com/w/cpp/container/vector";
         nl::json user_expressions = nl::json::object();
         xeus::execute_request_config config;
         config.silent = false;
@@ -208,13 +172,13 @@ TEST_SUITE("execute_request")
         REQUIRE(result["status"] == "ok");
     }
 
-
-    TEST_CASE("bad_status")
+    TEST_CASE("fetch_documentation_of_member_or_parameter")
     {
-        std::vector<const char*> Args = {"resource-dir"};
+        std::vector<const char*> Args = {/*"-v", "resource-dir", "....."*/};
         xcpp::interpreter interpreter((int)Args.size(), Args.data());
 
-        std::string code = "int x = ;";
+        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;
@@ -238,8 +202,44 @@ TEST_SUITE("execute_request")
             user_expressions
         );
         nl::json result = future.get();
-        REQUIRE(result["status"] == "error");
+        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"};
+    //     xcpp::interpreter interpreter((int)Args.size(), Args.data());
+
+    //     std::string code = "int x = ;";
+    //     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["status"] == "error");
+    // }
 }
 
 TEST_SUITE("inspect_request")