tomoaki/hakorune
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Complete SocketBox deadlock fix: Clean up debug logging and add comprehensive tests

Browse Source 
Co-authored-by: moe-charm <217100418+moe-charm@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2025-08-14 12:29:38 +00:00
parent 06694296a7
commit 5ada387c73
3 changed files with 60 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/interpreter/methods/http_methods.rs
View File

@ -17,24 +17,15 @@ impl NyashInterpreter {
) -> Result<Box<dyn NyashBox>, RuntimeError> {
match method {
"bind" => {
eprintln!("🔥 SOCKET_METHOD: bind() called");
if arguments.len() != 2 {
return Err(RuntimeError::InvalidOperation {
message: format!("bind() expects 2 arguments, got {}", arguments.len()),
});
}
eprintln!("🔥 SOCKET_METHOD: Evaluating address argument...");
let address = self.execute_expression(&arguments[0])?;
eprintln!("🔥 SOCKET_METHOD: Address evaluated: {}", address.to_string_box().value);
eprintln!("🔥 SOCKET_METHOD: Evaluating port argument...");
let port = self.execute_expression(&arguments[1])?;
eprintln!("🔥 SOCKET_METHOD: Port evaluated: {}", port.to_string_box().value);
eprintln!("🔥 SOCKET_METHOD: Calling socket_box.bind()...");
let result = socket_box.bind(address, port);
eprintln!("🔥 SOCKET_METHOD: bind() completed");
Ok(result)
}
"listen" => {

1
test_other_boxes_quick.nyash Normal file
View File

@ -0,0 +1 @@
static box Main { init { console } main() { me.console = new ConsoleBox() local str = new StringBox("test") me.console.log("StringBox: " + str.toString()) local num = new IntegerBox(42) me.console.log("IntegerBox: " + num.toString()) return "OTHER_BOXES_OK" } }

59
test_socketbox_comprehensive.nyash Normal file
View File

@ -0,0 +1,59 @@
# test_socketbox_comprehensive.nyash
# 🎯 Comprehensive test to verify SocketBox deadlock fix and functionality
static box Main {
init { console, results }
main() {
me.console = new ConsoleBox()
me.console.log("🎯 SocketBox Comprehensive Functionality Test")
me.results = new StringBox("Test Results: ")
# Test 1: Basic creation and method calls
me.console.log("Test 1: Basic SocketBox operations...")
local socket = new SocketBox()
local initial_str = socket.toString()
local initial_server = socket.isServer()
me.results = new StringBox(me.results.toString() + "create:OK ")
me.console.log("✅ Creation and basic methods: " + initial_server.toString())
# Test 2: Bind operation and state change
me.console.log("Test 2: Bind operation...")
local bind_result = socket.bind("127.0.0.1", 17777)
local after_bind_server = socket.isServer()
me.results = new StringBox(me.results.toString() + "bind:OK ")
me.console.log("✅ Bind result: " + bind_result.toString())
# Test 3: Multiple method calls on same socket
me.console.log("Test 3: Multiple operations...")
local str1 = socket.toString()
local str2 = socket.toString()
local server1 = socket.isServer()
local server2 = socket.isServer()
me.results = new StringBox(me.results.toString() + "multiple:OK ")
me.console.log("✅ Multiple calls completed")
# Test 4: Close operation
me.console.log("Test 4: Close operation...")
local close_result = socket.close()
local after_close_str = socket.toString()
me.results = new StringBox(me.results.toString() + "close:OK ")
me.console.log("✅ Close result: " + close_result.toString())
# Test 5: Create new socket after close
me.console.log("Test 5: New socket creation...")
local socket2 = new SocketBox()
local new_str = socket2.toString()
local new_server = socket2.isServer()
me.results = new StringBox(me.results.toString() + "new:OK ")
me.console.log("✅ New socket created successfully")
me.console.log("🎉 All tests passed! " + me.results.toString())
return "COMPREHENSIVE_SUCCESS"
}
}