[commtest] Add test for UpdateDeviceList RPC
ClosedPublic
Actions

Authored by bartek on Jan 24 2024, 2:06 AM.

Details

Reviewers

varun
will
michal

Commits

rCOMMee22cca2ef6c: [commtest] Add test for UpdateDeviceList RPC

Summary

Added integration test that tests D10799.
Reorganized some code to be reusable.

Depends on D10799

Test Plan

Ran both device list integration tests.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Jan 24 2024, 2:06 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJan 24 2024, 2:06 AM

Harbormaster completed remote builds in B26121: Diff 36092.Jan 24 2024, 2:35 AM

bartek published this revision for review.Jan 29 2024, 4:31 AM

bartek added a child revision: D10861: [identity] Add RPC for secondary device login.Jan 29 2024, 4:39 AM

varun accepted this revision.Jan 30 2024, 8:17 PM

This revision is now accepted and ready to land.Jan 30 2024, 8:17 PM

Rebase on review feedback from parent diffs

Harbormaster failed remote builds in B26484: Diff 36608!Feb 5 2024, 4:04 AM

Fix payload

Harbormaster completed remote builds in B26486: Diff 36610.Feb 5 2024, 4:49 AM

Closed by commit rCOMMee22cca2ef6c: [commtest] Add test for UpdateDeviceList RPC (authored by bartek). · Explain WhyFeb 12 2024, 1:26 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMee22cca2ef6c: [commtest] Add test for UpdateDeviceList RPC.

Revision Contents
Changeset List

Path

Size

services/

commtest/

tests/

identity_device_list_tests.rs

136 lines

Diff 36610

View Options

services/commtest/tests/identity_device_list_tests.rs

[commtest] Add test for UpdateDeviceList RPCClosedPublicActions