Extract a test helper method for uploading foo 2.0.0

Author: carols10cents

src/tests/all.rs

@@ -283,6 +283,10 @@ fn new_req_with_keywords(app: Arc<App>, krate: Crate, version: &str,
     return req;
 }
 
+fn new_req_body_foo_version_2() -> Vec<u8> {
+    new_req_body(::krate("foo"), "2.0.0", Vec::new(), Vec::new(), Vec::new())
+}
+
 fn new_req_body(krate: Crate, version: &str, deps: Vec<u::CrateDependency>,
                 kws: Vec<String>, cats: Vec<String>) -> Vec<u8> {
     let kws = kws.into_iter().map(u::Keyword).collect();

src/tests/krate.rs

@@ -462,9 +462,7 @@ fn new_crate_owner() {
     assert_eq!(::json::<CrateList>(&mut response).crates.len(), 1);
 
     // And upload a new crate as the first user
-    let body = ::new_req_body(
-        ::krate("foo"), "2.0.0", Vec::new(), Vec::new(), Vec::new()
-    );
+    let body = ::new_req_body_foo_version_2();
     req.mut_extensions().insert(u2);
     let mut response = ok_resp!(middle.call(req.with_path("/api/v1/crates/new")
                                                .with_method(Method::Put)

src/tests/team.rs

@@ -134,9 +134,7 @@ fn remove_team_as_named_owner() {
                             .with_body(body.as_bytes())));
 
     ::mock_user(&mut req, mock_user_on_only_x());
-    let body = ::new_req_body(
-        ::krate("foo"), "2.0.0", Vec::new(), Vec::new(), Vec::new()
-    );
+    let body = ::new_req_body_foo_version_2();
     let json = bad_resp!(middle.call(req.with_path("/api/v1/crates/new")
                                         .with_body(&body)
                                         .with_method(Method::Put)));
@@ -166,9 +164,7 @@ fn remove_team_as_team_owner() {
     assert!(json.errors[0].detail.contains("don't have permission"),
             "{:?}", json.errors);
 
-    let body = ::new_req_body(
-        ::krate("foo"), "2.0.0", Vec::new(), Vec::new(), Vec::new()
-    );
+    let body = ::new_req_body_foo_version_2();
     ok_resp!(middle.call(req.with_path("/api/v1/crates/new")
                             .with_body(&body)
                             .with_method(Method::Put)));
@@ -189,9 +185,7 @@ fn publish_not_owned() {
                             .with_body(body.as_bytes())));
 
     ::mock_user(&mut req, mock_user_on_only_x());
-    let body = ::new_req_body(
-        ::krate("foo"), "2.0.0", Vec::new(), Vec::new(), Vec::new()
-    );
+    let body = ::new_req_body_foo_version_2();
     let json = bad_resp!(middle.call(req.with_path("/api/v1/crates/new")
                                         .with_body(&body)
                                         .with_method(Method::Put)));
@@ -213,9 +207,7 @@ fn publish_owned() {
                             .with_body(body.as_bytes())));
 
     ::mock_user(&mut req, mock_user_on_only_x());
-    let body = ::new_req_body(
-        ::krate("foo"), "2.0.0", Vec::new(), Vec::new(), Vec::new()
-    );
+    let body = ::new_req_body_foo_version_2();
     ok_resp!(middle.call(req.with_path("/api/v1/crates/new")
                             .with_body(&body)
                             .with_method(Method::Put)));