Page MenuHomePhabricator

[native] Add base64 decode to CommUtilsModule
ClosedPublic

Authored by bartek on May 9 2023, 6:20 AM.
Tags
None
Referenced Files
F3403076: D7760.diff
Tue, Dec 3, 12:52 AM
F3402042: D7760.diff
Mon, Dec 2, 3:00 PM
F3399158: D7760.id26579.diff
Mon, Dec 2, 2:07 AM
F3399100: D7760.id26300.diff
Mon, Dec 2, 1:47 AM
F3399094: D7760.id26595.diff
Mon, Dec 2, 1:46 AM
Unknown Object (File)
Sat, Nov 23, 11:26 PM
Unknown Object (File)
Mon, Nov 4, 12:23 PM
Unknown Object (File)
Mon, Nov 4, 12:22 PM
Subscribers

Details

Summary

This adds JSI bindings for D7758 to the native CommUtilsModule.

Depends on D7758.

Test Plan

Encoded and decoded a Uint8Array into base64, using commCoreModule.base64{Encode,Decode}buffer() and console-log.

Used three different length of Uint8Arrays to test all padding values, e.g.:

new Uint8Array([1,2,3,4,5,6]);  // AQIDBAUG
new Uint8Array([1,2,3,4,5,6,7]); // AQIDBAUGBw==
new Uint8Array([1,2,3,4,5,6,7,8]); // AQIDBAUGBwg=

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.May 9 2023, 8:15 AM

Could you make sure in Test Plan that you tested decoding strings with each length of padding at the end (none, =, and ==)?

This revision is now accepted and ready to land.May 15 2023, 2:21 AM