> Thanks Ian. > > - Should the test_withIds add a third branch that doesn't get > selected/returned in order to test this properly? > > -
Hi Rick, Thanks for reviewing. The test does indeed add a third branch already. It just doesn't assign it to an unused variable. -- https://code.launchpad.net/~wallyworld/launchpad/sharing-details-timeout-997343/+merge/105774 Your team Launchpad code reviewers is subscribed to branch lp:launchpad. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
