use QEXPECT_FAIL instead of disabling the test

Author: jef-n

src/core/symbology-ng/qgsstylev2.cpp

@@ -336,7 +336,7 @@ bool QgsStyleV2::save( QString filename )
     filename = mFileName;
 
   // TODO evaluate the requirement of this function and change implementation accordingly
-  // TODO enable save TestStyleV2::testSaveLoad()
+  // TODO remove QEXPECT_FAIL from TestStyleV2::testSaveLoad() when done
 #if 0
   QDomDocument doc( "qgis_style" );
   QDomElement root = doc.createElement( "qgis_style" );

tests/src/core/testqgsstylev2.cpp

@@ -188,7 +188,8 @@ void TestStyleV2::testLoadColorRamps()
 
 void TestStyleV2::testSaveLoad()
 {
-#if 0
+  QEXPECT_FAIL("", "save() currently disabled in core", Abort);
+
   mStyle->save();
   mStyle->clear();
   mStyle->load( QgsApplication::userStyleV2Path() );
@@ -205,7 +206,6 @@ void TestStyleV2::testSaveLoad()
     if ( ramp )
       delete ramp;
   }
-#endif
 }