<table><tr><td style="">kossebau added a comment.
</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/D4430" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Still no matching unit tests added for now. There are two things that could see tests:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">the exposed <tt style="background: #ebebeb; font-size: 13px;">UnknownDeclarationProblem::findMatchingIncludeFiles(...)</tt> method</li>
<li class="remarkup-list-item">the <tt style="background: #ebebeb; font-size: 13px;">included_files</tt> variable properly filled with data</li>
</ol>

<p>Both things will need me to invest some more time, as I yet have to grasp things around DUChain enough to write a proper test for the exposed method.<br />
And classhelper code is without any testing approach so far, from what I found (also nothing seen in oldcpp), so that needs some more thinking how to do this properly.</p>

<p>Given 5.1 is planned to be released this week and me lacking enough time for the above now, I propose to have the current patch in already without dedicated unit tests, as it fixes a regression for cpp->clang when it comes to code generation from file templates (missing <tt style="background: #ebebeb; font-size: 13px;">included_files</tt> variable content). Given my dissatisfaction with the current code generation abilities, you should see me working on tests in the near future at least, as part of other development.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4430" rel="noreferrer">https://phabricator.kde.org/D4430</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop, mwolff<br /><strong>Cc: </strong>kdevelop-devel<br /></div>