Paths

Table of Contentst

Differential D4543

[services] Backup - Add unit test for getUtf8Length
AbandonedPublic
Actions

Authored by • jakub on Jul 15 2022, 8:55 AM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

tomek
• karol

Summary

Depends on D4544

Add unit test for function for getting proper length of string encoded in utf8 multibyte format.

Test Plan

cd services && yarn run-unit-tests backup

Diff Detail

Repository

Branch

unittest

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• jakub created this revision.Jul 15 2022, 8:55 AM

Herald added subscribers: • abosh, • karol, atul and 3 others. · View Herald TranscriptJul 15 2022, 8:55 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 15 2022, 8:57 AM

Harbormaster failed remote builds in B10548: Diff 14513!

• jakub edited the summary of this revision. (Show Details)Jul 15 2022, 10:02 AM

• jakub edited the summary of this revision. (Show Details)

• jakub added a parent revision: D4544: [services] Backup - Add function to get utf8 string length.

• jakub added reviewers: tomek, • karol.Jul 15 2022, 10:23 AM

Adding missing <algorithm> library

• jakub edited the summary of this revision. (Show Details)Jul 15 2022, 12:06 PM

• jakub edited the test plan for this revision. (Show Details)

Harbormaster completed remote builds in B10553: Diff 14521.Jul 15 2022, 12:17 PM

• jakub requested review of this revision.Jul 15 2022, 12:17 PM

Can you make sure that lines don't exceed 80 chars?

• karol requested changes to this revision.Jul 18 2022, 9:08 AM

• karol added inline comments.

services/backup/test/ToolsTest.cpp
113 ↗	(On Diff #14521)	I think we do not want to use random data for tests. Tests should be constructed so we can easily repeat them. I think the data sets should be just hardcoded.
148 ↗	(On Diff #14521)	please, add a new line

This revision now requires changes to proceed.Jul 18 2022, 9:08 AM

Replacing random data sets with already generated examples

Harbormaster failed remote builds in B10892: Diff 14979!Jul 26 2022, 2:18 PM

• karol accepted this revision.Jul 27 2022, 11:47 AM

This revision is now accepted and ready to land.Jul 27 2022, 11:47 AM

tomek requested changes to this revision.Jul 27 2022, 12:06 PM

tomek added inline comments.

services/backup/test/ToolsTest.cpp
24–26	What is the reason for putting the last two chars in their own lines? I guess the string can fit in one line.
83	Doesn't matter that much, but we don't need to copy the data during iteration

This revision now requires changes to proceed.Jul 27 2022, 12:06 PM

Abandoning, explained in previous diff D4544

Revision Contents
Changeset List

Path

Size

services/

backup/

test/

86 lines

Diff 14979

services/backup/test/ToolsTest.cpp

Loading...