Paths

Table of Contentst

Differential D11982

[commtest] Add test for device list signing
ClosedPublic
Actions

Authored by bartek on May 10 2024, 10:35 AM.

Details

Reviewers

will
kamil

Commits

rCOMMef0d1382b0b1: [commtest] Add test for device list signing

Summary

Added integration test to commtest that verifies device list signature verification in Identity UpdateDeviceList RPC.

Also did some small refactors to existing tests to reuse some code.

Depends on D11959

Test Plan

Commtest

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.May 10 2024, 10:35 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMay 10 2024, 10:35 AM

Harbormaster completed remote builds in B28828: Diff 40063.May 10 2024, 10:54 AM

bartek published this revision for review.May 10 2024, 11:40 AM

bartek added inline comments.

services/commtest/tests/identity_device_list_tests.rs
213 ↗	(On Diff #40063)

kamil accepted this revision.May 10 2024, 12:20 PM

This revision is now accepted and ready to land.May 10 2024, 12:20 PM

Fix typo, rebase before landing

Harbormaster completed remote builds in B28833: Diff 40068.May 10 2024, 1:59 PM

Closed by commit rCOMMef0d1382b0b1: [commtest] Add test for device list signing. · Explain WhyMay 10 2024, 2:24 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMef0d1382b0b1: [commtest] Add test for device list signing.

Revision Contents
Changeset List

Path

Size

services/

commtest/

src/

identity/

olm_account_infos.rs

6 lines

tests/

identity_device_list_tests.rs

210 lines

Diff 40073

View Options

services/commtest/src/identity/olm_account_infos.rs