Fixing Python3 support for encoding mocked payloads.

jkeyes · jkeyes · commit d91d62333d24 · 2015-04-02T03:10:37.000+01:00
diff --git a/tests/unit/test_request.py b/tests/unit/test_request.py
@@ -82,7 +82,7 @@ def it_raises_an_unexpected_typed_error(self):
                 }
             ]
         }
-        content = json.dumps(payload)
+        content = json.dumps(payload).encode('utf-8')
         resp = Mock(content=content, status_code=200, headers=headers)
         with patch('requests.request') as mock_method:
             mock_method.return_value = resp
@@ -104,7 +104,7 @@ def it_raises_an_unexpected_untyped_error(self):
                 }
             ]
         }
-        content = json.dumps(payload)
+        content = json.dumps(payload).encode('utf-8')
         resp = Mock(content=content, status_code=200, headers=headers)
         with patch('requests.request') as mock_method:
             mock_method.return_value = resp
@@ -130,7 +130,7 @@ def it_raises_a_bad_request_error(self):
         for code in ['missing_parameter', 'parameter_invalid', 'bad_request']:
             payload['errors'][0]['type'] = code
 
-            content = json.dumps(payload)
+            content = json.dumps(payload).encode('utf-8')
             resp = Mock(content=content, status_code=200, headers=headers)
             with patch('requests.request') as mock_method:
                 mock_method.return_value = resp
@@ -151,7 +151,7 @@ def it_raises_an_authentication_error(self):
         for code in ['unauthorized', 'forbidden']:
             payload['errors'][0]['type'] = code
 
-            content = json.dumps(payload)
+            content = json.dumps(payload).encode('utf-8')
             resp = Mock(content=content, status_code=200, headers=headers)
             with patch('requests.request') as mock_method:
                 mock_method.return_value = resp
@@ -169,7 +169,7 @@ def it_raises_resource_not_found_by_type(self):
                 }
             ]
         }
-        content = json.dumps(payload)
+        content = json.dumps(payload).encode('utf-8')
         resp = Mock(content=content, status_code=200, headers=headers)
         with patch('requests.request') as mock_method:
             mock_method.return_value = resp
@@ -187,7 +187,7 @@ def it_raises_rate_limit_exceeded(self):
                 }
             ]
         }
-        content = json.dumps(payload)
+        content = json.dumps(payload).encode('utf-8')
         resp = Mock(content=content, status_code=200, headers=headers)
         with patch('requests.request') as mock_method:
             mock_method.return_value = resp
@@ -205,7 +205,7 @@ def it_raises_a_service_unavailable_error(self):
                 }
             ]
         }
-        content = json.dumps(payload)
+        content = json.dumps(payload).encode('utf-8')
         resp = Mock(content=content, status_code=200, headers=headers)
         with patch('requests.request') as mock_method:
             mock_method.return_value = resp
@@ -223,7 +223,7 @@ def it_raises_a_multiple_matching_users_error(self):
                 }
             ]
         }
-        content = json.dumps(payload)
+        content = json.dumps(payload).encode('utf-8')
         resp = Mock(content=content, status_code=200, headers=headers)
         with patch('requests.request') as mock_method:
             mock_method.return_value = resp
diff --git a/tests/unit/test_user.py b/tests/unit/test_user.py
@@ -361,7 +361,7 @@ def it_raises_a_multiple_matching_users_error_when_receiving_a_conflict(self):
             'x-ratelimit-remaining': 500,
             'x-ratelimit-reset': 1427932858
         }
-        content = json.dumps(payload)
+        content = json.dumps(payload).encode('utf-8')
         resp = Mock(content=content, status_code=200, headers=headers)
         with patch('requests.request') as mock_method:
             mock_method.return_value = resp

Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ def it_raises_an_unexpected_typed_error(self):`
`82`	`82`	`}`
`83`	`83`	`]`
`84`	`84`	`}`
`85`		`- content = json.dumps(payload)`
	`85`	`+ content = json.dumps(payload).encode('utf-8')`
`86`	`86`	`resp = Mock(content=content, status_code=200, headers=headers)`
`87`	`87`	`with patch('requests.request') as mock_method:`
`88`	`88`	`mock_method.return_value = resp`
`@@ -104,7 +104,7 @@ def it_raises_an_unexpected_untyped_error(self):`
`104`	`104`	`}`
`105`	`105`	`]`
`106`	`106`	`}`
`107`		`- content = json.dumps(payload)`
	`107`	`+ content = json.dumps(payload).encode('utf-8')`
`108`	`108`	`resp = Mock(content=content, status_code=200, headers=headers)`
`109`	`109`	`with patch('requests.request') as mock_method:`
`110`	`110`	`mock_method.return_value = resp`
`@@ -130,7 +130,7 @@ def it_raises_a_bad_request_error(self):`
`130`	`130`	`for code in ['missing_parameter', 'parameter_invalid', 'bad_request']:`
`131`	`131`	`payload['errors'][0]['type'] = code`
`132`	`132`
`133`		`- content = json.dumps(payload)`
	`133`	`+ content = json.dumps(payload).encode('utf-8')`
`134`	`134`	`resp = Mock(content=content, status_code=200, headers=headers)`
`135`	`135`	`with patch('requests.request') as mock_method:`
`136`	`136`	`mock_method.return_value = resp`
`@@ -151,7 +151,7 @@ def it_raises_an_authentication_error(self):`
`151`	`151`	`for code in ['unauthorized', 'forbidden']:`
`152`	`152`	`payload['errors'][0]['type'] = code`
`153`	`153`
`154`		`- content = json.dumps(payload)`
	`154`	`+ content = json.dumps(payload).encode('utf-8')`
`155`	`155`	`resp = Mock(content=content, status_code=200, headers=headers)`
`156`	`156`	`with patch('requests.request') as mock_method:`
`157`	`157`	`mock_method.return_value = resp`
`@@ -169,7 +169,7 @@ def it_raises_resource_not_found_by_type(self):`
`169`	`169`	`}`
`170`	`170`	`]`
`171`	`171`	`}`
`172`		`- content = json.dumps(payload)`
	`172`	`+ content = json.dumps(payload).encode('utf-8')`
`173`	`173`	`resp = Mock(content=content, status_code=200, headers=headers)`
`174`	`174`	`with patch('requests.request') as mock_method:`
`175`	`175`	`mock_method.return_value = resp`
`@@ -187,7 +187,7 @@ def it_raises_rate_limit_exceeded(self):`
`187`	`187`	`}`
`188`	`188`	`]`
`189`	`189`	`}`
`190`		`- content = json.dumps(payload)`
	`190`	`+ content = json.dumps(payload).encode('utf-8')`
`191`	`191`	`resp = Mock(content=content, status_code=200, headers=headers)`
`192`	`192`	`with patch('requests.request') as mock_method:`
`193`	`193`	`mock_method.return_value = resp`
`@@ -205,7 +205,7 @@ def it_raises_a_service_unavailable_error(self):`
`205`	`205`	`}`
`206`	`206`	`]`
`207`	`207`	`}`
`208`		`- content = json.dumps(payload)`
	`208`	`+ content = json.dumps(payload).encode('utf-8')`
`209`	`209`	`resp = Mock(content=content, status_code=200, headers=headers)`
`210`	`210`	`with patch('requests.request') as mock_method:`
`211`	`211`	`mock_method.return_value = resp`
`@@ -223,7 +223,7 @@ def it_raises_a_multiple_matching_users_error(self):`
`223`	`223`	`}`
`224`	`224`	`]`
`225`	`225`	`}`
`226`		`- content = json.dumps(payload)`
	`226`	`+ content = json.dumps(payload).encode('utf-8')`
`227`	`227`	`resp = Mock(content=content, status_code=200, headers=headers)`
`228`	`228`	`with patch('requests.request') as mock_method:`
`229`	`229`	`mock_method.return_value = resp`
Original file line number	Diff line number	Diff line change
`@@ -361,7 +361,7 @@ def it_raises_a_multiple_matching_users_error_when_receiving_a_conflict(self):`
`361`	`361`	`'x-ratelimit-remaining': 500,`
`362`	`362`	`'x-ratelimit-reset': 1427932858`
`363`	`363`	`}`
`364`		`- content = json.dumps(payload)`
	`364`	`+ content = json.dumps(payload).encode('utf-8')`
`365`	`365`	`resp = Mock(content=content, status_code=200, headers=headers)`
`366`	`366`	`with patch('requests.request') as mock_method:`
`367`	`367`	`mock_method.return_value = resp`