<table><tr><td style="">jjazeix added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11034">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D11034#inline-54613">View Inline</a><span style="color: #4b4d51; font-weight: bold;">himanshuvishwakarma</span> wrote in <span style="color: #4b4d51; font-weight: bold;">ApplicationSettingsTest.cpp:41</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Yaa, I tried both methods but both of them compelled to change in class ApplicationSettings. which we have to test. So I find a nice approach by this our default configuration is not going to change.</p>

<p style="padding: 0; margin: 8px;">In this method we are first saving the default data(which is already present in the config file ), then test the unit test with our data and again reset the default data.</p>

<p style="padding: 0; margin: 8px;">In this approach, we don't need to make the dummy class for the testing and not to change in the ApplicationSettings class.</p>

<p style="padding: 0; margin: 8px;">now everything is working fine.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">What happens if for some reason the test crashes while there is a change done?<br />
Also, you are still adding dummy things in the real configuration that has nothing to do with the real application.<br />
You should not touch real application data when doing tests.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D11034#inline-54669">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ApplicationSettingsTest.cpp:44</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #aa4000">private</span> <span style="color: #a0a000">slots</span><span class="p">:</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">  <span style="color: #aa4000">void</span> <span class="n">ApplicationSettingsInitializationTest</span><span class="p">();</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">  <span style="color: #aa4000">void</span> <span style="color: #004012">ApplicationSettingsTest</span><span class="p">();</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">indentation should be 4 spaces on c++ files.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D11034#inline-54670">View Inline</a><span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:15</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">add_executable(CoreApplicationSettingsTest ApplicationSettingsTest.cpp)
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">target_link_libraries(CoreApplicationSettingsTest ${CORE_TEST_LIBRARIES})
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">can you do a macro or function to avoid duplicating these 3 lines everytime?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R2 GCompris</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11034">https://phabricator.kde.org/D11034</a></div></div><br /><div><strong>To: </strong>himanshuvishwakarma, jjazeix, dmadaan, rudranilbasu, timotheegiet, GCompris<br /><strong>Cc: </strong>KDE Edu, GCompris, harrymecwan, ganeshredcobra, nityanandkumar, echarruau, rahulyadav, narvaez, scagarwal, apol, timotheegiet, hkaelberer, jjazeix, bcoudoin<br /></div>