diff options
author | Alan Conway <aconway@apache.org> | 2007-08-31 20:51:22 +0000 |
---|---|---|
committer | Alan Conway <aconway@apache.org> | 2007-08-31 20:51:22 +0000 |
commit | 761e10501fe5ea51f9d8c40d9a200ae27193ab23 (patch) | |
tree | e2d4bdfdc0b9383661947378a1f183387501637c /cpp/src/tests/BrokerChannelTest.cpp | |
parent | 655b3b5806bafdd784f6a9c242e26341bd6aeccc (diff) | |
download | qpid-python-761e10501fe5ea51f9d8c40d9a200ae27193ab23.tar.gz |
* Summary:
- Moved BrokerChannel functionality into Session.
- Moved ChannelHandler methods handling into SessionAdapter.
- Updated all handlers to use session.
(We're still using AMQP channel methods in SessionAdapter)
Roles & responsibilities:
Session:
- represents an _open_ session, may be active or suspended.
- ows all session state including handler chains.
- attahced to SessionAdapter when active, not when suspended.
SessionAdapter:
- reprents the association of a channel with a session.
- owned by Connection, kept in the session map.
- channel open == SessionAdapter.getSessio() != 0
Anything that depends on attachment to a channel, connection or
protocol should be in SessionAdpater. Anything that suvives a
session suspend belongs in Session.
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@571575 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'cpp/src/tests/BrokerChannelTest.cpp')
-rw-r--r-- | cpp/src/tests/BrokerChannelTest.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/cpp/src/tests/BrokerChannelTest.cpp b/cpp/src/tests/BrokerChannelTest.cpp index 0787405eb7..e975ec1b12 100644 --- a/cpp/src/tests/BrokerChannelTest.cpp +++ b/cpp/src/tests/BrokerChannelTest.cpp @@ -18,6 +18,11 @@ * under the License. * */ + +// FIXME aconway 2007-08-30: Rewrite as a Session test. +// There is an issue with the tests use of DeliveryAdapter +// which is no longer exposed on Session (part of SemanticHandler.) +// #include "qpid/broker/BrokerChannel.h" #include "qpid/broker/BrokerQueue.h" #include "qpid/broker/FanOutExchange.h" |