Page MenuHomePhabricator
Differential D9200

[Tunnelbroker] add authentication tests
ClosedPublic
Actions

Authored by kamil on Sep 14 2023, 3:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 24, 6:13 PM
Unknown Object (File)
Fri, Jan 24, 6:13 PM
Unknown Object (File)
Fri, Jan 24, 6:13 PM
Unknown Object (File)
Fri, Jan 24, 6:12 PM
Unknown Object (File)
Fri, Jan 24, 6:00 PM
Unknown Object (File)
Tue, Jan 21, 10:20 PM
Unknown Object (File)
Fri, Jan 17, 7:30 PM
Unknown Object (File)
Fri, Jan 17, 7:01 PM
View All Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
michal
varun
• jon
Commits
rCOMMb3c4cff3502d: [Tunnelbroker] add authentication tests
Summary

I implemented those to make sure D8918 works in both cases, so publishing, maybe will be helpful.

Depends on D8918

Test Plan
  • Start identity service, tunnelbroker
cd services/commtest
cargo test --test identity_tunnelbroker_tests

Diff Detail

Repository
rCOMM Comm
Branch
land-tb-auth
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Sep 14 2023, 3:10 AM
kamil held this revision as a draft.
Herald added a reviewer: • jon. · View Herald TranscriptSep 14 2023, 3:10 AM
Herald added subscribers: tomek, ashoat. · View Herald Transcript
kamil published this revision for review.Sep 14 2023, 3:20 AM
Harbormaster completed remote builds in B22626: Diff 31123.Sep 14 2023, 3:28 AM
bartek accepted this revision.Sep 14 2023, 3:39 AM
bartek added inline comments.
services/commtest/tests/identity_tunnelbroker_tests.rs
45 ↗(On Diff #31123)

Nit: Are these unwraps hiding some meaningful error? It's usually a good practice to use expect("error message") instead. This way if this fails, we at least know what exactly failed. Otherwise, error messages are rather vague.

This revision is now accepted and ready to land.Sep 14 2023, 3:39 AM
michal accepted this revision.Sep 14 2023, 3:41 AM
kamil updated this revision to Diff 31127.Sep 14 2023, 3:57 AM

unwrap -> expect

Harbormaster completed remote builds in B22630: Diff 31127.Sep 14 2023, 4:16 AM
Closed by commit rCOMMb3c4cff3502d: [Tunnelbroker] add authentication tests (authored by kamil). · Explain WhySep 14 2023, 5:23 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMb3c4cff3502d: [Tunnelbroker] add authentication tests.

Revision Contents
Changeset List

PathSize
services/
commtest/
tests/
66 lines

Diff 31127

View Options

services/commtest/tests/identity_tunnelbroker_tests.rs

Loading...