Code review comment for lp:~saviq/unity-2d/test_launcher-menu-focus

Revision history for this message
MichaƂ Sawicz (saviq) wrote :

> It's a bit confusing when you say
> "Check that there's only one action"
> and then do
> verify_equal( 2, 0, 'There was an unexpected number of actions in the menu'
> ) {
> actions.count
> }
> The same for the others you say very there's 2 and check against >= 4

Problem is that's actually what we have - check for yourself with tdriver - we have ~2*action_count of QActions. I could maybe check only for those that are active. Let me investigate.

« Back to merge proposal