Skip to content

Commit fa1c596

Browse files
committed
Merge pull request #61 from jakednoble/master
attachment_test bug fixed
2 parents bbedd6a + 971d339 commit fa1c596

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

test/attachment_test.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -204,11 +204,11 @@ def test_FromJSONEmptyObject(self):
204204
attachment = Attachment.from_json('{}')
205205
self.assertIsNone(attachment.usage_type)
206206
self.assertIsNone(attachment.display)
207-
self.assertIsNone(attachment.description)
207+
self.assertFalse(hasattr(attachment, '_description'))
208208
self.assertIsNone(attachment.content_type)
209209
self.assertIsNone(attachment.length)
210210
self.assertIsNone(attachment.sha2)
211-
self.assertIsNone(attachment.fileurl)
211+
self.assertFalse(hasattr(attachment, '_fileurl'))
212212

213213
def test_FromJSONExceptionEmpty(self):
214214
with self.assertRaises(ValueError):
@@ -221,6 +221,8 @@ def test_FromJSONusage_type(self):
221221
self.assertIsNone(attachment.content_type)
222222
self.assertIsNone(attachment.length)
223223
self.assertIsNone(attachment.sha2)
224+
self.assertFalse(hasattr(attachment, '_description'))
225+
self.assertFalse(hasattr(attachment, '_fileurl'))
224226

225227
def test_fromJSONcontent_type(self):
226228
attachment = Attachment.from_json('{"usage_type":"test", "content_type":"test"}')

0 commit comments

Comments
 (0)