gclient_scm: add test to catch bug fixed by r208010

gclient_scm: add test to catch bug fixed by r208010

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=26577

[M-A Ruel, 2009-09-18 13:07:23]

http://codereview.chromium.org/214017/diff/1/2
File tests/gclient_test.py (right):

http://codereview.chromium.org/214017/diff/1/2#newcode1142
Line 1142: # TODO(msb): fix bug and enable assertion below
what bug? be just enough descriptive so that someone else could do it, or at
least understand what you are referring to. :)

[msb, 2009-09-18 16:23:27]

maruel@chromium.org (maruel@chromium.org) wrote:
>
> http://codereview.chromium.org/214017/diff/1/2
> File tests/gclient_test.py (right):
>
> http://codereview.chromium.org/214017/diff/1/2#newcode1142
> Line 1142: # TODO(msb): fix bug and enable assertion below
> what bug? be just enough descriptive so that someone else could do it,
> or at least understand what you are referring to. :)
>

You get dupes of 'a' so the comparison fails. I suspect this is proably OK.

I was thinking folks could uncomment the assertion to see the bug. But yeah,
I could save them the effort by describing the bug.

> http://codereview.chromium.org/214017

[M-A Ruel, 2009-09-18 16:31:13]

lgtm