Unity 8

Code review comment for lp:~unity-team/unity8/lp1475678.surface-occlude

  1. lp1475678.surface-occlude
  2. Merge into trunk
Revision history for this message
Daniel d'Andrada (dandrader) wrote :

On 29/10/2015 12:32, Nick Dedekind wrote:
> These are supposed to be unit tests which are testing the stage aren't they, so we shouldn't be testing the decorations.

Kind of. If you skip this part you're missing the
interactions/integration between the decoration and the Stage. As I
previously explained. So your test is not wrong, but it's definitely
less valuable in my opinion.

But ok, won't block because of this.

« Back to merge proposal