Merge pull request #9521 from Turbo87/move-tests

Move tests from `all` test crate into `crates_io`

Author: Turbo87

Cargo.toml

@@ -36,10 +36,6 @@ opt-level = 2
 name = "crates_io"
 doctest = true
 
-[[test]]
-name = "all"
-path = "src/tests/all.rs"
-
 [features]
 default = ["slow-tests"]
 

src/lib.rs

@@ -58,6 +58,8 @@ pub mod storage;
 pub mod tasks;
 pub mod team_repo;
 mod test_util;
+#[cfg(test)]
+mod tests;
 pub mod typosquat;
 pub mod util;
 pub mod views;

src/tests/account_lock.rs

@@ -1,4 +1,4 @@
-use crate::{util::RequestHelper, TestApp};
+use crate::tests::{util::RequestHelper, TestApp};
 use chrono::{DateTime, Duration, NaiveDateTime, Utc};
 use http::StatusCode;
 use insta::assert_snapshot;
@@ -8,7 +8,7 @@ const LOCK_REASON: &str = "test lock reason";
 
 fn lock_account(app: &TestApp, user_id: i32, until: Option<NaiveDateTime>) {
     app.db(|conn| {
-        use crates_io::schema::users;
+        use crate::schema::users;
         use diesel::prelude::*;
 
         diesel::update(users::table)

src/tests/authentication.rs

@@ -1,7 +1,7 @@
-use crate::util::{MockRequestExt, RequestHelper, Response};
-use crate::TestApp;
+use crate::tests::util::{MockRequestExt, RequestHelper, Response};
+use crate::tests::TestApp;
 
-use crate::util::encode_session_header;
+use crate::tests::util::encode_session_header;
 use http::{header, Method, StatusCode};
 use insta::assert_snapshot;
 

src/tests/blocked_routes.rs

@@ -1,5 +1,5 @@
-use crate::builders::{CrateBuilder, VersionBuilder};
-use crate::util::{RequestHelper, TestApp};
+use crate::tests::builders::{CrateBuilder, VersionBuilder};
+use crate::tests::util::{RequestHelper, TestApp};
 use http::StatusCode;
 
 #[tokio::test(flavor = "multi_thread")]

src/tests/builders/dependency.rs

@@ -1,4 +1,4 @@
-use crates_io::views::krate_publish as u;
+use crate::views::krate_publish as u;
 
 /// A builder for constructing a dependency of another crate.
 pub struct DependencyBuilder {

src/tests/builders/krate.rs

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

src/tests/builders/publish.rs

@@ -1,7 +1,7 @@
+use crate::models::DependencyKind;
+use crate::views::krate_publish as u;
 use bytes::{BufMut, Bytes, BytesMut};
 use cargo_manifest::{DependencyDetail, DepsSet, MaybeInherited};
-use crates_io::models::DependencyKind;
-use crates_io::views::krate_publish as u;
 use std::collections::BTreeMap;
 
 use crates_io_tarball::TarballBuilder;

src/tests/builders/version.rs

@@ -1,12 +1,12 @@
-use crates_io::{
+use crate::{
     models::{Crate, NewVersion, Version},
     schema::{dependencies, versions},
     util::errors::AppResult,
 };
 use std::collections::BTreeMap;
 
+use crate::util::errors::internal;
 use chrono::NaiveDateTime;
-use crates_io::util::errors::internal;
 use diesel::prelude::*;
 
 /// A builder to create version records for the purpose of inserting directly into the database.

src/tests/categories.rs

@@ -1,4 +1,4 @@
-use crates_io::schema::categories;
+use crate::schema::categories;
 use crates_io_test_db::TestDatabase;
 use diesel::*;
 
@@ -50,7 +50,7 @@ fn sync_adds_new_categories() {
     let test_db = TestDatabase::new();
     let mut conn = test_db.connect();
 
-    ::crates_io::boot::categories::sync_with_connection(ALGORITHMS_AND_SUCH, &mut conn).unwrap();
+    crate::boot::categories::sync_with_connection(ALGORITHMS_AND_SUCH, &mut conn).unwrap();
 
     let categories = select_slugs(&mut conn);
     assert_eq!(categories, vec!["algorithms", "algorithms::such"]);
@@ -61,8 +61,8 @@ fn sync_removes_missing_categories() {
     let test_db = TestDatabase::new();
     let mut conn = test_db.connect();
 
-    ::crates_io::boot::categories::sync_with_connection(ALGORITHMS_AND_SUCH, &mut conn).unwrap();
-    ::crates_io::boot::categories::sync_with_connection(ALGORITHMS, &mut conn).unwrap();
+    crate::boot::categories::sync_with_connection(ALGORITHMS_AND_SUCH, &mut conn).unwrap();
+    crate::boot::categories::sync_with_connection(ALGORITHMS, &mut conn).unwrap();
 
     let categories = select_slugs(&mut conn);
     assert_eq!(categories, vec!["algorithms"]);
@@ -73,8 +73,8 @@ fn sync_adds_and_removes() {
     let test_db = TestDatabase::new();
     let mut conn = test_db.connect();
 
-    ::crates_io::boot::categories::sync_with_connection(ALGORITHMS_AND_SUCH, &mut conn).unwrap();
-    ::crates_io::boot::categories::sync_with_connection(ALGORITHMS_AND_ANOTHER, &mut conn).unwrap();
+    crate::boot::categories::sync_with_connection(ALGORITHMS_AND_SUCH, &mut conn).unwrap();
+    crate::boot::categories::sync_with_connection(ALGORITHMS_AND_ANOTHER, &mut conn).unwrap();
 
     let categories = select_slugs(&mut conn);
     assert_eq!(categories, vec!["algorithms", "another"]);
@@ -86,7 +86,7 @@ fn test_real_categories() {
     let mut conn = test_db.connect();
 
     const TOML: &str = include_str!("../boot/categories.toml");
-    assert_ok!(crates_io::boot::categories::sync_with_connection(
+    assert_ok!(crate::boot::categories::sync_with_connection(
         TOML, &mut conn
     ));
 }