From 8dd200a35ee9985b49e0d5fea2006a4446eea1fc Mon Sep 17 00:00:00 2001 From: Julien Letessier Date: Thu, 15 Jan 2026 09:15:59 +0100 Subject: [PATCH] Clean up pubsub response handling Remove stray statements introduced during staging and normalize test runner order. --- src/ConnectConnection.cpp | 8 ++++---- test/run.sh | 12 ++++++------ 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/ConnectConnection.cpp b/src/ConnectConnection.cpp index b5665c6..5a975d1 100644 --- a/src/ConnectConnection.cpp +++ b/src/ConnectConnection.cpp @@ -191,13 +191,13 @@ void ConnectConnection::handleResponse(Handler* h) mAcceptConnection->append(req); if (mAcceptConnection->inPendSub()) { mParser.reset(); -mParser.reset(); -h->handleResponse(this, req, res); -mSentRequests.pop_front(); -return; return; } mSentRequests.push_front(req); + mParser.reset(); + h->handleResponse(this, req, res); + mSentRequests.pop_front(); + return; } break; default: diff --git a/test/run.sh b/test/run.sh index 12c397b..969ed09 100755 --- a/test/run.sh +++ b/test/run.sh @@ -73,15 +73,15 @@ uv run python3 test/basic.py || BASIC_EXIT=$? uv run python3 test/pubsub_minimal.py -p 7617 || PUBSUB_REDIS_EXIT=$? uv run python3 test/pubsub_minimal.py -p 6379 || PUBSUB_MINIMAL_EXIT=$? uv run python3 test/pubsub.py || PUBSUB_EXIT=$? +uv run python3 test/pubsub_message_response.py -p 7617 || PUBSUB_MESSAGE_EXIT=$? uv run python3 test/pubsub_subscription_order.py -p 7617 || PUBSUB_ORDER_EXIT=$? uv run python3 test/pubsub_parser_reset.py -p 7617 || PUBSUB_RESET_EXIT=$? -uv run python3 test/eval_cross_shard.py -p 7617 || EVAL_CROSS_SHARD_EXIT=$? -uv run python3 test/msetnx_atomicity.py -p 7617 || MSETNX_ATOMICITY_EXIT=$? -uv run python3 test/mget_wrong_type.py -p 7617 || MGET_WRONG_TYPE_EXIT=$? -uv run python3 test/transaction_forbid.py -p 7617 || TRANSACTION_FORBID_EXIT=$? -uv run python3 test/pubsub_long_name.py -p 7617 || PUBSUB_LONG_EXIT=$? uv run python3 test/null_response_handling.py -p 7617 || NULL_RESPONSE_EXIT=$? -uv run python3 test/pubsub_message_response.py -p 7617 || PUBSUB_MESSAGE_EXIT=$? +uv run python3 test/pubsub_long_name.py -p 7617 || PUBSUB_LONG_EXIT=$? +uv run python3 test/transaction_forbid.py -p 7617 || TRANSACTION_FORBID_EXIT=$? +uv run python3 test/mget_wrong_type.py -p 7617 || MGET_WRONG_TYPE_EXIT=$? +uv run python3 test/msetnx_atomicity.py -p 7617 || MSETNX_ATOMICITY_EXIT=$? +uv run python3 test/eval_cross_shard.py -p 7617 || EVAL_CROSS_SHARD_EXIT=$? TEST_EXIT=$((BASIC_EXIT + PUBSUB_REDIS_EXIT + PUBSUB_MINIMAL_EXIT + PUBSUB_EXIT + PUBSUB_MESSAGE_EXIT + PUBSUB_ORDER_EXIT + PUBSUB_RESET_EXIT + NULL_RESPONSE_EXIT + PUBSUB_LONG_EXIT + TRANSACTION_FORBID_EXIT + MGET_WRONG_TYPE_EXIT + MSETNX_ATOMICITY_EXIT + EVAL_CROSS_SHARD_EXIT)) exit $TEST_EXIT