Changeset 9541

Timestamp:

Mar 10, 2013, 11:58:49 AM (11 years ago)

Author:

landauf

Message:

added more tests

Location:

code/branches/testing

Files:

• src/libraries/util/output/OutputStream.h (modified) (1 diff)
• test/util/output/OutputDefinitionsTest.cc (modified) (1 diff)
• test/util/output/OutputStreamTest.cc (modified) (1 diff)

code/branches/testing/src/libraries/util/output/OutputStream.h

@@ -104 +104 @@
             inline const OutputLevel getOutputLevel() const { return this->level_; }
             inline const OutputContextContainer* getOutputContext() const { return this->context_; }
+            inline bool acceptsOutput() const { return this->bAcceptsOutput_; }
 
         private:

code/branches/testing/test/util/output/OutputDefinitionsTest.cc

@@ -8 +8 @@
         EXPECT_EQ(0x0000, level::none);
     }
+
+    TEST(OutputDefinitionsTest, OutputContextContainer)
+    {
+        OutputContextContainer container1;
+        container1.mask = 1;
+        container1.sub_id = 1;
+        container1.name = "1";
+
+        OutputContextContainer container2;
+        container2.mask = 1;
+        container2.sub_id = 1;
+        container2.name = "1";
+
+        EXPECT_TRUE(container1 == container2);
+
+        {
+            OutputContextContainer container3(container2);
+            EXPECT_TRUE(container1 == container3);
+            container3.mask = 2;
+            EXPECT_FALSE(container1 == container3);
+        }
+
+        {
+            OutputContextContainer container3(container2);
+            EXPECT_TRUE(container1 == container3);
+            container3.sub_id = 2;
+            EXPECT_FALSE(container1 == container3);
+        }
+
+        {
+            OutputContextContainer container3(container2);
+            EXPECT_TRUE(container1 == container3);
+            container3.name = "2";
+            EXPECT_FALSE(container1 == container3);
+        }
+    }
 }

code/branches/testing/test/util/output/OutputStreamTest.cc

@@ -1 +1 @@
 #include <gtest/gtest.h>
+#include <gmock/gmock.h>
 #include "util/Output.h"
+#include "util/output/OutputStream.h"
+#include "util/output/OutputManager.h"
+#include "util/output/MemoryWriter.h"
+#include "util/SharedPtr.h"
 
 namespace orxonox
 {
+    namespace context
+    {
+        namespace
+        {
+            REGISTER_OUTPUT_CONTEXT(unittest1);
+        }
+    }
+
+    namespace
+    {
+        class MockOutputManager : public OutputManager
+        {
+            public:
+                MOCK_METHOD3(pushMessage, void(OutputLevel, const OutputContextContainer&, const std::string&));
+        };
+    }
+
+    // test level and context in constructor
+    TEST(OutputStreamTest, Constructor)
+    {
+        OutputStream stream(level::user_warning, context::unittest1());
+
+        EXPECT_EQ(level::user_warning, stream.getOutputLevel());
+        EXPECT_EQ(&context::unittest1(), stream.getOutputContext());
+    }
+
+    // test setOutputAttributes
+    TEST(OutputStreamTest, SetOutputAttributes)
+    {
+        OutputStream stream;
+        stream.setOutputAttributes(level::user_warning, context::unittest1());
+
+        EXPECT_EQ(level::user_warning, stream.getOutputLevel());
+        EXPECT_EQ(&context::unittest1(), stream.getOutputContext());
+    }
+
+    // test manipulator applied
+    TEST(OutputStreamTest, ManipulatorApplied)
+    {
+        OutputStream stream;
+        stream << "test";
+        EXPECT_EQ(4u, stream.str().size());
+
+        // apply manipulator
+        stream << std::ends;
+        EXPECT_EQ(5u, stream.str().size());
+    }
+
+    // test manipulator only applied if output accepted
+    TEST(OutputStreamTest, ManipulatorOnlyAppliedIfOutputAccepted)
+    {
+        // disable MemoryWriter because otherwise we would always accept all output
+        OutputManager::getInstance().getMemoryWriter().disable();
+
+        {
+            OutputStream stream;
+            EXPECT_TRUE(stream.acceptsOutput());
+            stream << std::ends;
+            EXPECT_EQ(1u, stream.str().size());
+        }
+
+        {
+            OutputStream stream(level::verbose, context::undefined());
+            EXPECT_FALSE(stream.acceptsOutput());
+            stream << std::ends;
+            EXPECT_EQ(0u, stream.str().size());
+        }
+    }
+
+    // Fixture
+    class OutputStreamTestWithMockedOutputManager : public ::testing::Test
+    {
+        public:
+            virtual void SetUp()
+            {
+                this->manager_ = new MockOutputManager();
+                OutputManager::Testing::getInstancePointer() = this->manager_;
+            }
+
+            virtual void TearDown()
+            {
+                OutputManager::Testing::getInstancePointer() = new OutputManager();
+            }
+
+        protected:
+            MockOutputManager* manager_;
+    };
+
+    // test endl-manipulator sends output to OutputManager
+    TEST_F(OutputStreamTestWithMockedOutputManager, EndlSendsToOutputManager)
+    {
+        OutputStream stream;
+
+        const OutputContextContainer& context = context::unittest1();
+
+        stream.setOutputAttributes(level::user_warning, context);
+        EXPECT_TRUE(stream.acceptsOutput());
+        EXPECT_EQ(context, *stream.getOutputContext());
+
+        EXPECT_CALL(*this->manager_, pushMessage(level::user_warning, context, "some output"));
+
+        stream << "some output" << std::endl;
+    }
+
+    // test multiple lines sent to OutputManager
+    TEST_F(OutputStreamTestWithMockedOutputManager, MultipleLinesSentToOutputManager)
+    {
+        OutputStream stream;
+
+        const OutputContextContainer& context = context::unittest1();
+
+        stream.setOutputAttributes(level::user_warning, context);
+        EXPECT_TRUE(stream.acceptsOutput());
+        EXPECT_EQ(context, *stream.getOutputContext());
+
+        EXPECT_CALL(*this->manager_, pushMessage(level::user_warning, context, "some output"));
+        EXPECT_CALL(*this->manager_, pushMessage(level::user_warning, context, "more output"));
+        EXPECT_CALL(*this->manager_, pushMessage(level::user_warning, context, "yet another line of output"));
+
+        stream << "some output" << std::endl;
+        stream << "more output" << std::endl;
+        stream << "yet another line of output" << std::endl;
+    }
+
+    // test output only sent if accepted
+    TEST_F(OutputStreamTestWithMockedOutputManager, OutputOnlySentToOutputManagerIfAccepted)
+    {
+        // disable MemoryWriter because otherwise we would always accept all output
+        OutputManager::getInstanceAndCreateListeners().getMemoryWriter().disable();
+
+        OutputStream stream;
+
+        const OutputContextContainer& context = context::undefined();
+
+        stream.setOutputAttributes(level::verbose, context);
+        EXPECT_FALSE(stream.acceptsOutput());
+
+        EXPECT_CALL(*this->manager_, pushMessage(::testing::_, ::testing::_, ::testing::_)).Times(0);
+
+        stream << "some output" << std::endl;
+    }
+
+    // test desctructor sends remaining output to OutputManager
+    TEST_F(OutputStreamTestWithMockedOutputManager, DestructorSendsRemainingOutputToOutputManager)
+    {
+        OutputStream stream;
+
+        const OutputContextContainer& context = context::undefined();
+
+        stream.setOutputAttributes(level::verbose, context);
+        EXPECT_TRUE(stream.acceptsOutput());
+
+        EXPECT_CALL(*this->manager_, pushMessage(::testing::_, ::testing::_, "some output [missing endl]"));
+
+        stream << "some output";
+    }
 }