Skip to content
This repository has been archived by the owner on May 26, 2022. It is now read-only.

Commit

Permalink
Fix 2.7 mock compatibility
Browse files Browse the repository at this point in the history
  • Loading branch information
robinske committed Nov 30, 2018
1 parent bee1c96 commit 7a0073b
Showing 1 changed file with 8 additions and 17 deletions.
25 changes: 8 additions & 17 deletions tests/test_one_touch.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
from mock import MagicMock
from authy.api.resources import OneTouch
from authy.api.resources import OneTouchResponse
from authy import AuthyException
Expand All @@ -9,17 +10,12 @@
else:
import unittest

if sys.version_info < (3, 0):
from mock import MagicMock
else:
from unittest.mock import MagicMock


class OneTouchTest(unittest.TestCase):

def setUp(self):
self.resource = MagicMock()
self.resource.return_value = OneTouch
self.resource = MagicMock(
OneTouch(test_helper.API_URL, test_helper.API_KEY))
self.response = MagicMock()
otr = MagicMock(OneTouchResponse(self.resource, self.response))
otr.errors = MagicMock(return_value={})
Expand All @@ -30,7 +26,7 @@ def setUp(self):

self.resource.send_request = MagicMock(return_value=otr)
self.resource.get_approval_status = MagicMock(return_value=otr)

self.resource.clean_logos = OneTouch.clean_logos
self.resource.__make_http_query = OneTouch._OneTouch__make_http_query
self.resource.validate_one_touch_signature = OneTouch.validate_one_touch_signature
self.resource.api_key = 'foobar123'
Expand Down Expand Up @@ -82,7 +78,7 @@ def test_validate_request_blank_user_id(self):
message = "Login requested for a CapTrade Bank account."

with self.assertRaises(AuthyException) as context:
self.resource._validate_request(self, user_id, message)
self.resource._validate_request(self.resource, user_id, message)

self.assertEqual(
"Invalid authy id, user id is requred and must be an integer value.", str(context.exception))
Expand All @@ -94,7 +90,7 @@ def test_validate_request_blank_message(self):
message = ''

with self.assertRaises(AuthyException) as context:
self.resource._validate_request(self, user_id, message)
self.resource._validate_request(self.resource, user_id, message)

self.assertEqual(
"Invalid message - should not be empty. It is required", str(context.exception))
Expand Down Expand Up @@ -132,25 +128,21 @@ def side_effect(user_id, message):
self.resource._validate_request.assert_called_once

def test_clean_logos_invalid_key(self):
self.resource.clean_logos = OneTouch.clean_logos

logos = [dict(wrong='default', url='https://www.python.org/static/img/python-logo.png'),
dict(res='low', url='https://www.python.org/static/img/python-logo.png')]

with self.assertRaises(AuthyException) as context:
self.resource.clean_logos(self, logos)
self.resource.clean_logos(self.resource, logos)

self.assertEqual(
"Invalid logos list. Only res and url required", str(context.exception))

def test_clean_logos_invalid_data_type(self):
self.resource.clean_logos = OneTouch.clean_logos

logos = dict(
res='default', url='https://www.python.org/static/img/python-logo.png')

with self.assertRaises(AuthyException) as context:
self.resource.clean_logos(self, logos)
self.resource.clean_logos(self.resource, logos)

self.assertEqual(
"Invalid logos list. Only res and url required", str(context.exception))
Expand Down Expand Up @@ -186,7 +178,6 @@ def test_ONETOUCH_CALLBACK_CHECK_WD_GET_METHOD(self):
self.assertEqual(touch, True)

def test_ONETOUCH_CALLBACK_CHECK_WD_GET_METHOD_INVAILED_NONCE(self):
self.resource.validate_one_touch_signature = OneTouch.validate_one_touch_signature
touch = self.resource.validate_one_touch_signature(self.resource,
test_helper.GET_REQ_SIGNATURE,
'INVAILED NONCE',
Expand Down

0 comments on commit 7a0073b

Please sign in to comment.