fix: pick method crash when object has hasOwnProperty key

[wschurman]

Why

pick was calling object.hasOwnProperty(prop), but if the object had a property hasOwnProperty, it'd be used.

How

Fix is to use the Object prototype's method.

Test Plan

Run added test.

[wschurman]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• fix: correctness of TSMockitoExtensions isEqualWithEntityAware for SerializableKeyMaps of different length #614
• fix: fix possible composite key collision in StubDatabaseAdapter test utility #613
• chore: update comment for IEntityMetricsAdapter for implementors #612
• fix: pick method crash when object has hasOwnProperty key #611 👈 (View in Graphite)
• fix: fix behavior of AuthorizationResultBasedEntityAssociationLoader when field value is empty string #610
• feat: support numeric enums in StrictEnumField #609
• feat: validate GenericRedisCacheContext #608
• fix: skip when AllowIfAllSubRulesAllowPrivacyPolicyRule is empty #607
• fix: correct cycle guard behavior for deletion #606
• fix: handle potential infinite recursion for canViewerDeleteAsync #605
• feat: clean up transactional dataloader at transaction end #604
• fix: correct negative caching behavior in GenericSecondaryEntityCache #603
• fix: update cache key generation for secondary caches #602
• fix!: replace makeKeyFn with delimiter to strengthen guarantee of proper escaping #601
• fix: prevent collision in companion when entities share same name or same table name #600
• chore: remove unnecessary ViewerScopedEntityMutatorFactory #599
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (99daddc) to head (34de712).

Additional details and impacted files

@@                                                             Coverage Diff                                                              @@
##           05-14-fix_fix_behavior_of_authorizationresultbasedentityassociationloader_when_field_value_is_empty_string      #611   +/-   ##
============================================================================================================================================
  Coverage                                                                                                      100.00%   100.00%           
============================================================================================================================================
  Files                                                                                                             109       109           
  Lines                                                                                                           17738     17738           
  Branches                                                                                                         1539      1539           
============================================================================================================================================
  Hits                                                                                                            17738     17738           

Flag	Coverage Δ
integration	28.62% <0.00%> (ø)
unittest	94.53% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.