tests/routes/crates/reverse_dependencies: update default versions

Author: eth3lbert

src/tests/routes/crates/reverse_dependencies.rs

@@ -1,5 +1,6 @@
 use crate::builders::{CrateBuilder, VersionBuilder};
 use crate::util::{RequestHelper, TestApp};
+use crates_io::models::update_default_version;
 use http::StatusCode;
 use insta::{assert_json_snapshot, assert_snapshot};
 
@@ -10,14 +11,16 @@ async fn reverse_dependencies() {
 
     app.db(|conn| {
         let c1 = CrateBuilder::new("c1", user.id).expect_build(conn);
-        CrateBuilder::new("c2", user.id)
+        let c2 = CrateBuilder::new("c2", user.id)
             .version(VersionBuilder::new("1.0.0").dependency(&c1, None))
             .version(
                 VersionBuilder::new("1.1.0")
                     .dependency(&c1, None)
                     .dependency(&c1, Some("foo")),
             )
             .expect_build(conn);
+        let _ = update_default_version(c1.id, conn);
+        let _ = update_default_version(c2.id, conn);
     });
 
     let response = anon
@@ -49,10 +52,12 @@ async fn reverse_dependencies_when_old_version_doesnt_depend_but_new_does() {
         let c1 = CrateBuilder::new("c1", user.id)
             .version("1.1.0")
             .expect_build(conn);
-        CrateBuilder::new("c2", user.id)
+        let c2 = CrateBuilder::new("c2", user.id)
             .version("1.0.0")
             .version(VersionBuilder::new("2.0.0").dependency(&c1, None))
             .expect_build(conn);
+        let _ = update_default_version(c1.id, conn);
+        let _ = update_default_version(c2.id, conn);
     });
 
     let response = anon
@@ -74,10 +79,12 @@ async fn reverse_dependencies_when_old_version_depended_but_new_doesnt() {
         let c1 = CrateBuilder::new("c1", user.id)
             .version("1.0.0")
             .expect_build(conn);
-        CrateBuilder::new("c2", user.id)
+        let c2 = CrateBuilder::new("c2", user.id)
             .version(VersionBuilder::new("1.0.0").dependency(&c1, None))
             .version("2.0.0")
             .expect_build(conn);
+        let _ = update_default_version(c1.id, conn);
+        let _ = update_default_version(c2.id, conn);
     });
 
     let response = anon
@@ -99,13 +106,16 @@ async fn prerelease_versions_not_included_in_reverse_dependencies() {
         let c1 = CrateBuilder::new("c1", user.id)
             .version("1.0.0")
             .expect_build(conn);
-        CrateBuilder::new("c2", user.id)
+        let c2 = CrateBuilder::new("c2", user.id)
             .version("1.1.0-pre")
             .expect_build(conn);
-        CrateBuilder::new("c3", user.id)
+        let c3 = CrateBuilder::new("c3", user.id)
             .version(VersionBuilder::new("1.0.0").dependency(&c1, None))
             .version("1.1.0-pre")
             .expect_build(conn);
+        let _ = update_default_version(c1.id, conn);
+        let _ = update_default_version(c2.id, conn);
+        let _ = update_default_version(c3.id, conn);
     });
 
     let response = anon
@@ -127,10 +137,12 @@ async fn yanked_versions_not_included_in_reverse_dependencies() {
         let c1 = CrateBuilder::new("c1", user.id)
             .version("1.0.0")
             .expect_build(conn);
-        CrateBuilder::new("c2", user.id)
+        let c2 = CrateBuilder::new("c2", user.id)
             .version("1.0.0")
             .version(VersionBuilder::new("2.0.0").dependency(&c1, None))
             .expect_build(conn);
+        let _ = update_default_version(c1.id, conn);
+        let _ = update_default_version(c2.id, conn);
     });
 
     let response = anon
@@ -174,7 +186,7 @@ async fn reverse_dependencies_includes_published_by_user_when_present() {
         let c1 = CrateBuilder::new("c1", user.id)
             .version("1.0.0")
             .expect_build(conn);
-        CrateBuilder::new("c2", user.id)
+        let c2 = CrateBuilder::new("c2", user.id)
             .version(VersionBuilder::new("2.0.0").dependency(&c1, None))
             .expect_build(conn);
 
@@ -187,9 +199,12 @@ async fn reverse_dependencies_includes_published_by_user_when_present() {
             .unwrap();
 
         // c3's version will have the published by info recorded
-        CrateBuilder::new("c3", user.id)
+        let c3 = CrateBuilder::new("c3", user.id)
             .version(VersionBuilder::new("3.0.0").dependency(&c1, None))
             .expect_build(conn);
+        let _ = update_default_version(c1.id, conn);
+        let _ = update_default_version(c2.id, conn);
+        let _ = update_default_version(c3.id, conn);
     });
 
     let response = anon
@@ -212,10 +227,12 @@ async fn reverse_dependencies_query_supports_u64_version_number_parts() {
     app.db(|conn| {
         let c1 = CrateBuilder::new("c1", user.id).expect_build(conn);
         // The crate that depends on c1...
-        CrateBuilder::new("c2", user.id)
+        let c2 = CrateBuilder::new("c2", user.id)
             // ...has a patch version at the limits of what the semver crate supports
             .version(VersionBuilder::new(&large_but_valid_version_number).dependency(&c1, None))
             .expect_build(conn);
+        let _ = update_default_version(c1.id, conn);
+        let _ = update_default_version(c2.id, conn);
     });
 
     let response = anon