Skip to content

Commit 5eb918d

Browse files
committed
remove the extra log and update README.md
1 parent b5e1bec commit 5eb918d

File tree

4 files changed

+12
-11
lines changed

4 files changed

+12
-11
lines changed

README.md

+1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ A small library providing the solution to generate DynamoDB update expression by
55

66
## Release History
77

8+
* 0.1.18 remove extra logs from .17
89
* 0.1.17 Fix issue #7
910
* 0.1.16 Fix issue #6
1011
* 0.1.15 updated the dependencies library, especially there is a fix for array object in deepmerge module

index.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -442,10 +442,10 @@ exports.generateUpdateExpression = function (original, updates, options) {
442442
if(options.arrayMerge && options.arrayMerge === "replaceMerge") {
443443
emptyArrays(original);
444444
delete options.arrayMerge;
445-
console.log(original);
446445
}
447446

448447
var merged = merge(original, updates);
448+
449449
return updateExpressionGenerator(deepDiffMapper.map(
450450
original, merged
451451
), options, null);

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "dynamodb-update-expression",
3-
"version": "0.1.17",
3+
"version": "0.1.18",
44
"description": "A small library providing the solution to return the update expression.",
55
"main": "index.js",
66
"scripts": {

test.js

+9-9
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,12 @@ describe('update expression with options', function () {
142142
"family": [
143143
{
144144
"id": 1,
145-
"role": "father"
146-
}
145+
"role": "brother"
146+
},
147+
{
148+
"id": 2,
149+
"role": "mother"
150+
} // Original will be REPLACED by this (because of: deepmerge library bug)
147151
],
148152
"profile": {
149153
"jobTitle": "Manager",
@@ -166,12 +170,8 @@ describe('update expression with options', function () {
166170
"family": [
167171
{
168172
"id": 1,
169-
"role": "brother"
170-
},
171-
{
172-
"id": 2,
173-
"role": "mother"
174-
} // Original will be REPLACED by this (because of: deepmerge library bug)
173+
"role": "father"
174+
}
175175
],
176176
// Nested Object
177177
"profile": {
@@ -210,7 +210,7 @@ describe('update expression with options', function () {
210210
.be.equal(2);
211211
test.object(result.ExpressionAttributeValues[":family"]).isArray();
212212
test.should(result.ExpressionAttributeValues[":family"][0].role)
213-
.be.equal('brother');
213+
.be.equal('father');
214214
done();
215215
});
216216

0 commit comments

Comments
 (0)