<table><tr><td style="">dfaure 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/D13043">View Revision</a></tr></table><br /><div><div><p>BTW, the commit message refers to ModelTest, but that's the old modeltest.cpp, not the new QAbstractItemModelTester, right?</p>

<p>I'm working on QConcatenateTablesProxyModel (<a href="https://codereview.qt-project.org/166323" class="remarkup-link" target="_blank" rel="noreferrer">https://codereview.qt-project.org/166323</a>) and there QAbstractItemModelTester passes with asserts for all these invalid calls. The old modeltest is now considered wrong ;)</p>

<p>Not saying this commit should be reverted or anything, this is just to let you know about QAbstractItemModelTester and the fact that the new idea is narrow contract, asserting is good, while the old modeltest assumed very tolerant model implementations.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R275 KItemModels</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13043">https://phabricator.kde.org/D13043</a></div></div><br /><div><strong>To: </strong>apol, Frameworks, broulik<br /><strong>Cc: </strong>dfaure, broulik, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>