fix: Added support for gawking objects with symbol keys.

fix(gulp): Fixed lint task to stop when error occurs.
chore(lint): Set no-debugger rule to warning.
chore: Updated npm deps.

.eslintrc

@@ -27,6 +27,7 @@
         "chai-expect/missing-assertion": "error",
         "chai-expect/terminating-properties": "warn",
         "curly": [ "error", "all" ],
+        "no-debugger": "warn",
         "eol-last": "error",
         "indent": [ "error", "tab", { "SwitchCase": 1 } ],
         "keyword-spacing": [ "error" ],

gulpfile.js

@@ -26,7 +26,7 @@ exports.clean = parallel(cleanCoverage, cleanDist, cleanDocs);
 /*
  * lint tasks
  */
-async function lint(pattern) {
+function lint(pattern) {
 	return gulp.src(pattern)
 		.pipe($.plumber())
 		.pipe($.eslint())

package.json

@@ -53,7 +53,7 @@
     "gulp-load-plugins": "^1.5.0",
     "gulp-plumber": "^1.2.1",
     "gulp-sourcemaps": "^2.6.4",
-    "mocha": "^5.2.0",
+    "mocha": "^6.0.2",
     "nyc": "^13.1.0",
     "sinon": "^7.2.2",
     "sinon-chai": "^3.3.0"

src/index.js

@@ -184,7 +184,7 @@ export default function gawk(value, parent) {
 		});
 
 		// gawk any object properties
-		for (const key of Object.getOwnPropertyNames(gawked)) {
+		for (const key of Reflect.ownKeys(gawked)) {
 			if (key !== '__gawk__' && gawked[key] && typeof gawked[key] === 'object') {
 				gawked[key] = gawk(gawked[key], gawked);
 			}

test/test-object.js

@@ -108,6 +108,21 @@ describe('gawk() object', () => {
 		expect(isGawked(gobj)).to.be.true;
 	});
 
+	it('should gawk object with symbol keys', () => {
+		const s = Symbol();
+		const gobj = gawk({
+			foo: {
+				name: 'foo!'
+			},
+			[s]: {
+				name: 'symbol!'
+			}
+		});
+		expect(isGawked(gobj)).to.be.true;
+		expect(gobj.foo).to.deep.equal({ name: 'foo!' });
+		expect(gobj[s]).to.deep.equal({ name: 'symbol!' });
+	});
+
 	it('should not allow __gawk__ to be set', () => {
 		const gobj = gawk({});
 		expect(isGawked(gobj)).to.be.true;

test/test-watch.js

@@ -669,4 +669,25 @@ describe('gawk.unwatch()', () => {
 		expect(callback).to.be.calledOnce;
 		expect(gobj.foo).to.equal('BAZ');
 	});
+
+	it('should notify when a symbol property value changes', () => {
+		const s = Symbol();
+		const gobj = gawk({
+			foo: {
+				name: 'foo'
+			},
+			[s]: {
+				name: 'symbol'
+			}
+		});
+		const callback = spy();
+
+		gawk.watch(gobj, callback);
+
+		gobj.foo.name = 'foo!';
+		expect(callback).to.be.calledOnce;
+
+		gobj[s].name = 'symbol!';
+		expect(callback).to.be.calledTwice;
+	});
 });