X-Git-Url: http://g0dil.de/git?a=blobdiff_plain;f=PPI%2FConnectors.test.cc;h=20b7e3e405b21d715d3b8197b35e06fc44c188f9;hb=b89e3166f7680755683dccee5e48cb3a820185c0;hp=694697b0016c00fba7c1d6faf2e90b105ef91860;hpb=b356790e396900fd46a677b39f4e182822bb5c1f;p=senf.git

diff --git a/PPI/Connectors.test.cc b/PPI/Connectors.test.cc
index 694697b..20b7e3e 100644
--- a/PPI/Connectors.test.cc
+++ b/PPI/Connectors.test.cc
@@ -1,8 +1,8 @@
 // $Id$
 //
-// Copyright (C) 2007 
-// Fraunhofer Institut fuer offene Kommunikationssysteme (FOKUS)
-// Kompetenzzentrum fuer Satelitenkommunikation (SatCom)
+// Copyright (C) 2007
+// Fraunhofer Institute for Open Communication Systems (FOKUS)
+// Competence Center NETwork research (NET), St. Augustin, GERMANY
 //     Stefan Bund <g0dil@berlios.de>
 //
 // This program is free software; you can redistribute it and/or modify
@@ -21,7 +21,7 @@
 // 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 
 /** \file
-    \brief Connectors.test unit tests */
+    \brief Connectors unit tests */
 
 //#include "Connectors.test.hh"
 //#include "Connectors.test.ih"
@@ -31,14 +31,14 @@
 #include "DebugModules.hh"
 #include "Setup.hh"
 
-#include <boost/test/auto_unit_test.hpp>
+#include "../Utils/auto_unit_test.hh"
 #include <boost/test/test_tools.hpp>
 
 #define prefix_
 ///////////////////////////////cc.p////////////////////////////////////////
 
-namespace debug = senf::ppi::module::debug;
 namespace ppi = senf::ppi;
+namespace debug = ppi::module::debug;
 
 // For each type of connector we use the corresponding debug module. Additionally, we always need
 // the corresponding connected module since otherwise the connectors cannot be connected anywhere
@@ -49,10 +49,11 @@ BOOST_AUTO_UNIT_TEST(connector)
     // It doesn't matter, which type of connectors we use here since they are all based on
     // Connector.
 
-    debug::ActivePacketSource source;
-    debug::PassivePacketSink target;
+    debug::ActiveSource source;
+    debug::PassiveSink target;
 
     ppi::connect(source.output,target.input);
+    ppi::init();
 
     BOOST_CHECK_EQUAL( & source.output.module(), & source );
     BOOST_CHECK_EQUAL( & target.input.module(), & target );
@@ -62,15 +63,15 @@ BOOST_AUTO_UNIT_TEST(connector)
 
 BOOST_AUTO_UNIT_TEST(passiveConnector)
 {
-    debug::ActivePacketSource source;
-    debug::PassivePacketSink target;
+    debug::ActiveSource source;
+    debug::PassiveSink target;
 
     ppi::connect(source.output,target.input);
+    ppi::init();
 
-    // onRequest is implicitly tested within the PassivePacketSink implementation which is tested
+    // onRequest is implicitly tested within the PassiveSink implementation which is tested
     // in DebugModules.test.cc
 
-#if 0
     target.input.throttle();
     BOOST_CHECK( target.input.throttled() );
     BOOST_CHECK( target.input.nativeThrottled() );
@@ -78,7 +79,463 @@ BOOST_AUTO_UNIT_TEST(passiveConnector)
     target.input.unthrottle();
     BOOST_CHECK( ! target.input.throttled() );
     BOOST_CHECK( ! target.input.nativeThrottled() );
-#endif
+
+    BOOST_CHECK_EQUAL( & target.input.peer(), & source.output );
+}
+
+namespace {
+    
+    bool called = false;
+    
+    void handler() { called = true; }
+}
+
+BOOST_AUTO_UNIT_TEST(activeConnector)
+{
+    debug::ActiveSource source;
+    debug::PassiveSink target;
+
+    ppi::connect(source.output,target.input);
+    ppi::init();
+
+    source.output.onThrottle(handler);
+    BOOST_CHECK( ! called );
+    target.input.throttle();
+    BOOST_CHECK( called );
+    called = false;
+    target.input.unthrottle();
+    BOOST_CHECK( ! called );
+    source.output.onThrottle();
+    source.output.onUnthrottle(handler);
+    BOOST_CHECK( ! called );
+    target.input.throttle();
+    BOOST_CHECK( ! called );
+    target.input.unthrottle();
+    BOOST_CHECK( called );
+    source.output.onUnthrottle();
+    called = false;
+    BOOST_CHECK( ! called );
+    target.input.throttle();
+    target.input.unthrottle();
+    BOOST_CHECK( ! called );
+
+    BOOST_CHECK_EQUAL( & source.output.peer(), & target.input );
+}
+
+BOOST_AUTO_UNIT_TEST(inputConnector)
+{
+    debug::ActiveSource source;
+    debug::PassiveSink target;
+
+    ppi::connect(source.output,target.input);
+    ppi::init();
+
+    // operator() is implicitly tested within the Active/PassiveSink implementation which is
+    // tested in DebugModules.test.cc
+
+    // peek() is implicitly tested within the Active/PassiveSink implementation
+
+    BOOST_CHECK_EQUAL ( & target.input.peer(), & source.output );
+    
+    BOOST_CHECK( target.input.begin() == target.input.end() );
+    BOOST_CHECK_EQUAL( target.input.queueSize(), 0u );
+    BOOST_CHECK( target.input.empty() );
+}
+
+BOOST_AUTO_UNIT_TEST(outputConnector)
+{
+    debug::ActiveSource source;
+    debug::PassiveSink target;
+
+    ppi::connect(source.output,target.input);
+    ppi::init();
+
+    // operator() is implicitly tested within the Active/PassiveSource implementation which is
+    // tested in DebugModules.test.cc
+
+    BOOST_CHECK_EQUAL( & source.output.peer(), & target.input );
+}
+
+namespace {
+
+    class PassiveInputTest
+        : public ppi::module::Module
+    {
+        SENF_PPI_MODULE(PassiveInputTest);
+
+    public:
+        ppi::connector::PassiveInput<> input;
+
+        PassiveInputTest() : counter() {
+            noroute(input);
+            input.onRequest(&PassiveInputTest::request);
+        }
+
+        void request() {
+            ++ counter;
+        }
+
+        unsigned counter;
+    };
+}
+
+BOOST_AUTO_UNIT_TEST(passiveInput)
+{
+    debug::ActiveSource source;
+    PassiveInputTest target;
+
+    ppi::connect(source,target);
+    ppi::init();
+
+    BOOST_CHECK_EQUAL( & target.input.peer(), & source.output );
+    
+    target.input.throttle();
+    senf::Packet p (senf::DataPacket::create());
+    source.submit(p);
+    
+    BOOST_CHECK_EQUAL( target.counter, 0u );
+    BOOST_CHECK( target.input );
+    BOOST_CHECK_EQUAL( target.input.queueSize(), 1u );
+    target.input.unthrottle();
+    BOOST_CHECK( target.input );
+    BOOST_CHECK_EQUAL( target.counter, 1u );
+    
+    BOOST_CHECK( target.input() == p );
+    BOOST_CHECK( ! target.input );
+    
+    source.submit(p);
+    
+    BOOST_CHECK_EQUAL( target.counter, 2u );
+    BOOST_CHECK( target.input.throttled() );
+    BOOST_CHECK( target.input() == p );
+    BOOST_CHECK( ! target.input.throttled() );
+
+    target.input.qdisc(ppi::ThresholdQueueing(2,0));
+
+    source.submit(p);
+    BOOST_CHECK ( ! target.input.throttled() );
+    source.submit(p);
+    BOOST_CHECK( target.input.throttled() );
+    target.input();
+    BOOST_CHECK( target.input.throttled() );
+    target.input();
+    BOOST_CHECK( ! target.input.throttled() );
+}
+
+BOOST_AUTO_UNIT_TEST(passiveOutput)
+{
+    debug::PassiveSource source;
+    debug::ActiveSink target;
+
+    ppi::connect(source,target);
+    ppi::init();
+
+    senf::Packet p (senf::DataPacket::create());
+    source.submit(p);
+
+    BOOST_CHECK_EQUAL( & source.output.peer(), & target.input );
+
+    BOOST_CHECK( source.output );
+
+    source.submit(p);
+    BOOST_CHECK( target.request() == p );
+    
+    // connect() is tested indirectly via ppi::connect
+}
+
+BOOST_AUTO_UNIT_TEST(activeInput)
+{
+    debug::PassiveSource source;
+    debug::ActiveSink target;
+
+    ppi::connect(source,target);
+    ppi::init();
+
+    BOOST_CHECK_EQUAL( & target.input.peer(), & source.output );
+
+    BOOST_CHECK ( ! target.input );
+
+    senf::Packet p (senf::DataPacket::create());
+    source.submit(p);
+
+    BOOST_CHECK( target.input );
+    BOOST_CHECK( target.request() == p );
+
+    source.submit(p);
+    target.input.request();
+    BOOST_CHECK_EQUAL( target.input.queueSize(), 1u );
+    BOOST_CHECK( target.input );
+    BOOST_CHECK( target.request() == p );
+}
+
+BOOST_AUTO_UNIT_TEST(activeOutput)
+{
+    debug::ActiveSource source;
+    debug::PassiveSink target;
+
+    ppi::connect(source,target);
+    ppi::init();
+    
+    BOOST_CHECK_EQUAL( & source.output.peer(), & target.input );
+    BOOST_CHECK( source.output );
+    target.input.throttle();
+    BOOST_CHECK( ! source.output );
+
+    // connect() is tested indirectly via ppi::connect
+}
+
+namespace {
+
+    template <class PacketType = senf::DataPacket>
+    class TypedPassiveInput
+        : public ppi::module::Module
+    {
+        SENF_PPI_MODULE(TypedPassiveInput);
+
+    public:
+        ppi::connector::PassiveInput<PacketType> input;
+
+        TypedPassiveInput() {
+            noroute(input);
+            input.onRequest(&TypedPassiveInput::request);
+        }
+
+        void request() {
+            (void) input();
+            (void) input.read();
+        }
+    };
+
+    template <class PacketType = senf::DataPacket>
+    class TypedActiveInput
+        : public ppi::module::Module
+    {
+        SENF_PPI_MODULE(TypedActiveInput);
+
+    public:
+        ppi::connector::ActiveInput<PacketType> input;
+
+        TypedActiveInput() {
+            noroute(input);
+        }
+    };
+
+    template <class PacketType = senf::DataPacket>
+    class TypedPassiveOutput
+        : public ppi::module::Module
+    {
+        SENF_PPI_MODULE(TypedPassiveOutput);
+
+    public:
+        ppi::connector::PassiveOutput<PacketType> output;
+
+        TypedPassiveOutput() {
+            noroute(output);
+            output.onRequest(&TypedPassiveOutput::request);
+        }
+
+        void request() {
+            senf::DataPacket pkg (senf::DataPacket::create());
+            output(pkg);
+            output.write(pkg);
+        }
+    };
+
+    template <class PacketType = senf::DataPacket>
+    class TypedActiveOutput
+        : public ppi::module::Module
+    {
+        SENF_PPI_MODULE(TypedActiveOutput);
+
+    public:
+        ppi::connector::ActiveOutput<PacketType> output;
+
+        TypedActiveOutput() {
+            noroute(output);
+        }
+    };
+
+    struct MyPacketType : public senf::PacketTypeBase
+    {};
+
+    typedef senf::ConcretePacket<MyPacketType> MyPacket;
+
+}
+
+BOOST_AUTO_UNIT_TEST(typedInput)
+{
+    debug::ActiveSource source;
+    TypedPassiveInput<> target;
+
+    ppi::connect(source,target);
+    ppi::init();
+
+    senf::Packet p (senf::DataPacket::create());
+    source.submit(p);
+}
+
+BOOST_AUTO_UNIT_TEST(tyepdOutput)
+{
+    TypedPassiveOutput<> source;
+    debug::ActiveSink target;
+
+    ppi::connect(source,target);
+    ppi::init();
+    
+    (void) target.request();
+}
+
+BOOST_AUTO_UNIT_TEST(connectorTest)
+{
+    {
+        TypedPassiveInput<> input;
+        TypedActiveOutput<MyPacket> output;
+        BOOST_CHECK_THROW( ppi::connect(output, input), 
+                           ppi::connector::IncompatibleConnectorsException );
+    }
+    {
+        TypedPassiveInput<MyPacket> input;
+        TypedActiveOutput<> output;
+        BOOST_CHECK_THROW( ppi::connect(output, input), 
+                           ppi::connector::IncompatibleConnectorsException );
+    }
+    {
+        TypedPassiveInput<> input;
+        TypedActiveOutput<> output;
+        SENF_CHECK_NO_THROW( ppi::connect(output, input) );
+    }
+    { 
+        TypedPassiveInput<> input;
+        debug::ActiveSource output;
+        SENF_CHECK_NO_THROW( ppi::connect(output, input) );
+    }
+    {
+        debug::ActiveSink input;
+        TypedPassiveOutput<> output;
+        SENF_CHECK_NO_THROW( ppi::connect(output, input) );
+    }
+    {
+        debug::ActiveSink input;
+        debug::PassiveSource output;
+        SENF_CHECK_NO_THROW( ppi::connect(output, input) );
+    }
+}
+
+BOOST_AUTO_UNIT_TEST(delayedConnect)
+{
+    {
+        debug::PassiveSource source;
+        debug::ActiveSink target;
+
+        ppi::init();
+
+        BOOST_CHECK( ! target.input );
+        BOOST_CHECK( ! target.request() );
+
+        ppi::connect(source, target);
+        ppi::init();
+
+        BOOST_CHECK( ! target.input );
+
+        senf::Packet p (senf::DataPacket::create());
+        source.submit(p);
+        BOOST_CHECK( target.request() == p );
+    }
+
+    {
+        debug::PassiveSource source;
+        debug::ActiveSink target;
+
+        ppi::init();
+
+        senf::Packet p (senf::DataPacket::create());
+        source.submit(p);
+
+        BOOST_CHECK( ! target.input );
+        BOOST_CHECK( ! target.request() );
+
+        ppi::connect(source, target);
+        ppi::init();
+
+        BOOST_CHECK( target.input );
+        BOOST_CHECK( target.request() == p );
+    }
+
+    {
+        debug::ActiveSource source;
+        debug::PassiveSink target;
+
+        ppi::init();
+
+        BOOST_CHECK( ! source.output );
+        SENF_CHECK_NO_THROW( source.output(senf::DataPacket::create()) );
+
+        ppi::connect(source, target);
+        ppi::init();
+        
+        BOOST_CHECK( source.output );
+
+        senf::Packet p (senf::DataPacket::create());
+        source.submit(p);
+
+        BOOST_CHECK( target.front() == p );        
+        BOOST_CHECK_EQUAL( target.size(), 1u );
+    }
+
+    {
+        debug::ActiveSource source;
+        debug::PassiveSink target;
+
+        ppi::init();
+
+        BOOST_CHECK( ! source.output );
+        SENF_CHECK_NO_THROW( source.output(senf::DataPacket::create()) );
+        target.throttle();
+
+        ppi::connect(source, target);
+        ppi::init();
+        
+        BOOST_CHECK( ! source.output );
+        target.unthrottle();
+        BOOST_CHECK( source.output );
+    }
+}
+
+BOOST_AUTO_UNIT_TEST(disconnect)
+{
+    {
+        debug::PassiveSource source;
+        debug::ActiveSink target;
+
+        ppi::connect(source, target);
+        ppi::init();
+
+        BOOST_CHECK( ! target.input );
+
+        senf::Packet p (senf::DataPacket::create());
+        source.submit(p);
+
+        BOOST_CHECK( target.input );
+        
+        target.input.disconnect();
+        ppi::init();
+        
+        BOOST_CHECK( ! target.input );
+    }
+    {
+        debug::ActiveSource source;
+        debug::PassiveSink target;
+
+        ppi::connect(source, target);
+        ppi::init();
+
+        BOOST_CHECK( source.output );
+
+        source.output.disconnect();
+        ppi::init();
+
+        BOOST_CHECK( ! source.output );
+    }
 }
 
 ///////////////////////////////cc.e////////////////////////////////////////