TestApp: Change crates_from_index_head() to take crate name instead of raw path

Author: Turbo87

src/tests/krate/publish.rs

@@ -161,7 +161,7 @@ fn new_with_renamed_dependency() {
     token.enqueue_publish(crate_to_publish).good();
     app.run_pending_background_jobs();
 
-    let crates = app.crates_from_index_head("ne/w-/new-krate");
+    let crates = app.crates_from_index_head("new-krate");
     assert_eq!(crates.len(), 1);
     assert_eq!(crates[0].name, "new-krate");
     assert_eq!(crates[0].vers, "1.0.0");
@@ -187,7 +187,7 @@ fn new_with_underscore_renamed_dependency() {
     token.enqueue_publish(crate_to_publish).good();
     app.run_pending_background_jobs();
 
-    let crates = app.crates_from_index_head("ne/w-/new-krate");
+    let crates = app.crates_from_index_head("new-krate");
     assert_eq!(crates.len(), 1);
     assert_eq!(crates[0].name, "new-krate");
     assert_eq!(crates[0].vers, "1.0.0");
@@ -528,7 +528,7 @@ fn new_krate_git_upload() {
     token.enqueue_publish(crate_to_publish).good();
     app.run_pending_background_jobs();
 
-    let crates = app.crates_from_index_head("3/f/fgt");
+    let crates = app.crates_from_index_head("fgt");
     assert_eq!(crates.len(), 1);
     assert_eq!(crates[0].name, "fgt");
     assert_eq!(crates[0].vers, "1.0.0");
@@ -549,7 +549,7 @@ fn new_krate_git_upload_appends() {
     token.enqueue_publish(crate_to_publish).good();
     app.run_pending_background_jobs();
 
-    let crates = app.crates_from_index_head("3/f/fpp");
+    let crates = app.crates_from_index_head("fpp");
     assert!(crates.len() == 2);
     assert_eq!(crates[0].name, "FPP");
     assert_eq!(crates[0].vers, "0.0.1");

src/tests/krate/yanking.rs

@@ -49,7 +49,7 @@ fn yank_works_as_intended() {
     token.enqueue_publish(crate_to_publish).good();
     app.run_pending_background_jobs();
 
-    let crates = app.crates_from_index_head("3/f/fyk");
+    let crates = app.crates_from_index_head("fyk");
     assert_eq!(crates.len(), 1);
     assert_some_eq!(crates[0].yanked, false);
 
@@ -60,7 +60,7 @@ fn yank_works_as_intended() {
     // yank it
     token.yank("fyk", "1.0.0").good();
 
-    let crates = app.crates_from_index_head("3/f/fyk");
+    let crates = app.crates_from_index_head("fyk");
     assert_eq!(crates.len(), 1);
     assert_some_eq!(crates[0].yanked, true);
 
@@ -70,7 +70,7 @@ fn yank_works_as_intended() {
     // un-yank it
     token.unyank("fyk", "1.0.0").good();
 
-    let crates = app.crates_from_index_head("3/f/fyk");
+    let crates = app.crates_from_index_head("fyk");
     assert_eq!(crates.len(), 1);
     assert_some_eq!(crates[0].yanked, false);
 
@@ -80,7 +80,7 @@ fn yank_works_as_intended() {
     // yank it
     cookie.yank("fyk", "1.0.0").good();
 
-    let crates = app.crates_from_index_head("3/f/fyk");
+    let crates = app.crates_from_index_head("fyk");
     assert_eq!(crates.len(), 1);
     assert_some_eq!(crates[0].yanked, true);
 
@@ -90,7 +90,7 @@ fn yank_works_as_intended() {
     // un-yank it
     cookie.unyank("fyk", "1.0.0").good();
 
-    let crates = app.crates_from_index_head("3/f/fyk");
+    let crates = app.crates_from_index_head("fyk");
     assert_eq!(crates.len(), 1);
     assert_some_eq!(crates[0].yanked, false);
 

src/tests/util/test_app.rs

@@ -10,7 +10,7 @@ use cargo_registry::{
 };
 use std::{rc::Rc, sync::Arc, time::Duration};
 
-use cargo_registry::git::Repository as WorkerRepository;
+use cargo_registry::git::{Repository as WorkerRepository, Repository};
 use diesel::PgConnection;
 use git2::Repository as UpstreamRepository;
 use reqwest::{blocking::Client, Proxy};
@@ -134,12 +134,12 @@ impl TestApp {
     }
 
     /// Obtain a list of crates from the index HEAD
-    pub fn crates_from_index_head(&self, path: &str) -> Vec<cargo_registry::git::Crate> {
-        let path = std::path::Path::new(path);
+    pub fn crates_from_index_head(&self, crate_name: &str) -> Vec<cargo_registry::git::Crate> {
+        let path = Repository::relative_index_file(crate_name);
         let index = self.upstream_repository();
         let tree = index.head().unwrap().peel_to_tree().unwrap();
         let blob = tree
-            .get_path(path)
+            .get_path(&path)
             .unwrap()
             .to_object(index)
             .unwrap()