Changeset 9535

Timestamp:

Mar 4, 2013, 10:24:30 PM (11 years ago)

Author:

landauf

Message:

more tests

Location:

code/branches/testing

Files:

• src/libraries/util/output/ConsoleWriter.h (modified) (2 diffs)
• src/libraries/util/output/LogWriter.h (modified) (2 diffs)
• src/libraries/util/output/MemoryWriter.h (modified) (2 diffs)
• src/libraries/util/output/OutputDefinitions.h (modified) (1 diff)
• src/libraries/util/output/OutputManager.h (modified) (1 diff)
• test/util/output/ConsoleWriterTest.cc (modified) (1 diff)
• test/util/output/MemoryWriterTest.cc (modified) (1 diff)
• test/util/output/OutputListenerTest.cc (modified) (2 diffs)

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

@@ -50 +50 @@
     {
         public:
+            ConsoleWriter();
+            ConsoleWriter(const ConsoleWriter&);
+            virtual ~ConsoleWriter();
+
             static ConsoleWriter& getInstance();
 
@@ -59 +63 @@
 
         private:
-            ConsoleWriter();
-            ConsoleWriter(const ConsoleWriter&);
-            virtual ~ConsoleWriter();
-
             bool bEnabled_; ///< If false, the instance will not write output to the console.
     };

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

@@ -57 +57 @@
     {
         public:
+            LogWriter();
+            LogWriter(const LogWriter&);
+            virtual ~LogWriter();
+
             static LogWriter& getInstance();
 
@@ -65 +69 @@
 
         private:
-            LogWriter();
-            LogWriter(const LogWriter&);
-            virtual ~LogWriter();
-
             void openFile();
             void closeFile();

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

@@ -67 +67 @@
 
         public:
+            MemoryWriter();
+            MemoryWriter(const MemoryWriter&);
+            virtual ~MemoryWriter();
+
             static MemoryWriter& getInstance();
 
@@ -76 +80 @@
 
         private:
-            MemoryWriter();
-            MemoryWriter(const MemoryWriter&);
-            virtual ~MemoryWriter();
-
             std::vector<Message> messages_; ///< Stores all output messages from the creation of this instance until disable() is called.
     };

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

@@ -105 +105 @@
         OutputContextSubID sub_id;  ///< The id of the sub-context (or context::no_subcontext if this container doesn't define a sub-context)
         std::string name;           ///< The name of this context
+
+        inline bool operator==(const OutputContextContainer& other) const
+        {
+            return this->mask == other.mask && this->sub_id == other.sub_id && this->name == other.name;
+        }
     };
 

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

@@ -102 +102 @@
             std::string getDefaultPrefix(OutputLevel level, const OutputContextContainer& context) const;
 
-        protected:
             inline const std::vector<OutputListener*>& getListeners() const
                 { return this->listeners_; }

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

@@ -1 +1 @@
 #include <gtest/gtest.h>
 #include "util/Output.h"
+#include "util/output/ConsoleWriter.h"
+#include "util/output/OutputManager.h"
 
 namespace orxonox
 {
+    TEST(ConsoleWriterTest, Disable)
+    {
+        EXPECT_EQ(0U, OutputManager::getInstance().getListeners().size());
+        ConsoleWriter writer;
+        EXPECT_EQ(1U, OutputManager::getInstance().getListeners().size());
+        writer.disable();
+        EXPECT_EQ(0U, OutputManager::getInstance().getListeners().size());
+    }
+
+    TEST(ConsoleWriterTest, Enable)
+    {
+        ConsoleWriter writer;
+        writer.disable();
+        EXPECT_EQ(0U, OutputManager::getInstance().getListeners().size());
+        writer.enable();
+        EXPECT_EQ(1U, OutputManager::getInstance().getListeners().size());
+    }
 }

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

@@ -1 +1 @@
 #include <gtest/gtest.h>
+#include <gmock/gmock.h>
 #include "util/Output.h"
+#include "util/output/MemoryWriter.h"
+#include "util/output/OutputManager.h"
 
 namespace orxonox
 {
+    namespace
+    {
+        class MockOutputListener : public OutputListener
+        {
+            public:
+                MOCK_METHOD3(output, void(OutputLevel, const OutputContextContainer&, const std::vector<std::string>&));
+        };
+    }
+
+    TEST(MemoryWriterTest, Disable)
+    {
+        EXPECT_EQ(0U, OutputManager::getInstance().getListeners().size());
+        MemoryWriter writer;
+        EXPECT_EQ(1U, OutputManager::getInstance().getListeners().size());
+        writer.disable();
+        EXPECT_EQ(0U, OutputManager::getInstance().getListeners().size());
+    }
+
+    TEST(MemoryWriterTest, ResendOutput)
+    {
+        MemoryWriter writer;
+
+        std::vector<std::string> lines;
+        lines.push_back("random line of output");
+        lines.push_back("another line of output");
+
+        writer.unfilteredOutput(level::user_info, context::undefined(), lines);
+        writer.unfilteredOutput(level::verbose, context::xml(), lines);
+
+        MockOutputListener other;
+        other.setLevelMask(level::all);
+
+        EXPECT_CALL(other, output(level::user_info, context::undefined(), lines));
+        EXPECT_CALL(other, output(level::verbose, context::xml(), lines));
+
+        writer.resendOutput(&other);
+    }
 }

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

@@ -313 +313 @@
 
         std::vector<std::string> lines;
-        EXPECT_CALL(listener, output(level, ::testing::_, lines)).Times(1);
+        EXPECT_CALL(listener, output(level, context, lines)).Times(1);
 
         listener.unfilteredOutput(level, context, lines);
@@ -330 +330 @@
 
         std::vector<std::string> lines;
-        EXPECT_CALL(listener, output(level, ::testing::_, lines)).Times(0);
+        EXPECT_CALL(listener, output(level, context, lines)).Times(0);
 
         listener.unfilteredOutput(level, context, lines);