D11034: CoreApplicationSettingsTest

[Himanshu Vishwakarma]

himanshuvishwakarma updated this revision to Diff 30019.
himanshuvishwakarma added a comment.


  Is this possible solution can be acceptable ??
  
  I tried your solution i.e. add a protected constructor taking the path as parameter and use it in to instantiate the inherited class
  But adding a constructor give the ambiguity error in this line https://github.com/gcompris/GCompris-qt/blob/master/src/core/ApplicationSettings.h#L263

REPOSITORY
  R2 GCompris

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D11034?vs=29685&id=30019

REVISION DETAIL
  https://phabricator.kde.org/D11034

AFFECTED FILES
  src/core/ApplicationSettings.cpp
  src/core/ApplicationSettings.h
  tests/core/ApplicationSettingsTest.cpp
  tests/core/CMakeLists.txt

To: himanshuvishwakarma, jjazeix, dmadaan, rudranilbasu, timotheegiet, #gcompris
Cc: #kde_edu, #gcompris, harrymecwan, ganeshredcobra, nityanandkumar, echarruau, rahulyadav, narvaez, scagarwal, apol, timotheegiet, hkaelberer, jjazeix, bcoudoin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180320/975ba1fd/attachment.html>