-
Notifications
You must be signed in to change notification settings - Fork 12
Open
Labels
Description
Ever since #312 was merged (determined by bisecting) cargo test prints confusing errors to the log:
$ cargo test
Compiling drawbridge-type v0.2.2 (/home/rvolosatovs/src/github.com/profianinc/drawbridge/crates/type)
Compiling drawbridge-server v0.2.2 (/home/rvolosatovs/src/github.com/profianinc/drawbridge/crates/server)
Compiling drawbridge-client v0.2.4 (/home/rvolosatovs/src/github.com/profianinc/drawbridge/crates/client)
Compiling drawbridge v0.2.3 (/home/rvolosatovs/src/github.com/profianinc/drawbridge)
Finished test [unoptimized + debuginfo] target(s) in 10.20s
Running unittests src/lib.rs (target/debug/deps/drawbridge-14d727548dbf868b)
running 0 tests
test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
Running unittests src/main.rs (target/debug/deps/drawbridge-d1d50076176f8765)
running 0 tests
test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
Running tests/mod.rs (target/debug/deps/mod-464ee95f1093a176)
running 1 test
2022-11-17T19:36:32.048772Z ERROR app::auth::oidc: failed to verify token error=Error decoding token
Caused by:
Base64 error: Encoded text cannot have a 6-bit remainder.
2022-11-17T19:36:32.094596Z ERROR app::auth::oidc: failed to verify token error=Error decoding token
Caused by:
InvalidAudience
2022-11-17T19:36:32.141511Z ERROR app::auth::oidc: failed to verify token error=Error decoding token
Caused by:
ExpiredSignature
2022-11-17T19:36:32.287258Z ERROR app::auth::oidc: failed to verify token error=Error decoding token
Caused by:
InvalidIssuer
test app ... ok
test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 5.88s
Doc-tests drawbridge
running 0 tests
test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
The only reasonable solution seems to be disabling server-side logging, which in turn will complicate debugging.
If the test case was isolated, we could disable logging only for that specific test case.
I'd propose splitting the invalid token tests into a separate auth unit test and disable server-side logging for that specific test case as part of the effort.
Metadata
Metadata
Assignees
Labels
Type
Projects
Status
New