Fix docs, explicit content-type, tests

Author: pkopac

README.md

@@ -179,22 +179,28 @@ chartmogul.Plan.destroy(config, uuid='')
 #### [Invoices](https://dev.chartmogul.com/docs/invoices)
 
 ```python
-chartmogul.Invoice.create(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb', data={})
-chartmogul.Invoice.all(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb', page=2, per_page=10)
+import chartmogul.imp
+
+chartmogul.imp.Invoice.create(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb', data={})
+chartmogul.imp.Invoice.all(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb', page=2, per_page=10)
 ```
 
 #### [Transactions](https://dev.chartmogul.com/docs/transactions)
 
 ```python
-chartmogul.Transaction.create(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb', data={})
+import chartmogul.imp
+
+chartmogul.imp.Transaction.create(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb', data={})
 ```
 
 #### [Subscriptions](https://dev.chartmogul.com/docs/subscriptions)
 
 ```python
-chartmogul.Subscription.all(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb')
-chartmogul.Subscription.cancel(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb' data={'cancelled_at': ''})
-chartmogul.Subscription.modify(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb' data={'cancellation_dates': []})
+import chartmogul.imp
+
+chartmogul.imp.Subscription.all(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb')
+chartmogul.imp.Subscription.cancel(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb' data={'cancelled_at': ''})
+chartmogul.imp.Subscription.modify(config, uuid='cus_5915ee5a-babd-406b-b8ce-d207133fb4cb' data={'cancellation_dates': []})
 ```
 
 ### [Metrics API](https://dev.chartmogul.com/docs/introduction-metrics-api)
@@ -203,7 +209,7 @@ Available methods in Metrics API:
 
 
 ```python
-chartmogul.metrics.all(config,
+chartmogul.Metrics.all(config,
                        start_date='2015-01-01', # notice the _ here
                        end_date='2015-11-24',
                        interval='month',

chartmogul/__init__.py

@@ -27,7 +27,7 @@
 """
 
 __title__ = 'chartmogul'
-__version__ = '1.0.0'
+__version__ = '1.0.1'
 __build__ = 0x000000
 __author__ = 'ChartMogul Ltd'
 __license__ = 'MIT'

chartmogul/resource.py

@@ -115,11 +115,13 @@ def _request(cls, config, method, http_verb, path, data=None, **kwargs):
                 data = dumps(data, default=json_serial)
 
         return Promise(lambda resolve, _:
-                       resolve(getattr(requests, http_verb)(config.uri + path,
-                                                            data=data,
-                                                            params=params,
-                                                            auth=config.auth)
-                               )).then(cls._load).catch(annotateHTTPError)
+                       resolve(getattr(requests, http_verb)(
+                           config.uri + path,
+                           data=data,
+                           headers={'content-type': 'application/json'},
+                           params=params,
+                           auth=config.auth)
+                       )).then(cls._load).catch(annotateHTTPError)
 
     @classmethod
     def _expandPath(cls, path, kwargs):

test/api/test_plan.py

@@ -9,6 +9,7 @@ class PlanTestCase(unittest.TestCase):
     """
     Tests cursor query parameters & modify (patch a plan).
     """
+    maxDiff = None
 
     @requests_mock.mock()
     def test_cursor_list_plans(self, mock_requests):
@@ -71,3 +72,37 @@ def test_modify_plan(self, mock_requests):
         self.assertEqual(mock_requests.last_request.json(),
                          {"name": u"new_name"})
         self.assertEqual(str(plan), str(expected))
+
+    @requests_mock.mock()
+    def test_create_plan(self, mock_requests):
+        expected_plan_dict = {
+            "uuid": u"whatever_uuid",
+            "data_source_uuid": u"some_uuid",
+            "name": u"new_name",
+            "interval_count": 2,
+            "interval_unit": u"moonshines",
+            "external_id": u"custom_filter"
+        }
+        sent = {
+            "data_source_uuid": "ds_9bb53a1e-edfd-11e6-bf83-af49e978cb11",
+            "name": "Gold Plan",
+            "interval_count": 1,
+            "interval_unit": "month",
+            "external_id": "plan_0002"
+        }
+        mock_requests.register_uri(
+            'POST',
+            "https://api.chartmogul.com/v1/plans",
+            request_headers={'Authorization': 'Basic dG9rZW46c2VjcmV0'},
+            status_code=200,
+            json=expected_plan_dict
+        )
+        config = Config("token", "secret")  # is actually checked in mock
+        plan = Plan.create(
+            config,
+            data=sent).get()
+        expected = Plan(**expected_plan_dict)
+        self.assertEqual(mock_requests.call_count, 1, "expected call")
+        self.assertEqual(mock_requests.last_request.qs, {})
+        self.assertEqual(mock_requests.last_request.json(), sent)
+        self.assertEqual(str(plan), str(expected))