Return Item even when item is not found.
This commit is contained in:
parent
defd106523
commit
88cd009c4d
@ -275,8 +275,11 @@ class Table(object):
|
|||||||
try:
|
try:
|
||||||
if range_key:
|
if range_key:
|
||||||
return self.items[hash_key][range_key]
|
return self.items[hash_key][range_key]
|
||||||
else:
|
|
||||||
|
if hash_key in self.items:
|
||||||
return self.items[hash_key]
|
return self.items[hash_key]
|
||||||
|
|
||||||
|
raise KeyError
|
||||||
except KeyError:
|
except KeyError:
|
||||||
return None
|
return None
|
||||||
|
|
||||||
@ -525,6 +528,23 @@ class DynamoDBBackend(BaseBackend):
|
|||||||
range_value = None
|
range_value = None
|
||||||
|
|
||||||
item = table.get_item(hash_value, range_value)
|
item = table.get_item(hash_value, range_value)
|
||||||
|
# Update does not fail on new items, so create one
|
||||||
|
if item is None:
|
||||||
|
data = {
|
||||||
|
table.hash_key_attr: {
|
||||||
|
hash_value.type: hash_value.value,
|
||||||
|
},
|
||||||
|
}
|
||||||
|
if range_value:
|
||||||
|
data.update({
|
||||||
|
table.range_key_attr: {
|
||||||
|
range_value.type: range_value.value,
|
||||||
|
}
|
||||||
|
})
|
||||||
|
|
||||||
|
table.put_item(data)
|
||||||
|
item = table.get_item(hash_value, range_value)
|
||||||
|
|
||||||
if update_expression:
|
if update_expression:
|
||||||
item.update(update_expression)
|
item.update(update_expression)
|
||||||
else:
|
else:
|
||||||
|
@ -853,6 +853,41 @@ def test_update_item_range_key_set():
|
|||||||
})
|
})
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
|
@mock_dynamodb2
|
||||||
|
def test_update_item_does_not_exist_is_created():
|
||||||
|
table = _create_table_with_range_key()
|
||||||
|
|
||||||
|
item_key = {'forum_name': 'the-key', 'subject': '123'}
|
||||||
|
table.update_item(
|
||||||
|
Key=item_key,
|
||||||
|
AttributeUpdates={
|
||||||
|
'username': {
|
||||||
|
'Action': u'PUT',
|
||||||
|
'Value': 'johndoe2'
|
||||||
|
},
|
||||||
|
'created': {
|
||||||
|
'Action': u'PUT',
|
||||||
|
'Value': Decimal('4'),
|
||||||
|
},
|
||||||
|
'mapfield': {
|
||||||
|
'Action': u'PUT',
|
||||||
|
'Value': {'key': 'value'},
|
||||||
|
}
|
||||||
|
},
|
||||||
|
)
|
||||||
|
|
||||||
|
returned_item = dict((k, str(v) if isinstance(v, Decimal) else v)
|
||||||
|
for k, v in table.get_item(Key=item_key)['Item'].items())
|
||||||
|
dict(returned_item).should.equal({
|
||||||
|
'username': "johndoe2",
|
||||||
|
'forum_name': 'the-key',
|
||||||
|
'subject': '123',
|
||||||
|
'created': '4',
|
||||||
|
'mapfield': {'key': 'value'},
|
||||||
|
})
|
||||||
|
|
||||||
|
|
||||||
@mock_dynamodb2
|
@mock_dynamodb2
|
||||||
def test_boto3_query_gsi_range_comparison():
|
def test_boto3_query_gsi_range_comparison():
|
||||||
table = _create_table_with_range_key()
|
table = _create_table_with_range_key()
|
||||||
|
Loading…
x
Reference in New Issue
Block a user