From 1c740fa5431e4ef54ea4f6d6ea96c55695f026ba Mon Sep 17 00:00:00 2001
From: Emmanuel Milou <emmanuel.milou@savoirfairelinux.com>
Date: Thu, 9 Oct 2008 13:27:17 -0400
Subject: [PATCH] commented out the configuration test class

---
 test/configurationTest.cpp | 50 ++++++++++++++++++++++++++++++++++++--
 1 file changed, 48 insertions(+), 2 deletions(-)

diff --git a/test/configurationTest.cpp b/test/configurationTest.cpp
index d3bbfa5919..ba7349ecd1 100644
--- a/test/configurationTest.cpp
+++ b/test/configurationTest.cpp
@@ -1,22 +1,51 @@
+/*
+ *  Copyright (C) 2008 Savoir-Faire Linux inc.
+ *  Author: Emmanuel Milou <emmanuel.milou@savoirfairelinux.com>
+ *
+ *  This program is free software; you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation; either version 3 of the License, or
+ *  (at your option) any later version.
+ *
+ *  This program is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with this program; if not, write to the Free Software
+ *   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ */
+
+// Cppunit import
 #include <cppunit/extensions/HelperMacros.h>
 #include <cppunit/TestCaller.h>
 #include <cppunit/TestCase.h>
 #include <cppunit/TestSuite.h>
+
 #include <assert.h>
+#include <stdio.h>
 
+// Application import
 #include "manager.h"
 #include "global.h"
 #include "user_cfg.h"
 
-#include <stdio.h>
+/*
+ * @file configurationTest.cpp  
+ * @brief       Regroups unitary tests related to the user configuration.
+ *              Check if the default configuration has been successfully loaded
+ */
 
 using std::cout;
 using std::endl;
 
-// Cppunit import
 
 class ConfigurationTest : public CppUnit::TestCase {
 
+    /*
+     * Use cppunit library macros to add unit test the factory
+     */
     CPPUNIT_TEST_SUITE( ConfigurationTest );
         CPPUNIT_TEST( testDefaultValueAudio );
         CPPUNIT_TEST( testDefaultValuePreferences );
@@ -25,6 +54,9 @@ class ConfigurationTest : public CppUnit::TestCase {
     public:
         ConfigurationTest() : CppUnit::TestCase("Configuration Tests") {}
  
+        /*
+         * Unit tests related to the audio preferences
+         */
         void testDefaultValueAudio(){
             CPPUNIT_ASSERT( Manager::instance().getConfigString( AUDIO, ALSA_CARD_ID_IN ) == ALSA_DFT_CARD) ;
             CPPUNIT_ASSERT( Manager::instance().getConfigString( AUDIO, ALSA_CARD_ID_OUT ) == ALSA_DFT_CARD );
@@ -35,6 +67,9 @@ class ConfigurationTest : public CppUnit::TestCase {
             CPPUNIT_ASSERT( Manager::instance().getConfigString( AUDIO, VOLUME_MICRO ) == DFT_VOL_MICRO_STR);
         }
 
+        /*
+         * Unit tests related to the global settings
+         */
         void testDefaultValuePreferences(){
             CPPUNIT_ASSERT( Manager::instance().getConfigString( PREFERENCES, ZONE_TONE ) == DFT_ZONE );
             CPPUNIT_ASSERT( Manager::instance().getConfigString( PREFERENCES, CONFIG_ZEROCONF ) == CONFIG_ZEROCONF_DEFAULT_STR );
@@ -51,13 +86,24 @@ class ConfigurationTest : public CppUnit::TestCase {
    
         }
 
+        /*
+         * Code factoring - Common resources can be initialized here.
+         * This method is called by unitcpp before each test
+         */
         void setUp(){
+            // Load the default configuration
             Manager::instance().initConfigFile(false);
         }
 
+        /*
+         * Code factoring - Common resources can be released here.
+         * This method is called by unitcpp after each test
+         */
         void tearDown(){
+            // Not much to do
         }
 
 };
 
+/* Register our test module */
 CPPUNIT_TEST_SUITE_REGISTRATION( ConfigurationTest );
-- 
GitLab