Disable xsystem for WASM

Author: tharun571

src/xsystem.hpp

@@ -28,6 +28,14 @@ namespace xcpp
 
         void apply(const std::string& code, nl::json& kernel_res) override
         {
+#if defined(XEUS_CPP_EMSCRIPTEN_WASM_BUILD)
+            // WASM environment: Disable shell commands
+            kernel_res["status"] = "error";
+            kernel_res["ename"] = "UnsupportedEnvironment";
+            kernel_res["evalue"] = "Shell commands are not supported in the WASM environment.";
+            kernel_res["traceback"] = nl::json::array();
+#else
+            // Native environment: Execute shell commands
             std::regex re(spattern + R"((.*))");
             std::smatch to_execute;
             std::regex_search(code, to_execute, re);
@@ -65,6 +73,7 @@ namespace xcpp
                 kernel_res["evalue"] = "evalue";
                 kernel_res["traceback"] = nl::json::array();
             }
+#endif
         }
 
         [[nodiscard]] std::unique_ptr<xpreamble> clone() const override

test/test_interpreter.cpp

@@ -657,12 +657,20 @@ TEST_SUITE("xsystem_clone")
 TEST_SUITE("xsystem_apply")
 {
 #if defined(XEUS_CPP_EMSCRIPTEN_WASM_BUILD)
-    TEST_CASE("apply_xsystem"
-            * doctest::should_fail(true)
-            * doctest::description("TODO: Currently fails for the Emscripten build"))
+    TEST_CASE("apply_xsystem")
+    {
+        xcpp::xsystem system;
+        std::string code = "!echo Hello, World!";
+        nl::json kernel_res;
+
+        system.apply(code, kernel_res);
+
+        REQUIRE(kernel_res["status"] == "error");
+        REQUIRE(kernel_res["ename"] == "UnsupportedEnvironment");
+        REQUIRE(kernel_res["evalue"] == "Shell commands are not supported in the WASM environment.");
+    }
 #else
     TEST_CASE("apply_xsystem")
-#endif
     {
         xcpp::xsystem system;
         std::string code = "!echo Hello, World!";
@@ -672,6 +680,7 @@ TEST_SUITE("xsystem_apply")
 
         REQUIRE(kernel_res["status"] == "ok");
     }
+#endif
 }
 
 TEST_SUITE("xmagics_contains"){