Code review comment for lp:~thumper/launchpad/package-branch-edit-owner

Revision history for this message
Tim Penhey (thumper) wrote :

OK, for the first, since I was testing a widget that was being rendered, it seemed easiest to use a page test as it has the results of the widget being rendered.

I was wanting to use a standard widget (most of the time) for the rendering of the owner drop-down, and I didn't feel like getting into defining a new custom widget type.

Providing a real model method may well be preferable, and if we get to having a special user selector (instead of the search popup widget), then we may well need this.

I've added a comment to the top of the block of commands in the doctest.

« Back to merge proposal