Code review comment for lp:~gary-lasker/software-center/launcher-integration-unit-tests

Revision history for this message
Gary Lasker (gary-lasker) wrote :

Hi Michael. Ok, most recent comment first. Indeed, the settings for expected results were leftover from the way the tests worked previously and so indeed are superfluous. I have removed them in my branch. Thanks for noticing that!

Regarding your comment just previous to the most recent, yes, that was just a piece of cleanup that I missed. I fixed that now as well. Oh, please note that it wasn't the first four lines of code that needed to be removed, but rather some lines after that.

Thanks again!

« Back to merge proposal