[Merge] lp:~mardy/webbrowser-app/add-onlineaccount-support-for-container2 into lp:webbrowser-app

Olivier Tilloy olivier.tilloy at canonical.com
Thu May 29 09:13:13 UTC 2014


1442	+    QCOMPARE(dbPathChanged.count(), 0);
1462	+    QCOMPARE(dbPathChanged.count(), 0);

Those tests are useless, as you just instantiated the signal spy, so you are guaranteed that the count is 0.
-- 
https://code.launchpad.net/~mardy/webbrowser-app/add-onlineaccount-support-for-container2/+merge/211701
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.



More information about the Ubuntu-reviews mailing list