Comment 7 for bug 1277104

Is there any justification for this? Some of the resulting changes read very awkwardly to me, such as:

  self.assertEqual(2, len(collection))

Because I would never write the "expected" first, like this:

  if 2 == len(collection):
    pass

I would always start with the variable (the "actual"), like this:

  if len(collection) == 2:
    pass