D17911: Fix failing knewfilemenu test and underlying reason for its failure
David Faure
noreply at phabricator.kde.org
Fri Jan 4 21:37:12 GMT 2019
dfaure added a comment.
Hmm, let me read the unittest more closely.
The scenario is: "New Folder" exists. The user asks to create a new folder, the initial suggestion in the dialog is "New Folder (1)". The user removes the (1), and hits OK, i.e. the user is asking to create "New Folder" again.
The unittest is then just checking that "New Folder" exists, which is kind of broken since it does exist anyway, since that's the initial setup.
But now you're testing that "New Folder (1)" gets created? That seems really odd, it's not what the user is asking for.
REPOSITORY
R241 KIO
BRANCH
fix-test-and-code (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D17911
To: ngraham, #frameworks, dfaure, elvisangelaccio, aacid
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190104/7d6e1388/attachment.html>
More information about the Kde-frameworks-devel
mailing list