Separate the version of .isolated and .isolated.state.

Separate the version of .isolated and .isolated.state.

This permits making breaking changes to .state more easily.

- Sort returned values for load_isolate_for_config(), makes result easier to
  read.
- Add assert for ConfigSettings internal state.
- Add assert for 'includes' path on Windows.
- Add tests in preparation to functional changes. In fact the vast majority of
  this CL is test cases.

R=vadimsh@chromium.org
BUG=swarming:98

Committed: https://code.google.com/p/swarming/source/detail?repo=client&r=226ab80

[M-A, 2014-03-28 19:30:10]

This makes the next CL less of a code dump. I haven't completely fixed the code
yet but I'm soon there.

https://codereview.appspot.com/81960043/diff/1/isolate.py
File isolate.py (right):

https://codereview.appspot.com/81960043/diff/1/isolate.py#newcode457
isolate.py:457: 'child_isolated_files',
Sorted in alphabetical order.

https://codereview.appspot.com/81960043/diff/1/tests/isolate_format_test.py
File tests/isolate_format_test.py (right):

https://codereview.appspot.com/81960043/diff/1/tests/isolate_format_test.py#n...
tests/isolate_format_test.py:1266: ['OS=="amiga" or OS=="win"', {
Added a 4th value 'amiga' to get more coverage of edge cases and moved the
actions below so this data is copy-pastable.

https://codereview.appspot.com/81960043/diff/1/tests/isolate_test.py
File tests/isolate_test.py (right):

https://codereview.appspot.com/81960043/diff/1/tests/isolate_test.py#newcode1077
tests/isolate_test.py:1077: # TODO(maruel): It's going to be fixed soon, bug
#98.
I'll remove this and the early return on the previous test case in the next CL
that fixes this problem.

[vadimsh, 2014-03-28 19:51:09]

lgtm with question

https://codereview.appspot.com/81960043/diff/1/isolate.py
File isolate.py (right):

https://codereview.appspot.com/81960043/diff/1/isolate.py#newcode484
isolate.py:484: EXPECTED_VERSION = isolateserver.ISOLATED_FILE_VERSION + '1'
What does "+ '1'" mean exactly? Why '1'?

https://codereview.appspot.com/81960043/diff/1/isolate_format.py
File isolate_format.py (right):

https://codereview.appspot.com/81960043/diff/1/isolate_format.py#newcode505
isolate_format.py:505: assert values == {}
, values

[M-A, 2014-03-29 02:54:52]

https://codereview.appspot.com/81960043/diff/1/isolate.py
File isolate.py (right):

https://codereview.appspot.com/81960043/diff/1/isolate.py#newcode484
isolate.py:484: EXPECTED_VERSION = isolateserver.ISOLATED_FILE_VERSION + '1'
On 2014/03/28 19:51:09, vadimsh wrote:
> What does "+ '1'" mean exactly? Why '1'?

I wanted to embed the .isolated version in there so any little change will cause
the .state to not be loaded.

https://codereview.appspot.com/81960043/diff/1/isolate_format.py
File isolate_format.py (right):

https://codereview.appspot.com/81960043/diff/1/isolate_format.py#newcode505
isolate_format.py:505: assert values == {}
On 2014/03/28 19:51:09, vadimsh wrote:
> , values

Done.

https://codereview.appspot.com/81960043/diff/1/isolate_format.py#newcode505
isolate_format.py:505: assert values == {}
On 2014/03/28 19:51:09, vadimsh wrote:
> , values

Done.

[M-A, 2014-03-29 20:23:02]

Committed patchset #3 manually as r226ab80 (presubmit successful).