Simplify nested imports (#10250)

Author: Turbo87

src/bin/crates-admin/render_readmes.rs

@@ -1,9 +1,7 @@
 use anyhow::{anyhow, Context};
-use crates_io::{
-    db,
-    models::Version,
-    schema::{crates, readme_renderings, versions},
-};
+use crates_io::db;
+use crates_io::models::Version;
+use crates_io::schema::{crates, readme_renderings, versions};
 use futures_util::{StreamExt, TryStreamExt};
 use std::path::{Path, PathBuf};
 use std::{future, sync::Arc};

src/bin/crates-admin/transfer_crates.rs

@@ -1,9 +1,7 @@
 use crate::dialoguer;
-use crates_io::{
-    db,
-    models::{Crate, OwnerKind, User},
-    schema::{crate_owners, crates, users},
-};
+use crates_io::db;
+use crates_io::models::{Crate, OwnerKind, User};
+use crates_io::schema::{crate_owners, crates, users};
 
 use diesel::prelude::*;
 use diesel_async::{AsyncPgConnection, RunQueryDsl};

src/tests/builders/krate.rs

@@ -1,12 +1,8 @@
-use crate::{
-    models::{Category, Crate, Keyword, NewCrate},
-    schema::{crates, version_downloads},
-    util::errors::AppResult,
-};
+use crate::models::{update_default_version, Category, Crate, Keyword, NewCrate};
+use crate::schema::{crate_downloads, crates, version_downloads};
+use crate::util::errors::AppResult;
 
 use super::VersionBuilder;
-use crate::models::update_default_version;
-use crate::schema::crate_downloads;
 use chrono::NaiveDateTime;
 use diesel::prelude::*;
 use diesel_async::{AsyncPgConnection, RunQueryDsl};

src/tests/builders/version.rs

@@ -1,8 +1,6 @@
-use crate::{
-    models::{Crate, NewVersion, Version},
-    schema::dependencies,
-    util::errors::AppResult,
-};
+use crate::models::{Crate, NewVersion, Version};
+use crate::schema::dependencies;
+use crate::util::errors::AppResult;
 use std::collections::BTreeMap;
 
 use chrono::NaiveDateTime;

src/tests/mod.rs

@@ -1,11 +1,9 @@
+use crate::models::{Crate, CrateOwner, NewCategory, NewTeam, NewUser, OwnerKind, Team, User};
+use crate::schema::crate_owners;
 use crate::tests::util::{RequestHelper, TestApp};
-use crate::{
-    models::{Crate, CrateOwner, NewCategory, NewTeam, NewUser, OwnerKind, Team, User},
-    schema::crate_owners,
-    views::{
-        EncodableCategory, EncodableCategoryWithSubcategories, EncodableCrate, EncodableKeyword,
-        EncodableOwner, EncodableVersion, GoodCrate,
-    },
+use crate::views::{
+    EncodableCategory, EncodableCategoryWithSubcategories, EncodableCrate, EncodableKeyword,
+    EncodableOwner, EncodableVersion, GoodCrate,
 };
 
 use crate::tests::util::github::next_gh_id;

src/tests/owners.rs

@@ -1,15 +1,11 @@
-use crate::tests::{
-    add_team_to_crate,
-    builders::{CrateBuilder, PublishBuilder},
-    new_team,
-    util::{MockAnonymousUser, MockCookieUser, MockTokenUser, RequestHelper, Response},
-    TestApp,
+use crate::models::Crate;
+use crate::tests::builders::{CrateBuilder, PublishBuilder};
+use crate::tests::util::{
+    MockAnonymousUser, MockCookieUser, MockTokenUser, RequestHelper, Response,
 };
-use crate::{
-    models::Crate,
-    views::{
-        EncodableCrateOwnerInvitationV1, EncodableOwner, EncodablePublicUser, InvitationResponse,
-    },
+use crate::tests::{add_team_to_crate, new_team, TestApp};
+use crate::views::{
+    EncodableCrateOwnerInvitationV1, EncodableOwner, EncodablePublicUser, InvitationResponse,
 };
 
 use crate::schema::users;

src/tests/team.rs

@@ -1,12 +1,7 @@
-use crate::tests::{
-    add_team_to_crate,
-    builders::{CrateBuilder, PublishBuilder},
-    new_team, OwnerTeamsResponse, RequestHelper, TestApp,
-};
-use crate::{
-    models::{Crate, NewTeam},
-    schema::teams,
-};
+use crate::models::{Crate, NewTeam};
+use crate::schema::teams;
+use crate::tests::builders::{CrateBuilder, PublishBuilder};
+use crate::tests::{add_team_to_crate, new_team, OwnerTeamsResponse, RequestHelper, TestApp};
 
 use diesel::*;
 use diesel_async::RunQueryDsl;

src/tests/user.rs

@@ -1,9 +1,6 @@
 use crate::models::{ApiToken, Email, NewUser, User};
-use crate::tests::{
-    new_user,
-    util::{MockCookieUser, RequestHelper},
-    TestApp,
-};
+use crate::tests::util::{MockCookieUser, RequestHelper};
+use crate::tests::{new_user, TestApp};
 use crate::util::token::HashedToken;
 use diesel::prelude::*;
 use diesel_async::RunQueryDsl;

src/tests/util/chaosproxy.rs

@@ -1,14 +1,10 @@
 use anyhow::{anyhow, Context};
 use std::net::SocketAddr;
 use std::sync::Arc;
-use tokio::{
-    io::{AsyncReadExt, AsyncWriteExt},
-    net::{
-        tcp::{OwnedReadHalf, OwnedWriteHalf},
-        TcpListener, TcpStream,
-    },
-    sync::broadcast::Sender,
-};
+use tokio::io::{AsyncReadExt, AsyncWriteExt};
+use tokio::net::tcp::{OwnedReadHalf, OwnedWriteHalf};
+use tokio::net::{TcpListener, TcpStream};
+use tokio::sync::broadcast::Sender;
 use tracing::{debug, error};
 use url::Url;
 

src/tests/util/matchers.rs

@@ -1,7 +1,5 @@
-use googletest::{
-    description::Description,
-    matcher::{Matcher, MatcherBase, MatcherResult},
-};
+use googletest::description::Description;
+use googletest::matcher::{Matcher, MatcherBase, MatcherResult};
 use http::StatusCode;
 
 pub fn is_success() -> SuccessMatcher {