--- a/mpviewplugins/mpsettingsviewplugin/tsrc/unittest_mpsettingsaudioeffectswidget/src/unittest_mpsettingsaudioeffectswidget.cpp	Fri Jul 23 17:31:12 2010 -0500
+++ b/mpviewplugins/mpsettingsviewplugin/tsrc/unittest_mpsettingsaudioeffectswidget/src/unittest_mpsettingsaudioeffectswidget.cpp	Fri Aug 06 16:51:36 2010 -0500
@@ -29,11 +29,6 @@
 #include "MpSettingsAudioEffectsWidget.h"
 #undef private
 
-//This so we can test private functions
-//#include "../../src/mpsettingsaudioeffectswidget.cpp"
-
-
-
 /*!
  Make our test case a stand-alone executable that runs all the test functions.
  */
@@ -110,10 +105,11 @@
 }
 
 /*!
- Tests constructor.
+ Tests InitializeSlider.
  */
 void TestMpSettingsAudioEffectsWidget::testInitializeSlider()
 {
+    //test slider widget is set up correctly in initialize()
     mTest->initialize();
     QVERIFY(mTest->mMpEngine != 0 );
 
@@ -122,7 +118,6 @@
     QCOMPARE(mTest->mModel->rowCount(),2 );
     QCOMPARE(mTest->mModel->columnCount(),1 );
 
-
     QCOMPARE(  mTest->mModel->item( 0 )->type(), HbDataFormModelItem::SliderItem );
     QCOMPARE(  mTest->mModel->item( 1 )->type(), HbDataFormModelItem::ToggleValueItem );
 
@@ -140,8 +135,12 @@
 
 }
 
+/*!
+ Tests InitializeLoudnessOff.
+ */
 void TestMpSettingsAudioEffectsWidget::testInitializeLoudnessOff()
 {
+    //test loudness HbDataFormModelItem is set up correctly in initialize()
     MpEngineFactory::sharedEngine()->setLoudness( false );
     mTest->initialize();
     QVERIFY(  mTest->mModel->item( 1 )->type() == HbDataFormModelItem::ToggleValueItem );
@@ -149,8 +148,12 @@
     QCOMPARE(  mTest->mModel->item( 1 )->contentWidgetData("additionalText"), QVariant(hbTrId( "txt_mus_setlabel_loudness_val_on" )) );
 }
 
+/*!
+ Tests InitializeLoudnessOn.
+ */
 void TestMpSettingsAudioEffectsWidget::testInitializeLoudnessOn()
 {
+    //test loudness HbDataFormModelItem is set up correctly in initialize()
     MpEngineFactory::sharedEngine()->setLoudness( true );
     mTest->initialize();
     QVERIFY(  mTest->mModel->item( 1 )->type() == HbDataFormModelItem::ToggleValueItem );
@@ -159,17 +162,21 @@
 
 }
 
-
+/*!
+ Tests FilterSignals.
+ */
 void TestMpSettingsAudioEffectsWidget::testFilterSignals()
-{
+{   
+    //filtersignals should not change loudness value
     MpEngineFactory::sharedEngine()->setLoudness( false );
-    mTest->initialize();
-    QCOMPARE(MpEngineFactory::sharedEngine()->loudness(), false);
+    mTest->initialize();    
     QModelIndex index1 = mTest->mModel->indexFromItem((mTest->mModel->item(1)));
     mTest->filterSignals(index1,index1);
-    QCOMPARE(MpEngineFactory::sharedEngine()->loudness(), true);
+    QCOMPARE(MpEngineFactory::sharedEngine()->loudness(), false);
+   
 }
 
 
 
+
 // End of file