Add test case for polluted property to hasOwn suite

ericcornelissen · Nov 19, 2023 · 781c6b2 · 781c6b2
1 parent 14ad5fc
commit 781c6b2
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/src/executables.js b/src/executables.js
@@ -42,8 +42,8 @@ export function resolveExecutable(
     const path = hasOwn(env, "PATH")
       ? env.PATH
       : hasOwn(env, "Path")
-      ? env.Path
-      : undefined;
+        ? env.Path
+        : undefined;
     resolved = which(resolved, { path });
   } catch (_) {
     throw new Error(notFoundError(executable));

diff --git a/test/unit/reflection/has-own.test.js b/test/unit/reflection/has-own.test.js
@@ -25,3 +25,15 @@ testProp(
     t.is(actual, expected);
   },
 );
+
+testProp(
+  "polluted",
+  [fc.object(), fc.string(), fc.string()],
+  (t, object, property, value) => {
+    object = Object.assign(Object.create({ [property]: value }), object);
+
+    const actual = hasOwn(object, property);
+    const expected = Object.hasOwn(object, property);
+    t.is(actual, expected);
+  },
+);