Page MenuHomePhabricator
Differential D4503

[services] Backup - Fix missing hash in unit tests
ClosedPublic
Actions

Authored by • jakub on Jul 12 2022, 1:18 AM.
Tags
None
Referenced Files
F3386619: D4503.id14405.diff
Fri, Nov 29, 5:23 AM
Unknown Object (File)
Fri, Nov 8, 8:28 PM
Unknown Object (File)
Fri, Nov 8, 8:28 PM
Unknown Object (File)
Fri, Nov 8, 8:10 AM
Unknown Object (File)
Tue, Nov 5, 2:05 AM
Unknown Object (File)
Thu, Oct 31, 5:39 PM
Unknown Object (File)
Oct 22 2024, 10:04 AM
Unknown Object (File)
Oct 16 2024, 2:50 AM
View All Files
Subscribers
• abosh
• adrian
ashoat
atul
• karol
tomek

Details

Reviewers
tomek
• karol
Commits
rCOMMfeb0d5a36ecb: [services] Backup - Fix missing hash in unit tests
Summary

Depends on D4502
Fixing missing hash parameter in LogItem initialisation in generateLogItem function

Test Plan

cd services && yarn run-unit-tests backup

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• jakub created this revision.Jul 12 2022, 1:18 AM
Herald added subscribers: • abosh, • karol, atul and 3 others. · View Herald TranscriptJul 12 2022, 1:18 AM
• jakub added a child revision: D4504: [services] Backup - Remove unnecessary brackets in unit test.Jul 12 2022, 1:23 AM
• jakub edited the summary of this revision. (Show Details)
• jakub added a parent revision: D4502: [services] Backup - Fix unchanged variable names in unit tests scripts.
Harbormaster completed remote builds in B10460: Diff 14405.Jul 12 2022, 1:30 AM
• jakub requested review of this revision.Jul 12 2022, 1:30 AM
• jakub edited the summary of this revision. (Show Details)Jul 12 2022, 1:57 AM
• jakub edited the test plan for this revision. (Show Details)
• jakub edited the test plan for this revision. (Show Details)Jul 12 2022, 2:14 AM
ashoat added a comment.Jul 12 2022, 12:01 PM

Make sure to always add a reviewer when you put your diffs up! Otherwise nobody will look

• jakub added reviewers: tomek, • karol.Jul 14 2022, 2:42 AM
• karol accepted this revision.Jul 18 2022, 1:26 AM
• karol added 1 blocking reviewer(s): tomek.
tomek accepted this revision.Jul 18 2022, 5:35 AM
This revision is now accepted and ready to land.Jul 18 2022, 5:35 AM
Closed by commit rCOMMfeb0d5a36ecb: [services] Backup - Fix missing hash in unit tests (authored by • jakub). · Explain WhyJul 27 2022, 7:35 AM
This revision was automatically updated to reflect the committed changes.
• jakub added a commit: rCOMMfeb0d5a36ecb: [services] Backup - Fix missing hash in unit tests.

Revision Contents
Changeset List

PathSize
services/
backup/
test/
3 lines

Diff 15017

View Options

services/backup/test/DatabaseManagerTest.cpp

Loading...