fix(utils): default predicate was checking for key instead of value

Author: saneef

lib/utils/object.js

@@ -7,7 +7,7 @@
  * @param      {(key: string, value: any, obj: object ) => boolean | Array<string>}  keys    The keys
  * @return     {object}  Copy of the object with filtered values
  */
-function pick(obj, keys = (k) => Boolean(k)) {
+function pick(obj, keys = (_k, v) => Boolean(v)) {
   if (Array.isArray(keys)) {
     return pick(obj, (key) => keys.includes(key));
   }

tests/utils-object.spec.js

@@ -1,14 +1,14 @@
 const test = require("ava").default;
 const { pick, pickByKeyPrefix, mapKeys } = require("../lib/utils/object.js");
 
-test("pick: should pick values based on truthy", async (t) => {
-  t.deepEqual(pick({ a: "a", b: "b", c: false }, ["a", "b"]), {
+test("pick: should pick properites based on truthy values", async (t) => {
+  t.deepEqual(pick({ a: "a", b: "b", c: false }), {
     a: "a",
     b: "b",
   });
 });
 
-test("pick: should pick values based on predicate", async (t) => {
+test("pick: should pick properites based on predicate", async (t) => {
   t.deepEqual(
     pick({ a: "a", b: "b", c: false }, (_k, v) => !v),
     {
@@ -17,11 +17,11 @@ test("pick: should pick values based on predicate", async (t) => {
   );
 });
 
-test("pick: should pick values from array of keys", async (t) => {
+test("pick: should pick properites using array of keys", async (t) => {
   t.deepEqual(pick({ a: "a", b: "b", c: "c" }, ["a", "b"]), { a: "a", b: "b" });
 });
 
-test("pickByKeyPrefix: should pick values based on key prefix", async (t) => {
+test("pickByKeyPrefix: should pick properites based on key prefix", async (t) => {
   t.deepEqual(pickByKeyPrefix({ aKey: "a", anotherKey: "b", c: "c" }, "a"), {
     aKey: "a",
     anotherKey: "b",