cipd: Skip test on Windows for now.

cipd: Skip test on Windows for now.

The code compiles, but unit tests don't work because Windows handles symlinks
and other file system stuff differently.

R=iannucci@chromium.org
BUG=487483
Committed: https://chromium.googlesource.com/infra/infra/+/525831118c04806a91726c73c4a4fa179dc16ff3

[iannucci, 2015-05-13 08:18:19]

lgtm, though I would consider the startswith thing, even though it's a bit
weird. Being able to say 'win' and 'liunx' would be pretty sane, imo.

https://codereview.chromium.org/1133353002/diff/1/go/test.py
File go/test.py (right):

https://codereview.chromium.org/1133353002/diff/1/go/test.py#newcode92
go/test.py:92: "skip_testing": boolean or a list of platforms to skip tests on,
why not always a list of platforms?

https://codereview.chromium.org/1133353002/diff/1/go/test.py#newcode145
go/test.py:145: return sys.platform in skip
   return sys.platform.startswith(skip)

maybe?

Then "linux" would work. So would [""], actually (to replace 'true'). Or maybe
it's 1:20am and I'm loopy.

[iannucci, 2015-05-13 08:18:54]

technically... it would be really good if the platform values equaled the GOOS
values, since this is go after all.

[Vadim Sh., 2015-05-13 22:57:24]

https://codereview.chromium.org/1133353002/diff/1/go/test.py
File go/test.py (right):

https://codereview.chromium.org/1133353002/diff/1/go/test.py#newcode92
go/test.py:92: "skip_testing": boolean or a list of platforms to skip tests on,
On 2015/05/13 08:18:19, iannucci wrote:
> why not always a list of platforms?

Okay...

https://codereview.chromium.org/1133353002/diff/1/go/test.py#newcode145
go/test.py:145: return sys.platform in skip
On 2015/05/13 08:18:19, iannucci wrote:
>    return sys.platform.startswith(skip)
> 
> maybe?
> 
> Then "linux" would work. So would [""], actually (to replace 'true'). Or maybe
> it's 1:20am and I'm loopy.

[""] is sort of confusing... 

Made it GOOS ("darwin", "linux", "windows").

[Vadim Sh., 2015-05-13 22:57:29]

The CQ bit was checked by vadimsh@chromium.org

[Vadim Sh., 2015-05-13 22:57:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/1133353002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-05-13 22:57:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1133353002/20001

[commit-bot: I haz the power, 2015-05-13 23:01:03]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/525831118c04806a91726c73c4a4f...