8000 massaged the mock.patch namespaces to fix tests to mock, in particula… · Stackdriver/twilio-python@13ff004 · GitHub
[go: up one dir, main page]
Skip to content
This repository was archived by the owner on Feb 1, 2018. It is now read-only.

Commit 13ff004

Browse files
skylskyl
committed
massaged the mock.patch namespaces to fix tests to mock, in particular, twilio.rest.resources.base.make_twilio_request
1 parent c4a7d90 commit 13ff004

11 files changed

+100
-94
lines changed

‎tests/test_authorized_connect_apps.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ def setUp(self):
1818
self.auth = ("AC123", "token")
1919
self.resource = AuthorizedConnectApps(self.uri, self.auth)
2020

21-
@patch("twilio.rest.resources.make_twilio_request")
21+
@patch("twilio.rest.resources.base.make_twilio_request")
2222
def test_get(self, mock):
2323
mock.return_value = Mock()
2424
mock.return_value.content = '{"connect_app_sid": "SID"}'
@@ -27,7 +27,7 @@ def test_get(self, mock):
2727
mock.assert_called_with("GET", "/base/AuthorizedConnectApps/SID",
2828
auth=self.auth)
2929

30-
@patch("twilio.rest.resources.make_twilio_request")
30+
@patch("twilio.rest.resources.base.make_twilio_request")
3131
def test_list(self, mock):
3232
mock.return_value = Mock()
3333
mock.return_value.content = '{"authorized_connect_apps": []}'

‎tests/test_calls.py

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
from datetime import date
2-
from mock import patch, Mock
3-
from nose.tools import raises, assert_equals, assert_true
2+
from mock import patch
3+
from nose.tools import raises, assert_true
44
from twilio.rest.resources import Calls
55
from tools import create_mock_json
66

@@ -12,7 +12,7 @@
1212
list_resource = Calls(BASE_URI, AUTH)
1313

1414

15-
@patch("twilio.rest.resources.make_twilio_request")
15+
@patch("twilio.rest.resources.base.make_twilio_request")
1616
def test_create_call(mock):
1717
resp = create_mock_json("tests/resources/calls_instance.json")
1818
resp.status_code = 201
@@ -31,28 +31,30 @@ def test_create_call(mock):
3131
mock.assert_called_with("POST", uri, data=exp_params, auth=AUTH)
3232

3333

34-
@patch("twilio.rest.resources.make_twilio_request")
34+
@patch("twilio.rest.resources.base.make_twilio_request")
3535
def test_paging(mock):
3636
resp = create_mock_json("tests/resources/calls_list.json")
3737
mock.return_value = resp
3838

3939
uri = "%s/Calls" % (BASE_URI)
40-
list_resource.list(started_before=date(2010,12,5))
40+
list_resource.list(started_before=date(2010, 12, 5))
4141
exp_params = {'StartTime<': '2010-12-05'}
4242

4343
mock.assert_called_with("GET", uri, params=exp_params, auth=AUTH)
4444

45-
@patch("twilio.rest.resources.make_twilio_request")
45+
46+
@patch("twilio.rest.resources.base.make_twilio_request")
4647
def test_get(mock):
4748
resp = create_mock_json("tests/resources/calls_instance.json")
4849
mock.return_value = resp
4950

5051
uri = "%s/Calls/%s" % (BASE_URI, CALL_SID)
51-
r = list_resource.get(CALL_SID)
52+
list_resource.get(CALL_SID)
5253

5354
mock.assert_called_with("GET", uri, auth=AUTH)
5455

55-
@patch("twilio.rest.resources.make_twilio_request")
56+
57+
@patch("twilio.rest.resources.base.make_twilio_request")
5658
def test_hangup(mock):
5759
resp = create_mock_json("tests/resources/calls_instance.json")
5860
resp.status_code = 204
@@ -65,7 +67,8 @@ def test_hangup(mock):
6567
mock.assert_called_with("POST", uri, data=exp_data, auth=AUTH)
6668
assert_true(r)
6769

68-
@patch("twilio.rest.resources.make_twilio_request")
70+
71+
@patch("twilio.rest.resources.base.make_twilio_request")
6972
def test_cancel(mock):
7073
resp = create_mock_json("tests/resources/calls_instance.json")
7174
resp.status_code = 204
@@ -78,6 +81,7 @@ def test_cancel(mock):
7881
mock.assert_called_with("POST", uri, data=exp_data, auth=AUTH)
7982
assert_true(r)
8083

84+
8185
@raises(AttributeError)
8286
def test_create():
8387
list_resource.delete

‎tests/test_client.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,16 +3,16 @@
33
import unittest2 as unittest
44
else:
55
import unittest
6-
from twilio.rest import TwilioRestClient
7-
from twilio.rest import resources
6+
7+
from twilio.rest import TwilioRestClient, resources
88
from mock import patch, Mock
99

10+
1011
class RestClientTest(unittest.TestCase):
1112

1213
def setUp(self):
1314
self.client = TwilioRestClient("ACCOUNT_SID", "AUTH_TOKEN")
1415

15-
1616
@patch("twilio.rest.make_request")
1717
def test_request(self, mock):
1818
self.client.request("2010-04-01", method="GET")
@@ -21,14 +21,14 @@ def test_request(self, mock):
2121
auth=("ACCOUNT_SID", "AUTH_TOKEN"), data=None)
2222

2323
def test_connect_apps(self):
24-
self.assertIsInstance(self.client.connect_apps,
24+
self.assertIsInstance(self.client.connect_apps,
2525
resources.ConnectApps)
2626

2727
def test_authorized_apps(self):
28-
self.assertIsInstance(self.client.authorized_connect_apps,
28+
self.assertIsInstance(self.client.authorized_connect_apps,
2929
resources.AuthorizedConnectApps)
3030

31-
@patch("twilio.rest.resources.make_request")
31+
@patch("twilio.rest.resources.request.make_request")
3232
def test_conferences(self, mock):
3333
mock.return_value = Mock()
3434
mock.return_value.ok = True

‎tests/test_connect_apps.py

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,9 @@
44
import unittest2 as unittest
55
else:
66
import unittest
7+
78
from mock import Mock, patch
8-
from twilio import TwilioException
99
from twilio.rest.resources import ConnectApps
10-
from twilio.rest.resources import ConnectApp
1110

1211

1312
class ConnectAppTest(unittest.TestCase):
@@ -18,16 +17,16 @@ def setUp(self):
1817
self.auth = ("AC123", "token")
1918
self.resource = ConnectApps(self.uri, self.auth)
2019

21-
@patch("twilio.rest.resources.make_twilio_request")
20+
@patch("twilio.rest.resources.base.make_twilio_request")
2221
def test_get(self, mock):
2322
mock.return_value = Mock()
24-
mock.return_value.content = '{"sid": "SID"}'
23+
mock.return_value.content = '{"sid": "SID"}'
2524

2625
self.resource.get("SID")
2726
mock.assert_called_with("GET", "/base/ConnectApps/SID",
2827
auth=self.auth)
2928

30-
@patch("twilio.rest.resources.make_twilio_request")
29+
@patch("twilio.rest.resources.base.make_twilio_request")
3130
def test_list_with_paging(self, mock):
3231
mock.return_value = Mock()
3332
mock.return_value.content = '{"connect_apps": []}'
@@ -36,7 +35,7 @@ def test_list_with_paging(self, mock):
3635
mock.assert_called_with("GET", "/base/ConnectApps",
3736
params={"Page": 1, "PageSize": 50}, auth=self.auth)
3837

39-
@patch("twilio.rest.resources.make_twilio_request")
38+
@patch("twilio.rest.resources.base.make_twilio_request")
4039
def test_list(self, mock):
4140
mock.return_value = Mock()
4241
mock.return_value.content = '{"connect_apps": []}'
@@ -56,4 +55,3 @@ def test_delete(self):
5655
def test_update(self):
5756
with self.assertRaises(AttributeError):
5857
self.resource.update()
59-

‎tests/test_make_request.py

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,10 @@
99
import simplejson as json
1010

1111
import twilio
12-
from nose.tools import assert_equals
1312
from nose.tools import raises
14-
from mock import patch
15-
from mock import Mock
13+
from mock import patch, Mock
1614
from twilio import TwilioRestException
17-
from twilio.rest.resources import make_request
18-
from twilio.rest.resources import make_twilio_request
15+
from twilio.rest.resources.request import make_request, make_twilio_request
1916

2017
get_headers = {
2118
"User-Agent": "twilio-python/%s" % (twilio.__version__),
@@ -26,7 +23,7 @@
2623
post_headers["Content-Type"] = "application/x-www-form-urlencoded"
2724

2825

29-
@patch('twilio.rest.resources.Response')
26+
@patch('twilio.rest.resources.base.Response')
3027
@patch('httplib2.Http')
3128
def test_get_params(http_mock, response_mock):
3229
http = Mock()
@@ -37,7 +34,7 @@ def test_get_params(http_mock, response_mock):
3734
body=None, headers=None)
3835

3936

40-
@patch('twilio.rest.resources.Response')
37+
@patch('twilio.rest.resources.base.Response')
4138
@patch('httplib2.Http')
4239
def test_get_extra_paranms(http_mock, response_mock):
4340
http = Mock()
@@ -48,7 +45,7 @@ def test_get_extra_paranms(http_mock, response_mock):
4845
body=None, headers=None)
4946

5047

51-
@patch('twilio.rest.resources.Response')
48+
@patch('twilio.rest.resources.base.Response')
5249
@patch('httplib2.Http')
5350
def test_resp_uri(http_mock, response_mock):
5451
http = Mock()
@@ -59,15 +56,16 @@ def test_resp_uri(http_mock, response_mock):
5956
body=None, headers=None)
6057

6158

62-
@patch('twilio.rest.resources.make_request')
59+
@patch('twilio.rest.resources.request.make_request')
6360
def test_make_twilio_request_headers(mock):
6461
url = "http://random/url"
6562
make_twilio_request("POST", url)
6663
mock.assert_called_with("POST", "http://random/url.json",
6764
headers=post_headers)
6865

66+
6967
@raises(TwilioRestException)
70-
@patch('twilio.rest.resources.make_request')
68+
@patch('twilio.rest.resources.request.make_request')
7169
def test_make_twilio_request_bad_data(mock):
7270
resp = Mock()
7371
resp.ok = False

‎tests/test_members.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
1-
from mock import patch, Mock
2-
from datetime import date
3-
from mock import patch, Mock
4-
from nose.tools import raises, assert_equals, assert_true
1+
from mock import patch
52
from tools import create_mock_json
6-
from twilio.rest.resources import Member, Members
3+
from twilio.rest.resources import Members
74

85
QUEUE_SID = "QU1b9faddec3d54ec18488f86c83019bf0"
96
ACCOUNT_SID = "AC123"
@@ -15,7 +12,8 @@
1512

1613
list_resource = Members(BASE_URI, AUTH)
1714

18-
@patch("twilio.rest.resources.make_twilio_request")
15+
16+
@patch("twilio.rest.resources.base.make_twilio_request")
1917
def test_members_list(mock):
2018
resp = create_mock_json("tests/resources/members_list.json")
2119
mock.return_value = resp
@@ -25,7 +23,8 @@ def test_members_list(mock):
2523

2624
mock.assert_called_with("GET", uri, params={}, auth=AUTH)
2725

28-
@patch("twilio.rest.resources.make_twilio_request")
26+
27+
@patch("twilio.rest.resources.base.make_twilio_request")
2928
def test_members_dequeue_front(mock):
3029
resp = create_mock_json("tests/resources/members_instance.json")
3130
mock.return_value = resp
@@ -35,7 +34,8 @@ def test_members_dequeue_front(mock):
3534

3635
mock.assert_called_with("POST", uri, data={"Url": TWIML_URL}, auth=AUTH)
3736

38-
@patch("twilio.rest.resources.make_twilio_request")
37+
38+
@patch("twilio.rest.resources.base.make_twilio_request")
3939
def test_members_dequeue_call(mock):
4040
resp = create_mock_json("tests/resources/members_instance.json")
4141
mock.return_value = resp

‎tests/test_notifications.py

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
from datetime import date
2-
from mock import patch, Mock
3-
from nose.tools import raises, assert_equals, assert_true
2+
from mock import patch
3+
from nose.tools import raises, assert_true
44
from twilio.rest.resources import Notifications
55
from tools import create_mock_json
66

@@ -12,29 +12,32 @@
1212

1313
list_resource = Notifications(BASE_URI, AUTH)
1414

15-
@patch("twilio.rest.resources.make_twilio_request")
15+
16+
@patch("twilio.rest.resources.base.make_twilio_request")
1617
def test_paging(mock):
1718
resp = create_mock_json("tests/resources/notifications_list.json")
1819
mock.return_value = resp
1920

2021
uri = "%s/Notifications" % (BASE_URI)
21-
list_resource.list(before=date(2010,12,5))
22+
list_resource.list(before=date(2010, 12, 5))
2223
exp_params = {'MessageDate<': '2010-12-05'}
2324

2425
mock.assert_called_with("GET", uri, params=exp_params, auth=AUTH)
2526

26-
@patch("twilio.rest.resources.make_twilio_request")
27+
28+
@patch("twilio.rest.resources.base.make_twilio_request")
2729
def test_get(mock):
2830
resp = create_mock_json("tests/resources/notifications_instance.json")
2931
mock.return_value = resp
3032

3133
uri = "%s/Notifications/%s" % (BASE_URI, RE_SID)
32-
r = list_reosource.get(RE_SID)
34+
list_resource.get(RE_SID)
3335

3436
mock.assert_called_with("GET", uri, auth=AUTH)
3537

36-
@patch("twilio.rest.resources.make_twilio_request")
37-
def test_get(mock):
38+
39+
@patch("twilio.rest.resources.base.make_twilio_request")
40+
def test_get2(mock):
3841
resp = create_mock_json("tests/resources/notifications_instance.json")
3942
resp.status_code = 204
4043
mock.return_value = resp
@@ -45,10 +48,12 @@ def test_get(mock):
4548
mock.assert_called_with("DELETE", uri, auth=AUTH)
4649
assert_true(r)
4750

51+
4852
@raises(AttributeError)
4953
def test_create():
5054
list_resource.create
5155

56+
5257
@raises(AttributeError)
5358
def test_update():
5459
list_resource.update

0 commit comments

Comments
 (0)
0