On 6/2/2014 4:51 PM, Gijs Kruitbosch wrote: > Concretely, IMO in the code you cite there should be a blank line before > each of the 'parent' reassignments.
I definitely agree, and I would also use the "Assert." prefix to make the separation between action and check clearer (while if I understand correctly you'd prefer the shorter option). Cheers, Paolo _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform