Hides crates on index calls unless they have at least one available version

This makes the index call only show crates that have at least
one available version. Crates that have all versions yanked
will not show up anymore.

Fixes #958

Author: Maurício Linhares

src/krate.rs

@@ -564,6 +564,8 @@ pub fn index(req: &mut Request) -> CargoResult<Response> {
 
     let recent_downloads = sql::<Nullable<BigInt>>("SUM(crate_downloads.downloads)");
 
+    let not_yanked_versions = sql::<Bool>("crates.id = ANY (SELECT vs.crate_id FROM versions vs WHERE vs.crate_id = crates.id AND vs.yanked IS FALSE)");
+
     let mut query = crates::table
         .join(
             crate_downloads::table,
@@ -572,6 +574,7 @@ pub fn index(req: &mut Request) -> CargoResult<Response> {
                 crate_downloads::date.gt(date(now - 90.days())),
             ),
         )
+        .filter(not_yanked_versions.clone())
         .group_by(crates::id)
         .select((
             ALL_COLUMNS,

src/tests/krate.rs

@@ -1815,6 +1815,7 @@ fn yanked_versions_not_included_in_reverse_dependencies() {
     assert_eq!(deps.dependencies[0].crate_id, "c1");
 
     // TODO: have this test call `version.yank()` once the yank method is converted to diesel
+
     diesel::update(versions::table.filter(versions::num.eq("2.0.0")))
         .set(versions::yanked.eq(true))
         .execute(&*app.diesel_database.get().unwrap())
@@ -2108,6 +2109,45 @@ fn test_default_sort_recent() {
     assert_eq!(json.crates[1].downloads, 20);
 }
 
+/*  Given two crates, one with all versions yanked and another
+    with a good version only the one that doesn't have all versions
+    yanked is returned.
+ */
+#[test]
+fn test_hides_yanked_crate() {
+    let (_b, app, middle) = ::app();
+
+    {
+        let conn = app.diesel_database.get().unwrap();
+        let user = ::new_user("Oskar").create_or_update(&conn).unwrap();
+
+        ::CrateBuilder::new("green_ball", user.id)
+            .description("For fetching")
+            .downloads(0)
+            .recent_downloads(0)
+            .expect_build(&conn);
+
+        let yanked_crate = ::CrateBuilder::new("fully_yanked", user.id)
+            .description("Not here anymore")
+            .expect_build(&conn);
+
+        diesel::update(versions::table.filter(versions::crate_id.eq(yanked_crate.id)))
+            .set(versions::yanked.eq(true))
+            .execute(&*conn)
+            .unwrap();
+    }
+
+    let mut req = ::req(app, Method::Get, "/api/v1/crates");
+    let mut response = ok_resp!(middle.call(req.with_query("sort=recent-downloads")));
+    let json: CrateList = ::json(&mut response);
+
+    assert_eq!(json.meta.total, 1);
+
+    assert_eq!(json.crates[0].name, "green_ball");
+    assert_eq!(json.crates[0].recent_downloads, Some(0));
+    assert_eq!(json.crates[0].downloads, 0);
+}
+
 #[test]
 fn block_blacklisted_documentation_url() {
     let (_b, app, middle) = ::app();