Page MenuHomePhabricator
Differential D8749

[Identity/Tunnelbroker] Add integration tests
ClosedPublic
Actions

Authored by • jon on Aug 7 2023, 7:54 AM.
Tags
None
Referenced Files
F5902573: D8749.id30367.diff
Thu, Apr 17, 1:04 PM
F5898301: D8749.id30123.diff
Thu, Apr 17, 11:42 AM
F5894136: D8749.id30338.diff
Thu, Apr 17, 10:05 AM
F5891557: D8749.id.diff
Thu, Apr 17, 7:39 AM
Unknown Object (File)
Thu, Apr 17, 3:38 AM
Unknown Object (File)
Wed, Apr 16, 8:50 PM
Unknown Object (File)
Tue, Apr 15, 10:55 PM
Unknown Object (File)
Sat, Mar 29, 8:44 AM
View All Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
varun
Commits
rCOMM775d454deda1: [Identity/Tunnelbroker] Add integration tests
Summary

To make the review not as extensive, I split up the integration
tests into its own diff.

Follow up for: https://linear.app/comm/issue/ENG-4446

Depends on D8748

Test Plan
# Start localstack and rabbitmq
comm-dev services start

(cd services/identity && cargo run -- server)
(cd services/tunnelbroker && cargo run)

cd services/commtest
cargo test --test identity_tunnelbroker_tests

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• jon created this revision.Aug 7 2023, 7:54 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 7 2023, 7:54 AM
• jon added a child revision: D8750: [Keyserver/rust] Implement upload_one_time_keys to identity service.Aug 7 2023, 8:03 AM
Harbormaster completed remote builds in B21564: Diff 29585.Aug 7 2023, 8:11 AM
• jon requested review of this revision.Aug 7 2023, 8:11 AM
• jon removed a child revision: D8750: [Keyserver/rust] Implement upload_one_time_keys to identity service.Aug 8 2023, 8:54 AM
• jon added a child revision: D8816: [Native] Encode platform type in native_rust_library.Aug 15 2023, 9:33 AM
bartek accepted this revision.Aug 16 2023, 5:19 AM
This revision is now accepted and ready to land.Aug 16 2023, 5:19 AM
• jon updated this revision to Diff 30032.Aug 17 2023, 10:25 AM

Rebase on master

Harbormaster completed remote builds in B21858: Diff 30032.Aug 17 2023, 10:46 AM
• jon updated this revision to Diff 30123.Aug 21 2023, 12:06 AM

Use ConnectionInitializationMessage as message name

Harbormaster completed remote builds in B21931: Diff 30123.Aug 21 2023, 12:25 AM
• jon updated this revision to Diff 30338.Aug 25 2023, 4:52 PM

Rebase on top of master

Harbormaster completed remote builds in B22104: Diff 30338.Aug 25 2023, 5:11 PM
Closed by commit rCOMM775d454deda1: [Identity/Tunnelbroker] Add integration tests (authored by • jon). · Explain WhyAug 27 2023, 7:51 PM
This revision was automatically updated to reflect the committed changes.
• jon added a commit: rCOMM775d454deda1: [Identity/Tunnelbroker] Add integration tests.
kamil added a child revision: D8750: [Keyserver/rust] Implement upload_one_time_keys to identity service.Sep 8 2023, 7:28 AM
kamil removed a child revision: D8750: [Keyserver/rust] Implement upload_one_time_keys to identity service.Sep 8 2023, 7:29 AM
kamil added a child revision: D8750: [Keyserver/rust] Implement upload_one_time_keys to identity service.Sep 12 2023, 3:17 AM

Revision Contents
Changeset List

PathSize
services/
commtest/
tests/
110 lines
tunnelbroker/
src/
2 lines
shared/
tunnelbroker_messages/
src/
messages/
2 lines
9 lines

Diff 30367

View Options

services/commtest/tests/identity_tunnelbroker_tests.rs

Loading...
View Options

services/tunnelbroker/src/config.rs

Loading...
View Options

shared/tunnelbroker_messages/src/messages/mod.rs

Loading...
View Options

shared/tunnelbroker_messages/src/messages/session.rs

Loading...