Commit b05bd7f0 authored by Nugine's avatar Nugine
Browse files

fix(s3s): lints

parent 44c10027
Loading
Loading
Loading
Loading
+29 −29
Original line number Diff line number Diff line
use super::*;

use crate::service::S3Service;
// use crate::service::S3Service;

use stdx::mem::output_size;
// use stdx::mem::output_size;

#[test]
#[ignore]
fn track_future_size() {
    macro_rules! future_size {
        ($f:path, $v:expr) => {
            (stringify!($f), output_size(&$f), $v)
        };
    }
// #[test]
// #[ignore]
// fn track_future_size() {
//     macro_rules! future_size {
//         ($f:path, $v:expr) => {
//             (stringify!($f), output_size(&$f), $v)
//         };
//     }

    #[rustfmt::skip]
    let sizes = [
        future_size!(S3Service::call,                           2704),
        future_size!(call,                                      1512),
        future_size!(prepare,                                   1440),
        future_size!(SignatureContext::check,                   776),
        future_size!(SignatureContext::v2_check,                296),
        future_size!(SignatureContext::v2_check_presigned_url,  168),
        future_size!(SignatureContext::v2_check_header_auth,    184),
        future_size!(SignatureContext::v4_check,                752),
        future_size!(SignatureContext::v4_check_post_signature, 368),
        future_size!(SignatureContext::v4_check_presigned_url,  456),
        future_size!(SignatureContext::v4_check_header_auth,    640),
    ];
//     #[rustfmt::skip]
//     let sizes = [
//         future_size!(S3Service::call,                           2704),
//         future_size!(call,                                      1512),
//         future_size!(prepare,                                   1440),
//         future_size!(SignatureContext::check,                   776),
//         future_size!(SignatureContext::v2_check,                296),
//         future_size!(SignatureContext::v2_check_presigned_url,  168),
//         future_size!(SignatureContext::v2_check_header_auth,    184),
//         future_size!(SignatureContext::v4_check,                752),
//         future_size!(SignatureContext::v4_check_post_signature, 368),
//         future_size!(SignatureContext::v4_check_presigned_url,  456),
//         future_size!(SignatureContext::v4_check_header_auth,    640),
//     ];

    println!("{sizes:#?}");
    for (name, size, expected) in sizes {
        assert_eq!(size, expected, "{name:?} size changed: prev {expected}, now {size}");
    }
}
//     println!("{sizes:#?}");
//     for (name, size, expected) in sizes {
//         assert_eq!(size, expected, "{name:?} size changed: prev {expected}, now {size}");
//     }
// }

#[test]
fn error_custom_headers() {
+7 −1
Original line number Diff line number Diff line
@@ -103,7 +103,13 @@ impl S3Service {
        let duration = t0.elapsed();

        match result {
            Ok(ref res) => debug!(?duration, ?res),
            Ok(ref res) => {
                if res.status().is_server_error() {
                    error!(?duration, ?res);
                } else {
                    debug!(?duration, ?res);
                }
            }
            Err(ref err) => error!(?duration, ?err),
        }