Hi,
after this commit
https://cgit.kde.org/plasma-framework.git/commit/?id=9f08668147d2e99f8b5069ff50d5c54e672a87af
The iconitemtest started to fail. As we can see from that commit the set of changes starts with this:
if (sourceString.endsWith(QLatin1String(".svg")) || sourceString.endsWith(QLatin1String(".svgz"))) { m_icon = QIcon(localFile); m_imageIcon = QImage(); } else { m_icon = QIcon(localFile); m_imageIcon = QImage(); }
As we can see the code executed in the "if" block and the "else" block is exactly the same. My guess (and it's just a wild guess because I don't actually understand the code involved) is that you actually wanted to do this:
if (sourceString.endsWith(QLatin1String(".svg")) || sourceString.endsWith(QLatin1String(".svgz"))) { m_icon = QIcon(localFile); m_imageIcon = QImage(); } else { m_icon = QIcon(); m_imageIcon = QImage(localfile); }
So I'm attaching a patch doing that ↑