Merge pull request #11 from launchdarkly/eb/ch56381/diagnostic

eli-darkly · web-flow · commit 931287575637 · 2020-02-12T16:18:04.000-08:00
add description parameter so diagnostic events will report dataStoreType as "DynamoDB"
diff --git a/dynamodb_feature_store.js b/dynamodb_feature_store.js
@@ -11,7 +11,7 @@ function DynamoDBFeatureStore(tableName, options) {
   if (ttl === null || ttl === undefined) {
     ttl = defaultCacheTTLSeconds;
   }
-  return new CachingStoreWrapper(dynamoDBFeatureStoreInternal(tableName, options), ttl);
+  return new CachingStoreWrapper(dynamoDBFeatureStoreInternal(tableName, options), ttl, 'DynamoDB');
 }
 
 function dynamoDBFeatureStoreInternal(tableName, options) {
diff --git a/package.json b/package.json
@@ -18,6 +18,7 @@
     "eslint-formatter-pretty": "1.3.0",
     "jest": "24.7.1",
     "jest-junit": "6.3.0",
+    "launchdarkly-js-test-helpers": "^1.0.0",
     "launchdarkly-node-server-sdk": ">= 5.8.1",
     "typescript": "3.0.1"
   },

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ function DynamoDBFeatureStore(tableName, options) {`
`11`	`11`	`if (ttl === null \|\| ttl === undefined) {`
`12`	`12`	`ttl = defaultCacheTTLSeconds;`
`13`	`13`	`}`
`14`		`- return new CachingStoreWrapper(dynamoDBFeatureStoreInternal(tableName, options), ttl);`
	`14`	`+ return new CachingStoreWrapper(dynamoDBFeatureStoreInternal(tableName, options), ttl, 'DynamoDB');`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`function dynamoDBFeatureStoreInternal(tableName, options) {`