Skip to content

Commit 8c3140f

Browse files
committed
Marking some lines as noqa, and some tidy-up.
1 parent dbd66b3 commit 8c3140f

File tree

2 files changed

+47
-5
lines changed

2 files changed

+47
-5
lines changed

tests/unit/new/user_spec.py

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
import httpretty
2+
import json
3+
import re
4+
import mock
5+
import time
6+
7+
from describe import expect
8+
from datetime import datetime
9+
from intercom.collection_proxy import CollectionProxy
10+
from intercom.lib.flat_store import FlatStore
11+
from intercom.user import User
12+
from intercom.utils import create_class_instance
13+
from tests.unit import test_user
14+
15+
16+
get = httpretty.GET
17+
post = httpretty.POST
18+
delete = httpretty.DELETE
19+
20+
r = re.compile
21+
22+
23+
class DescribeIntercomUser:
24+
25+
@httpretty.activate
26+
def it_raises_multiplematchingusers_error_when_receiving_a_conflict(self):
27+
multiple_matching_error = {
28+
'type': 'error.list',
29+
'errors': [
30+
{
31+
'code': 'conflict',
32+
'message': 'Multiple existing users match this email address - must be more specific using user_id'
33+
}
34+
]
35+
}
36+
httpretty.register_uri(
37+
get, r("/users"),
38+
body=json.dumps(multiple_matching_error),
39+
status=400)
40+
print User.find(email='jo@example.com')

tests/unit/notification_spec.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
1+
# -*- coding: utf-8 -*-
2+
13
from describe import expect
24
from intercom.notification import Notification
35
from intercom.utils import create_class_instance
4-
from tests.unit import test_user_notification
56
from tests.unit import test_conversation_notification
7+
from tests.unit import test_user_notification
68

79

810
class DescribeIntercomNotification:
@@ -13,7 +15,7 @@ def it_converts_notification_hash_to_object(self):
1315

1416
def it_returns_correct_model_type_for_user(self):
1517
payload = Notification(**test_user_notification)
16-
User = create_class_instance('User')
18+
User = create_class_instance('User') # noqa
1719

1820
expect(payload.model).to.be_instance_of(User.__class__)
1921
expect(payload.model_type) == User.__class__
@@ -28,12 +30,12 @@ def it_returns_instance_of_user(self):
2830
expect(payload.model).to.be_instance_of(User.__class__)
2931

3032
def it_returns_instance_of_conversation(self):
31-
Conversation = create_class_instance('Conversation')
33+
Conversation = create_class_instance('Conversation') # noqa
3234
payload = Notification(**test_conversation_notification)
3335
expect(payload.model).to.be_instance_of(Conversation.__class__)
3436

3537
def it_returns_correct_model_type_for_conversation(self):
36-
Conversation = create_class_instance('Conversation')
38+
Conversation = create_class_instance('Conversation') # noqa
3739
payload = Notification(**test_conversation_notification)
3840
expect(payload.model_type) == Conversation.__class__
3941

@@ -42,7 +44,7 @@ def it_returns_correct_conversation_notification_topic(self):
4244
expect(payload.topic) == "conversation.user.created"
4345

4446
def it_returns_inner_user_object_for_conversation(self):
45-
User = create_class_instance('User')
47+
User = create_class_instance('User') # noqa
4648
payload = Notification(**test_conversation_notification)
4749
expect(payload.model.user).to.be_instance_of(User.__class__)
4850

0 commit comments

Comments
 (0)