Support symbol properties in contain()

Author: kanongil

lib/index.js

@@ -336,7 +336,8 @@ exports.contain = function (ref, values, options) {
         !Array.isArray(values)) {
 
         valuePairs = values;
-        values = Object.keys(values);
+        const symbols = Object.getOwnPropertySymbols(values).filter(Object.prototype.propertyIsEnumerable.bind(values));
+        values = [...Object.keys(values), ...symbols];
     }
     else {
         values = [].concat(values);
@@ -411,7 +412,7 @@ exports.contain = function (ref, values, options) {
         }
     }
     else {
-        const keys = Object.getOwnPropertyNames(ref);
+        const keys = Reflect.ownKeys(ref);
         for (let i = 0; i < keys.length; ++i) {
             const key = keys[i];
             const pos = values.indexOf(key);

test/index.js

@@ -1754,6 +1754,18 @@ describe('contain()', () => {
             expect(Hoek.contain(foo, { 'a': 1, 'b': 2, 'c': 3 }, { only: true })).to.be.true();
         }
     });
+
+    it('supports symbols', () => {
+
+        const sym = Symbol();
+
+        expect(Hoek.contain([sym], sym)).to.be.true();
+        expect(Hoek.contain({ [sym]: 1 }, sym)).to.be.true();
+        expect(Hoek.contain({ [sym]: 1, a: 2 }, { [sym]: 1 })).to.be.true();
+
+        expect(Hoek.contain([sym], Symbol())).to.be.false();
+        expect(Hoek.contain({ [sym]: 1 }, Symbol())).to.be.false();
+    });
 });
 
 describe('flatten()', () => {