Adding basic tests for shared utils.

Adding basic tests for shared utils.

[danielholevoet, 2013-03-18 18:24:22]

LGTMoi

https://codereview.appspot.com/7494047/diff/1/endpoints_proto_datastore/utils...
File endpoints_proto_datastore/utils_test.py (right):

https://codereview.appspot.com/7494047/diff/1/endpoints_proto_datastore/utils...
endpoints_proto_datastore/utils_test.py:28: # Make sure this does not fail
Isn't this comment unnecessary?

[dhermes, 2013-03-18 18:29:09]

FYI:

I'm not quite done with this yet, so maybe you want to un-LGTM it?

I want to add a test runner that will "just work" in most cases.

https://codereview.appspot.com/7494047/diff/1/endpoints_proto_datastore/utils...
File endpoints_proto_datastore/utils_test.py (right):

https://codereview.appspot.com/7494047/diff/1/endpoints_proto_datastore/utils...
endpoints_proto_datastore/utils_test.py:28: # Make sure this does not fail
Maybe? It's the point of overriding the default behavior of issubclass (from the
stdlib).

>>> issubclass(int, None)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: issubclass() arg 2 must be a class or tuple of classes

On 2013/03/18 18:24:22, danielholevoet wrote:
> Isn't this comment unnecessary?

[danielholevoet, 2013-03-18 18:30:31]

Not sure how to un-LGTMoi. Just let me know when the rest is there and I can
look again.

[dhermes, 2013-03-21 21:29:49]

Test runner for "any" environment.

[danielholevoet, 2013-03-21 21:49:14]

On 2013/03/21 21:29:49, dhermes wrote:
> Test runner for "any" environment.

LGTM++

[dhermes, 2013-03-21 21:50:49]

Added in
https://code.google.com/p/endpoints-proto-datastore/source/detail?r=1180a1787...