Extract cargo-registry-index package

Author: Turbo87

Diff for: Cargo.lock

@@ -31,6 +31,7 @@ rustdoc-args = [
 [dependencies]
 anyhow = "=1.0.52"
 base64 = "=0.13.0"
+cargo-registry-index = { path = "cargo-registry-index" }
 cargo-registry-markdown = { path = "cargo-registry-markdown" }
 cargo-registry-s3 = { path = "cargo-registry-s3" }
 chrono = { version = "=0.4.19", features = ["serde"] }
@@ -56,7 +57,6 @@ dotenv = "=0.15.0"
 flate2 = "=1.0.22"
 futures-channel = { version = "=0.3.19", default-features = false }
 futures-util = "=0.3.19"
-git2 = "=0.13.25"
 hex = "=0.4.3"
 http = "=0.2.6"
 hyper = { version = "=0.14.16", features = ["client", "http1"] }
@@ -89,6 +89,7 @@ tracing-subscriber = "=0.2.25"
 url = "=2.2.2"
 
 [dev-dependencies]
+cargo-registry-index = { path = "cargo-registry-index", features = ["testing"] }
 claim = "=0.5.0"
 conduit-test = "=0.10.0"
 hyper-tls = "=0.5.0"

Diff for: Cargo.toml

@@ -0,0 +1,25 @@
+[package]
+name = "cargo-registry-index"
+version = "0.0.0"
+license = "MIT OR Apache-2.0"
+repository = "https://github.com/rust-lang/crates.io"
+description = "crates.io package index utilities"
+edition = "2018"
+resolver = "2"
+
+[lib]
+path = "lib.rs"
+
+[features]
+testing = ["serde_json"]
+
+[dependencies]
+anyhow = "=1.0.52"
+base64 = "=0.13.0"
+dotenv = "=0.15.0"
+git2 = "=0.13.25"
+serde = { version = "=1.0.133", features = ["derive"] }
+swirl = { git = "https://github.com/sgrif/swirl.git", rev = "e87cf37" }
+tempfile = "=3.2.0"
+url = "=2.2.2"
+serde_json = { version = "=1.0.74", optional = true }

Diff for: cargo-registry-index/Cargo.toml

@@ -1,3 +1,9 @@
+#[macro_use]
+extern crate serde;
+
+#[cfg(feature = "testing")]
+pub mod testing;
+
 use anyhow::{anyhow, Context};
 use std::collections::HashMap;
 use std::io::Write;

Diff for: src/git.rs renamed to cargo-registry-index/lib.rs

@@ -25,13 +25,10 @@ impl UpstreamIndex {
     }
 
     /// Obtain a list of crates from the index HEAD
-    pub fn crates_from_index_head(
-        &self,
-        crate_name: &str,
-    ) -> anyhow::Result<Vec<cargo_registry::git::Crate>> {
+    pub fn crates_from_index_head(&self, crate_name: &str) -> anyhow::Result<Vec<crate::Crate>> {
         let repo = &self.repository;
 
-        let path = cargo_registry::git::Repository::relative_index_file(crate_name);
+        let path = crate::Repository::relative_index_file(crate_name);
 
         let head = repo.head()?;
         let tree = head.peel_to_tree()?;

Diff for: src/tests/git.rs renamed to cargo-registry-index/testing.rs

@@ -5,8 +5,8 @@ use std::sync::{Arc, Mutex, MutexGuard, PoisonError};
 use swirl::PerformError;
 
 use crate::db::{DieselPool, DieselPooledConn, PoolError};
-use crate::git::Repository;
 use crate::uploaders::Uploader;
+use cargo_registry_index::Repository;
 
 impl<'a> swirl::db::BorrowedConnection<'a> for DieselPool {
     type Connection = DieselPooledConn<'a>;

Diff for: src/background_jobs.rs

@@ -13,8 +13,8 @@
 #![warn(clippy::all, rust_2018_idioms)]
 
 use cargo_registry::config;
-use cargo_registry::git::{Repository, RepositoryConfig};
 use cargo_registry::{background_jobs::*, db};
+use cargo_registry_index::{Repository, RepositoryConfig};
 use diesel::r2d2;
 use reqwest::blocking::Client;
 use std::sync::{Arc, Mutex};

Diff for: src/bin/background-worker.rs

@@ -9,7 +9,6 @@ use std::sync::Arc;
 use swirl::Job;
 
 use crate::controllers::cargo_prelude::*;
-use crate::git;
 use crate::models::{
     insert_version_owner_action, Badge, Category, Crate, DependencyKind, Keyword, NewCrate,
     NewVersion, Rights, VersionAction,
@@ -227,7 +226,7 @@ pub fn publish(req: &mut dyn RequestExt) -> EndpointResult {
         };
 
         // Register this crate in our local git repo.
-        let git_crate = git::Crate {
+        let git_crate = cargo_registry_index::Crate {
             name: name.0,
             vers: vers.to_string(),
             cksum: hex_cksum,
@@ -310,7 +309,7 @@ pub fn add_dependencies(
     conn: &PgConnection,
     deps: &[EncodableCrateDependency],
     target_version_id: i32,
-) -> AppResult<Vec<git::Dependency>> {
+) -> AppResult<Vec<cargo_registry_index::Dependency>> {
     use self::dependencies::dsl::*;
     use diesel::insert_into;
 
@@ -345,7 +344,7 @@ pub fn add_dependencies(
             };
 
             Ok((
-                git::Dependency {
+                cargo_registry_index::Dependency {
                     name,
                     req: dep.version_req.to_string(),
                     features: dep.features.iter().map(|s| s.0.to_string()).collect(),

Diff for: src/controllers/krate/publish.rs

@@ -40,7 +40,6 @@ pub mod config;
 pub mod db;
 mod downloads_counter;
 pub mod email;
-pub mod git;
 pub mod github;
 pub mod metrics;
 pub mod middleware;

Diff for: src/lib.rs

@@ -2,9 +2,9 @@ use diesel::deserialize::{self, FromSql};
 use diesel::pg::Pg;
 use diesel::sql_types::Integer;
 
-use crate::git::DependencyKind as IndexDependencyKind;
 use crate::models::{Crate, Version};
 use crate::schema::*;
+use cargo_registry_index::DependencyKind as IndexDependencyKind;
 
 #[derive(Identifiable, Associations, Debug, Queryable, QueryableByName)]
 #[belongs_to(Version)]

Diff for: src/models/dependency.rs

@@ -37,7 +37,6 @@ mod builders;
 mod categories;
 mod category;
 mod dump_db;
-mod git;
 mod keyword;
 mod krate;
 mod metrics;

Diff for: src/tests/all.rs

@@ -1,17 +1,12 @@
 use super::{MockAnonymousUser, MockCookieUser, MockTokenUser};
-use crate::git::UpstreamIndex;
 use crate::record;
 use crate::util::{chaosproxy::ChaosProxy, fresh_schema::FreshSchema};
 use cargo_registry::config;
-use cargo_registry::{
-    background_jobs::Environment,
-    db::DieselPool,
-    git::{Credentials, RepositoryConfig},
-    App, Emails,
-};
+use cargo_registry::{background_jobs::Environment, db::DieselPool, App, Emails};
+use cargo_registry_index::testing::UpstreamIndex;
+use cargo_registry_index::{Credentials, Repository as WorkerRepository, RepositoryConfig};
 use std::{rc::Rc, sync::Arc, time::Duration};
 
-use cargo_registry::git::Repository as WorkerRepository;
 use diesel::PgConnection;
 use reqwest::{blocking::Client, Proxy};
 use std::collections::HashSet;
@@ -133,7 +128,7 @@ impl TestApp {
     }
 
     /// Obtain a list of crates from the index HEAD
-    pub fn crates_from_index_head(&self, crate_name: &str) -> Vec<cargo_registry::git::Crate> {
+    pub fn crates_from_index_head(&self, crate_name: &str) -> Vec<cargo_registry_index::Crate> {
         self.upstream_index()
             .crates_from_index_head(crate_name)
             .unwrap()

Diff for: src/tests/util/test_app.rs

@@ -1,7 +1,7 @@
 use crate::background_jobs::Environment;
-use crate::git::Crate;
 use crate::schema;
 use anyhow::Context;
+use cargo_registry_index::Crate;
 use chrono::Utc;
 use diesel::prelude::*;
 use std::fs::{self, OpenOptions};