bfj 6.1.1 → 6.1.2

package/HISTORY.md

@@ -1,5 +1,15 @@
 # History
 
+## 6.1.2
+
+### Bug fixes
+
+* eventify: escape object keys (910ad08)
+
+### Other changes
+
+* package: update deps (aafb4ff)
+
 ## 6.1.1
 
 ### Bug fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bfj",
-  "version": "6.1.1",
+  "version": "6.1.2",
   "description": "Big-friendly JSON. Asynchronous streaming functions for large JSON data sets.",
   "homepage": "https://gitlab.com/philbooth/bfj",
   "bugs": "https://gitlab.com/philbooth/bfj/issues",
@@ -29,19 +29,19 @@
     "node": ">= 6.0.0"
   },
   "dependencies": {
-    "bluebird": "^3.5.1",
-    "check-types": "^7.3.0",
-    "hoopy": "^0.1.2",
-    "tryer": "^1.0.0"
+    "bluebird": "^3.5.5",
+    "check-types": "^8.0.3",
+    "hoopy": "^0.1.4",
+    "tryer": "^1.0.1"
   },
   "devDependencies": {
-    "chai": "4.1.x",
-    "eslint": "4.19.x",
-    "mocha": "5.0.x",
-    "please-release-me": "^2.0.2",
-    "proxyquire": "1.8.x",
-    "request": "2.85.x",
-    "spooks": "2.0.x"
+    "chai": "^4.2.0",
+    "eslint": "^6.0.1",
+    "mocha": "^6.1.4",
+    "please-release-me": "^2.1.2",
+    "proxyquire": "^2.1.0",
+    "request": "^2.88.0",
+    "spooks": "^2.0.0"
   },
   "scripts": {
     "lint": "eslint src",

package/src/eventify.js

@@ -297,7 +297,7 @@ function eventify (data, options = {}) {
         return Promise.resolve()
       }
 
-      return emit(events.property, key)
+      return emit(events.property, escapeString(key))
         .then(() => proceed(item))
     })
   }

package/test/unit/eventify.js

@@ -1052,6 +1052,61 @@ suite('eventify:', () => {
       })
     })
 
+    suite('object with keys containing special characters:', () => {
+      setup(done => {
+        const emitter = eventify({ 'foo\n"bar"': 42 })
+
+        Object.keys(events).forEach(key => {
+          emitter.on(events[key], spooks.fn({
+            name: key,
+            log: log
+          }))
+        })
+
+        emitter.on(events.end, done)
+      })
+
+      test('object event occurred once', () => {
+        assert.strictEqual(log.counts.object, 1)
+      })
+
+      test('property event occurred once', () => {
+        assert.strictEqual(log.counts.property, 1)
+      })
+
+      test('property event was dispatched correctly', () => {
+        assert.strictEqual(log.args.property[0][0], 'foo\\n\\"bar\\"')
+      })
+
+      test('number event occurred once', () => {
+        assert.strictEqual(log.counts.number, 1)
+      })
+
+      test('number event was dispatched correctly', () => {
+        assert.strictEqual(log.args.number[0][0], 42)
+      })
+
+      test('endObject event occurred once', () => {
+        assert.strictEqual(log.counts.endObject, 1)
+      })
+
+      test('end event occurred once', () => {
+        assert.strictEqual(log.counts.end, 1)
+      })
+
+      test('literal event did not occur', () => {
+        assert.strictEqual(log.counts.literal, 0)
+      })
+
+      test('error event did not occur', () => {
+        assert.strictEqual(log.counts.error, 0)
+      })
+
+      test('dataError event did not occur', () => {
+        assert.strictEqual(log.counts.dataError, 0)
+      })
+    })
+
     suite('nested array:', () => {
       setup(done => {
         const emitter = eventify([ 'foo', [ 'bar', [ 'baz', 'qux' ] ] ])