From cb36a8b10fa58710c88ca441e91a7923472b94c0 Mon Sep 17 00:00:00 2001 From: Victor Blaga Date: Tue, 21 Jun 2016 17:46:37 +0200 Subject: [PATCH] Fix dynamodb_v2 update_item tests Update expression has the wrong syntax. Something like :bar denotes a placeholder which has to be filled by supplying an additional ExpressionAttributeNames --- tests/test_dynamodb2/test_dynamodb_table_without_range_key.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/test_dynamodb2/test_dynamodb_table_without_range_key.py b/tests/test_dynamodb2/test_dynamodb_table_without_range_key.py index c17d6683..5a32c6b2 100644 --- a/tests/test_dynamodb2/test_dynamodb_table_without_range_key.py +++ b/tests/test_dynamodb2/test_dynamodb_table_without_range_key.py @@ -435,7 +435,7 @@ def test_update_item_remove(): } # Then remove the SentBy field - conn.update_item("messages", key_map, update_expression="REMOVE :SentBy, :SentTo") + conn.update_item("messages", key_map, update_expression="REMOVE SentBy, SentTo") returned_item = table.get_item(username="steve") dict(returned_item).should.equal({ @@ -459,7 +459,7 @@ def test_update_item_set(): 'username': {"S": "steve"} } - conn.update_item("messages", key_map, update_expression="SET foo=:bar, blah=:baz REMOVE :SentBy") + conn.update_item("messages", key_map, update_expression="SET foo=bar, blah=baz REMOVE SentBy") returned_item = table.get_item(username="steve") dict(returned_item).should.equal({