Page MenuHomePhabricator

[native] Add base64 decode to CommUtilsModule
ClosedPublic

Authored by bartek on May 9 2023, 6:20 AM.
Tags
None
Referenced Files
F3405106: D7760.id26311.diff
Tue, Dec 3, 6:19 PM
F3404263: D7760.diff
Tue, Dec 3, 7:42 AM
F3403199: D7760.id26311.diff
Tue, Dec 3, 2:01 AM
F3403076: D7760.diff
Tue, Dec 3, 12:52 AM
Unknown Object (File)
Mon, Dec 2, 3:00 PM
Unknown Object (File)
Mon, Dec 2, 2:07 AM
Unknown Object (File)
Mon, Dec 2, 1:47 AM
Unknown Object (File)
Mon, Dec 2, 1:46 AM
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
Lint Not Applicable
Unit
Tests Not Applicable

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