Skip to content

Commit ff8b5ba

Browse files
committed
cargo fmt
1 parent b6e21bd commit ff8b5ba

File tree

7 files changed

+42
-23
lines changed

7 files changed

+42
-23
lines changed

src/controllers/krate/publish.rs

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,8 @@ pub fn publish(req: &mut dyn Request) -> CargoResult<Response> {
108108
// this file length.
109109
let file_length = read_le_u32(req.body())?;
110110

111-
let content_length = req.content_length()
111+
let content_length = req
112+
.content_length()
112113
.chain_error(|| human("missing header: Content-Length"))?;
113114
let max = krate
114115
.max_upload_size
@@ -130,9 +131,8 @@ pub fn publish(req: &mut dyn Request) -> CargoResult<Response> {
130131
license_file,
131132
// Downcast is okay because the file length must be less than the max upload size
132133
// to get here, and max upload sizes are way less than i32 max
133-
Some(file_length as i32)
134-
)?
135-
.save(&conn, &new_crate.authors)?;
134+
Some(file_length as i32),
135+
)?.save(&conn, &new_crate.authors)?;
136136

137137
// Link this new version to all dependencies
138138
let git_deps = dependency::add_dependencies(&conn, &new_crate.deps, version.id)?;
@@ -163,10 +163,15 @@ pub fn publish(req: &mut dyn Request) -> CargoResult<Response> {
163163
// If the git commands fail below, we shouldn't keep the crate on the
164164
// server.
165165
let max_unpack = cmp::max(app.config.max_unpack_size, max);
166-
let (cksum, mut crate_bomb, mut readme_bomb) = app
167-
.config
168-
.uploader
169-
.upload_crate(req, &krate, readme, file_length, max, max_unpack, vers)?;
166+
let (cksum, mut crate_bomb, mut readme_bomb) = app.config.uploader.upload_crate(
167+
req,
168+
&krate,
169+
readme,
170+
file_length,
171+
max,
172+
max_unpack,
173+
vers,
174+
)?;
170175
version.record_readme_rendering(&conn)?;
171176

172177
let mut hex_cksum = String::new();

src/models/version.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ pub struct Version {
2727
pub features: serde_json::Value,
2828
pub yanked: bool,
2929
pub license: Option<String>,
30-
pub crate_size: Option<i32>
30+
pub crate_size: Option<i32>,
3131
}
3232

3333
#[derive(Insertable, Debug)]
@@ -37,7 +37,7 @@ pub struct NewVersion {
3737
num: String,
3838
features: serde_json::Value,
3939
license: Option<String>,
40-
crate_size: Option<i32>
40+
crate_size: Option<i32>,
4141
}
4242

4343
impl Version {
@@ -72,7 +72,7 @@ impl Version {
7272
version_downloads: format!("/api/v1/crates/{}/{}/downloads", crate_name, num),
7373
authors: format!("/api/v1/crates/{}/{}/authors", crate_name, num),
7474
},
75-
crate_size
75+
crate_size,
7676
}
7777
}
7878

@@ -220,7 +220,7 @@ impl Queryable<versions::SqlType, Pg> for Version {
220220
features: row.6,
221221
yanked: row.7,
222222
license: row.8,
223-
crate_size: row.9
223+
crate_size: row.9,
224224
}
225225
}
226226
}

src/tests/all.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ use models::{Crate, CrateDownload, CrateOwner, Dependency, Keyword, Team, User,
4545
use models::{NewCategory, NewCrate, NewTeam, NewUser, NewVersion};
4646
use schema::*;
4747
use views::krate_publish as u;
48-
use views::{EncodableCrate, EncodableVersion, EncodableKeyword};
48+
use views::{EncodableCrate, EncodableKeyword, EncodableVersion};
4949

5050
macro_rules! t {
5151
($e:expr) => {

src/tests/krate.rs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ use tar;
2121
use cargo_registry::git;
2222
use cargo_registry::models::krate::MAX_NAME_LENGTH;
2323

24-
use {CrateList, CrateMeta, GoodCrate, CrateResponse};
24+
use {CrateList, CrateMeta, CrateResponse, GoodCrate};
2525

2626
use models::{ApiToken, Category, Crate};
2727
use schema::{crates, metadata, versions};
@@ -927,8 +927,10 @@ fn new_krate_gzip_bomb() {
927927
::sign_in(&mut req, &app);
928928
let len = 512 * 1024;
929929
let mut body = io::repeat(0).take(len);
930-
let body =
931-
::new_crate_to_body_with_io(&::new_crate("foo", "1.1.0"), &mut [("foo-1.1.0/a", &mut body, len)]);
930+
let body = ::new_crate_to_body_with_io(
931+
&::new_crate("foo", "1.1.0"),
932+
&mut [("foo-1.1.0/a", &mut body, len)],
933+
);
932934
let json = bad_resp!(middle.call(req.with_body(&body)));
933935
assert!(
934936
json.errors[0]
@@ -1308,7 +1310,9 @@ fn dependencies() {
13081310
let conn = app.diesel_database.get().unwrap();
13091311
let user = ::new_user("foo").create_or_update(&conn).unwrap();
13101312
let c1 = ::CrateBuilder::new("foo_deps", user.id).expect_build(&conn);
1311-
let v = ::new_version(c1.id, "1.0.0", None).save(&conn, &[]).unwrap();
1313+
let v = ::new_version(c1.id, "1.0.0", None)
1314+
.save(&conn, &[])
1315+
.unwrap();
13121316
let c2 = ::CrateBuilder::new("bar_deps", user.id).expect_build(&conn);
13131317
::new_dependency(&conn, &v, &c2);
13141318
}

src/tests/version.rs

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ use conduit::{Handler, Method};
1010

1111
use schema::versions;
1212
use views::EncodableVersion;
13-
use {GoodCrate, CrateResponse};
13+
use {CrateResponse, GoodCrate};
1414

1515
#[derive(Deserialize)]
1616
struct VersionList {
@@ -64,7 +64,9 @@ fn show() {
6464
let conn = app.diesel_database.get().unwrap();
6565
let user = ::new_user("foo").create_or_update(&conn).unwrap();
6666
let krate = ::CrateBuilder::new("foo_vers_show", user.id).expect_build(&conn);
67-
::new_version(krate.id, "2.0.0", Some(1234)).save(&conn, &[]).unwrap()
67+
::new_version(krate.id, "2.0.0", Some(1234))
68+
.save(&conn, &[])
69+
.unwrap()
6870
};
6971
req.with_path(&format!("/api/v1/versions/{}", v.id));
7072
let mut response = ok_resp!(middle.call(&mut req));
@@ -142,9 +144,17 @@ fn version_size() {
142144
let mut response = ok_resp!(middle.call(&mut show_req));
143145
let json: CrateResponse = ::json(&mut response);
144146

145-
let version1 = json.versions.iter().find(|v| v.num == "1.0.0").expect("Could not find v1.0.0");
147+
let version1 = json
148+
.versions
149+
.iter()
150+
.find(|v| v.num == "1.0.0")
151+
.expect("Could not find v1.0.0");
146152
assert_eq!(version1.crate_size, Some(35));
147153

148-
let version2 = json.versions.iter().find(|v| v.num == "2.0.0").expect("Could not find v2.0.0");
154+
let version2 = json
155+
.versions
156+
.iter()
157+
.find(|v| v.num == "2.0.0")
158+
.expect("Could not find v2.0.0");
149159
assert_eq!(version2.crate_size, Some(91));
150160
}

src/uploaders.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@ use s3;
66
use semver;
77
use tar;
88

9-
use util::{human, internal, CargoResult, ChainError};
109
use util::LimitErrorReader;
10+
use util::{human, internal, CargoResult, ChainError};
1111

1212
use std::env;
1313
use std::fs::{self, File};

src/views/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ pub struct EncodableVersion {
193193
// NOTE: Used by shields.io, altering `license` requires a PR with shields.io
194194
pub license: Option<String>,
195195
pub links: EncodableVersionLinks,
196-
pub crate_size: Option<i32>
196+
pub crate_size: Option<i32>,
197197
}
198198

199199
#[derive(Serialize, Deserialize, Debug)]

0 commit comments

Comments
 (0)