Code review comment for lp:~jjed/unity/tooltip-delay

Revision history for this message
Marco Trevisan (TreviƱo) (3v1n0) wrote :

Hi, I've checked again this branch against the new trunk, and while the behavior looks good (probably the 1sec delay duration can be discussed with design, but the code is fine), the tests need some tuning while the merge conflicts should be resolved:

Instead of self.assertTrue/assertFalse when possible use:
  self.assertThat(introspection.value, Eventually(Equals(True)))

Then move your tests into LauncherTooltipTests that

Also, probabily the HUD / BFB launcher icons special code to override the tooltip show, could be replaced by your codepath, that is more general and I guess can implement LauncherTestCase and define in its setUp self.set_unity_option('launcher_hide_mode', 0)

review: Needs Fixing

« Back to merge proposal