<table><tr><td style="">aacid 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/D17180">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/D17180#inline-118090">View Inline</a><span style="color: #4b4d51; font-weight: bold;">hriesenbeck</span> wrote in <span style="color: #4b4d51; font-weight: bold;">sessionmanagerfixedtest.cpp:215</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This writes to parleys config file. If this is not allowed in automated testing, the sub test for conjugation practice mode has to be removed.<br />
The writing to config file is necessary because parley EntryFilter class loads the document specific preferences from disk. The tenses for conjugation practice are document specific and needed for this test.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">right, changing peoples config when running tests is bad, but this should be hopefully be fixed by calling</p>

<p style="padding: 0; margin: 8px;">QStandardPaths::setTestModeEnabled(true);</p>

<p style="padding: 0; margin: 8px;">in initTestCase.</p>

<p style="padding: 0; margin: 8px;">Should the same be done for testentrytest.cpp ?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R340 Parley</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17180">https://phabricator.kde.org/D17180</a></div></div><br /><div><strong>To: </strong>hriesenbeck, KDE Edu<br /><strong>Cc: </strong>aacid, kde-edu, narvaez, apol<br /></div>