Page MenuHomePhabricator
Differential D9677

[lib] Add some unit tests for `invertObjectToMap<K, V>(obj: { [K]: V })`
ClosedPublic
Actions

Authored by atul on Nov 2 2023, 11:57 AM.
Tags
None
Referenced Files
F2209537: D9677.id32641.diff
Sun, Jul 7, 5:41 PM
Unknown Object (File)
Tue, Jul 2, 4:38 PM
Unknown Object (File)
Mon, Jul 1, 11:38 AM
Unknown Object (File)
Sun, Jun 30, 3:18 AM
Unknown Object (File)
Sat, Jun 29, 12:37 PM
Unknown Object (File)
Wed, Jun 26, 1:01 PM
Unknown Object (File)
Fri, Jun 14, 12:39 PM
Unknown Object (File)
Thu, Jun 13, 8:31 PM
View All 39 Files
Subscribers
will

Details

Reviewers
ashoat
ginsu
tomek
rohan
Commits
rCOMM0eb7a828a7d8: [lib] Add some unit tests for `invertObjectToMap<K, V>(obj: { [K]: V })`
Summary

Generated some basic unit tests for invertObjectToMap via GitHub Copilot.

One of the generated tests failed and surfaced possiblity that object entries could be "lost" if there were duplicate values... which is very often a possiblity. Added invariant to ensure values are unique and no entries are lost. Should be O(1) check so not worried about overhead.

Test Plan

Unit tests pass, made sure we had a test for BigInt values.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
utils/
4 lines
54 lines

Diff 32655

View Options

lib/utils/objects.js

Loading...
View Options

lib/utils/objects.test.js

Loading...