fix gclient_scm unittests

Let the unittest avoid the upstream change.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=79452

[TVL, 2011-03-24 19:20:46]

I believe this will fix the unittest for gclient_scm.py (i tried manually
running them)

[M-A Ruel, 2011-03-24 19:30:28]

ugh, it'd be highly desirable to fix the test instead, will check.

[TVL, 2011-03-24 19:34:31]

On 2011/03/24 19:30:28, Marc-Antoine Ruel wrote:
> ugh, it'd be highly desirable to fix the test instead, will check.

So the problem is the current url comes back as '.'.  So I check for that, or we
look at changing the url to also be '.' and remove the code here as things would
match.

[TVL, 2011-03-24 19:50:04]

On 2011/03/24 19:34:31, TVL wrote:
> On 2011/03/24 19:30:28, Marc-Antoine Ruel wrote:
> > ugh, it'd be highly desirable to fix the test instead, will check.
> 
> So the problem is the current url comes back as '.'.  So I check for that, or
we
> look at changing the url to also be '.' and remove the code here as things
would
> match.

Ah, we need the git:// so it creates the right type of backing scm module.  So
the test would have to force the scm modules some other way so we can change the
url to match.

[M-A Ruel, 2011-03-25 20:38:53]

http://codereview.chromium.org/6696080/diff/1/gclient_scm.py
File gclient_scm.py (right):

http://codereview.chromium.org/6696080/diff/1/gclient_scm.py#newcode220
gclient_scm.py:220: if current_url != url and url != 'git://foo':
Just add a
# TODO(maruel): Delete url != 'git://foo' since it's just to make the unit test
pass.

and commit.

I'm reworking the unit test but it'll take a moment, thanks for doing this.

lgtm