fix key prefix logic (#5)

Author: eli-darkly

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All notable changes to the LaunchDarkly Node.js SDK DynamoDB integration will be documented in this file. This project adheres to [Semantic Versioning](http://semver.org).
 
+## [1.1.2] - 2019-01-16
+### Fixed:
+- The prefix property added in 1.1.0 did not work correctly: data was written with prefixed keys, but was read with non-prefixed keys. This has been fixed.
+
 ## [1.1.1] - 2019-01-14
 ### Fixed:
 - Fixed a potential race condition that could occur if one process is reading a feature flag while another one is updating the entire data set.

dynamodb_feature_store.js

@@ -194,7 +194,7 @@ function dynamoDBFeatureStoreInternal(tableName, options) {
 
   function marshalItem(kind, item) {
     return {
-      namespace: kind.namespace,
+      namespace: namespaceForKind(kind),
       key: item.key,
       version: item.version,
       item: JSON.stringify(item)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ldclient-node-dynamodb-store",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "DynamoDB-backed feature store for the LaunchDarkly Node.js SDK",
   "main": "dynamodb_feature_store.js",
   "license": "Apache-2.0",
@@ -18,7 +18,7 @@
     "eslint-formatter-pretty": "1.3.0",
     "jest": "23.6.0",
     "jest-junit": "5.2.0",
-    "ldclient-node": ">= 5.7.0",
+    "ldclient-node": ">= 5.7.1",
     "typescript": "3.0.1"
   },
   "jest": {

tests/dynamodb_feature_store-test.js

@@ -90,6 +90,10 @@ describe('DynamoDBFeatureStore', function() {
     return new DynamoDBFeatureStore(table, {cacheTTL: 0});
   }
 
+  function makeStoreWithPrefix(prefix) {
+    return new DynamoDBFeatureStore(table, {prefix: prefix, cacheTTL: 0});
+  }
+
   function makeStoreWithHook(hook) {
     var store = makeStore();
     store.underlyingStore.testUpdateHook = hook;
@@ -101,7 +105,7 @@ describe('DynamoDBFeatureStore', function() {
   });
 
   describe('uncached', function() {
-    testBase.baseFeatureStoreTests(makeStoreWithoutCache, clearTable, false);
+    testBase.baseFeatureStoreTests(makeStoreWithoutCache, clearTable, false, makeStoreWithPrefix);
   });
 
   testBase.concurrentModificationTests(makeStore, makeStoreWithHook);