Merge branch 'binary-fields' of git://github.com/benmur/mongoengine
This commit is contained in:
commit
92150e07d3
@ -11,7 +11,8 @@ import decimal
|
|||||||
__all__ = ['StringField', 'IntField', 'FloatField', 'BooleanField',
|
__all__ = ['StringField', 'IntField', 'FloatField', 'BooleanField',
|
||||||
'DateTimeField', 'EmbeddedDocumentField', 'ListField', 'DictField',
|
'DateTimeField', 'EmbeddedDocumentField', 'ListField', 'DictField',
|
||||||
'ObjectIdField', 'ReferenceField', 'ValidationError',
|
'ObjectIdField', 'ReferenceField', 'ValidationError',
|
||||||
'DecimalField', 'URLField', 'GenericReferenceField']
|
'DecimalField', 'URLField', 'GenericReferenceField',
|
||||||
|
'BinaryField']
|
||||||
|
|
||||||
RECURSIVE_REFERENCE_CONSTANT = 'self'
|
RECURSIVE_REFERENCE_CONSTANT = 'self'
|
||||||
|
|
||||||
@ -442,3 +443,23 @@ class GenericReferenceField(BaseField):
|
|||||||
|
|
||||||
def prepare_query_value(self, op, value):
|
def prepare_query_value(self, op, value):
|
||||||
return self.to_mongo(value)['_ref']
|
return self.to_mongo(value)['_ref']
|
||||||
|
|
||||||
|
class BinaryField(BaseField):
|
||||||
|
"""A binary data field.
|
||||||
|
"""
|
||||||
|
|
||||||
|
def __init__(self, max_bytes=None, **kwargs):
|
||||||
|
self.max_bytes = max_bytes
|
||||||
|
super(BinaryField, self).__init__(**kwargs)
|
||||||
|
|
||||||
|
def to_mongo(self, value):
|
||||||
|
return pymongo.binary.Binary(value)
|
||||||
|
|
||||||
|
def to_python(self, value):
|
||||||
|
return str(value)
|
||||||
|
|
||||||
|
def validate(self, value):
|
||||||
|
assert isinstance(value, str)
|
||||||
|
|
||||||
|
if self.max_bytes is not None and len(value) > self.max_bytes:
|
||||||
|
raise ValidationError('Binary value is too long')
|
||||||
|
@ -495,6 +495,61 @@ class FieldTest(unittest.TestCase):
|
|||||||
Post.drop_collection()
|
Post.drop_collection()
|
||||||
User.drop_collection()
|
User.drop_collection()
|
||||||
|
|
||||||
|
def test_binary_fields(self):
|
||||||
|
"""Ensure that binary fields can be stored and retrieved.
|
||||||
|
"""
|
||||||
|
class Attachment(Document):
|
||||||
|
content_type = StringField()
|
||||||
|
blob = BinaryField()
|
||||||
|
|
||||||
|
BLOB = '\xe6\x00\xc4\xff\x07'
|
||||||
|
MIME_TYPE = 'application/octet-stream'
|
||||||
|
|
||||||
|
Attachment.drop_collection()
|
||||||
|
|
||||||
|
attachment = Attachment(content_type=MIME_TYPE, blob=BLOB)
|
||||||
|
attachment.save()
|
||||||
|
|
||||||
|
attachment_1 = Attachment.objects().first()
|
||||||
|
self.assertEqual(MIME_TYPE, attachment_1.content_type)
|
||||||
|
self.assertEqual(BLOB, attachment_1.blob)
|
||||||
|
|
||||||
|
Attachment.drop_collection()
|
||||||
|
|
||||||
|
def test_binary_validation(self):
|
||||||
|
"""Ensure that invalid values cannot be assigned to binary fields.
|
||||||
|
"""
|
||||||
|
class Attachment(Document):
|
||||||
|
blob = BinaryField()
|
||||||
|
|
||||||
|
class AttachmentRequired(Document):
|
||||||
|
blob = BinaryField(required=True)
|
||||||
|
|
||||||
|
class AttachmentSizeLimit(Document):
|
||||||
|
blob = BinaryField(max_bytes=4)
|
||||||
|
|
||||||
|
Attachment.drop_collection()
|
||||||
|
AttachmentRequired.drop_collection()
|
||||||
|
AttachmentSizeLimit.drop_collection()
|
||||||
|
|
||||||
|
attachment = Attachment()
|
||||||
|
attachment.validate()
|
||||||
|
attachment.blob = 2
|
||||||
|
self.assertRaises(ValidationError, attachment.validate)
|
||||||
|
|
||||||
|
attachment_required = AttachmentRequired()
|
||||||
|
self.assertRaises(ValidationError, attachment_required.validate)
|
||||||
|
attachment_required.blob = '\xe6\x00\xc4\xff\x07'
|
||||||
|
attachment_required.validate()
|
||||||
|
|
||||||
|
attachment_size_limit = AttachmentSizeLimit(blob='\xe6\x00\xc4\xff\x07')
|
||||||
|
self.assertRaises(ValidationError, attachment_size_limit.validate)
|
||||||
|
attachment_size_limit.blob = '\xe6\x00\xc4\xff'
|
||||||
|
attachment_size_limit.validate()
|
||||||
|
|
||||||
|
Attachment.drop_collection()
|
||||||
|
AttachmentRequired.drop_collection()
|
||||||
|
AttachmentSizeLimit.drop_collection()
|
||||||
|
|
||||||
if __name__ == '__main__':
|
if __name__ == '__main__':
|
||||||
unittest.main()
|
unittest.main()
|
||||||
|
Loading…
x
Reference in New Issue
Block a user