Nested resource class methods

[ob-stripe]

r? @brandur-stripe
cc @stripe/api-libraries

Basically a port of stripe/stripe-ruby#597 for the Python library.

A couple differences:

• implemented as a decorator, because Python

• the update methods are called modify_<nested_resource> instead of update_<nested_resource> to match the pre-existing naming convention in the library.

Everything else should be the same.

[brandur-stripe]

Man Olivier you are on a roll here! Nice work — I'm especially loving the really extensive test suite, and the fact that the decorator class has tests directly.

I'll wait on a little discussion in stripe/stripe-ruby#597, but LGTM.

[brandur-stripe]

@ob-stripe Relating to stripe/stripe-ruby#600, mind if we require an explicit list of supported operations here?

[ob-stripe]

Pushed another commit to require an explicit list of operations.

[brandur-stripe]

Thanks OB!

I think you missed one. From CI:

Traceback (most recent call last):
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/unittest2/loader.py", line 456, in _find_test_path
    module = self._get_module_from_name(name)
  File "/home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages/unittest2/loader.py", line 395, in _get_module_from_name
    __import__(name)
  File "/home/travis/build/stripe/stripe-python/stripe/test/resources/test_api_resource.py", line 121, in <module>
    class NestedResourceClassMethodsTests(StripeApiTestCase):
  File "/home/travis/build/stripe/stripe-python/stripe/test/resources/test_api_resource.py", line 122, in NestedResourceClassMethodsTests
    @stripe.resource.nested_resource_class_methods('nested')
  File "/home/travis/build/stripe/stripe-python/stripe/resource.py", line 113, in nested_resource_class_methods
    raise ValueError("operations list required")
ValueError: operations list required

[ob-stripe]

Oops! Should be fixed now.

[brandur-stripe]

Released as 1.68.0.