fix(biblioteca): el recuento de cada estado vuelve a coincidir con su listado

Al dar ficha personal a los juegos del préstamo familiar, los recuentos de la
barra lateral empezaron a contarlos y el listado seguía escondiéndolos: «Sin
clasificar 1744» llevaba a «ningún juego coincide». Un número que lleva a una
pantalla vacía es peor que no tener número.

`list_statuses` y `list_collections` aplican ahora las dos mismas exclusiones que
`list_games` por defecto —los prestados sin confirmar y los archivados no están
en la biblioteca, así que tampoco en sus recuentos—, que es lo que ya hacía
`library_stats`. Las tres consultas describen lo mismo y ahora lo dicen igual.

Clasificar un juego prestado sigue sirviendo: dentro del ámbito de Steam Family
el filtro por estado lo encuentra, porque allí se piden a propósito. Dos pruebas
lo fijan: que el recuento de un estado coincida con lo que enseña al pulsarlo, y
que el ámbito de Family sí devuelva los prestados por estado.

De paso, los archivados dejan de inflar los recuentos, que era un desajuste
anterior del mismo tipo.

Author: SwonDev

src-tauri/src/db/mod.rs

@@ -2396,7 +2396,7 @@ mod durability_tests {
 
 #[cfg(test)]
 mod pruebas_del_recuento_de_familia {
-    use super::{library, migrations, seed_defaults};
+    use super::{library, migrations, organization, seed_defaults};
     use rusqlite::Connection;
 
     fn base() -> Connection {
@@ -2500,6 +2500,59 @@ mod pruebas_del_recuento_de_familia {
         assert!(url.ends_with("library_600x900_2x.jpg"), "quedó en {url}");
     }
 
+    #[test]
+    fn el_recuento_de_un_estado_coincide_con_lo_que_enseña_al_pulsarlo() {
+        // Un número en la barra lateral que lleva a «ningún juego coincide» es
+        // peor que no tener número. Pasó al dar ficha personal al catálogo de
+        // Family: el recuento los contaba y el listado los escondía.
+        let connection = base();
+        juego_propio(&connection, 10);
+        juego_prestado(&connection, 20);
+        juego_prestado(&connection, 30);
+
+        let estados = organization::list_statuses(&connection).expect("estados");
+        let sin_clasificar = estados
+            .iter()
+            .find(|estado| estado.id == "unclassified")
+            .expect("existe el estado por defecto");
+
+        let request = crate::models::GameListRequest {
+            status_id: Some("unclassified".to_string()),
+            ..Default::default()
+        };
+        let listado = library::list_games(&connection, &request, None).expect("listar");
+
+        assert_eq!(
+            sin_clasificar.game_count as usize,
+            listado.items.len(),
+            "la barra lateral ofrece {} y el listado enseña {}",
+            sin_clasificar.game_count,
+            listado.items.len()
+        );
+        assert_eq!(
+            sin_clasificar.game_count, 1,
+            "sólo el propio está en la biblioteca"
+        );
+    }
+
+    #[test]
+    fn el_ambito_de_familia_sí_enseña_los_prestados_por_estado() {
+        // Y clasificar un juego prestado tiene que servir de algo: dentro de su
+        // ámbito, el filtro por estado lo encuentra.
+        let connection = base();
+        juego_prestado(&connection, 20);
+
+        let request = crate::models::GameListRequest {
+            status_id: Some("unclassified".to_string()),
+            ownership_source: Some("family_shared".to_string()),
+            ..Default::default()
+        };
+        let listado = library::list_games(&connection, &request, None).expect("listar");
+
+        assert_eq!(listado.items.len(), 1);
+        assert_eq!(listado.items[0].app_id, 20);
+    }
+
     #[test]
     fn sin_catalogo_de_familia_la_cifra_es_cero_y_no_una_ausencia() {
         // Cero es una respuesta: significa que no hay nada prestado. La interfaz

src-tauri/src/db/organization.rs

@@ -43,9 +43,20 @@ struct GameFacts {
 
 pub fn list_statuses(connection: &Connection) -> AppResult<Vec<StatusDefinition>> {
     let mut statement = connection.prepare(
-        "SELECT s.id, s.name, s.color, s.position, s.built_in, COUNT(p.app_id)
+        // Las mismas dos exclusiones que aplica `library::list_games` por
+        // defecto: los juegos prestados sin confirmar y los archivados no están
+        // en la biblioteca, así que tampoco pueden estar en sus recuentos.
+        // Cuando el catálogo de Family pasó a tener ficha personal, esta
+        // consulta empezó a contarlo y la barra ofrecía mil setecientos juegos
+        // que el listado escondía.
+        "SELECT s.id, s.name, s.color, s.position, s.built_in,
+                COUNT(CASE WHEN g.app_id IS NOT NULL THEN 1 END)
            FROM statuses s
       LEFT JOIN game_personal p ON p.status_id = s.id
+      LEFT JOIN games g ON g.app_id = p.app_id
+                       AND NOT (g.ownership_source = 'family_shared'
+                                AND g.family_availability <> 'confirmed')
+                       AND NOT EXISTS (SELECT 1 FROM game_archive a WHERE a.app_id = g.app_id)
        GROUP BY s.id
        ORDER BY s.position ASC, s.name COLLATE NOCASE ASC",
     )?;
@@ -152,10 +163,16 @@ pub fn reorder_statuses(connection: &mut Connection, ordered_ids: &[String]) ->
 
 pub fn list_collections(connection: &Connection) -> AppResult<Vec<CollectionSummary>> {
     let mut statement = connection.prepare(
+        // Mismo criterio que en los estados: la cifra tiene que coincidir con lo
+        // que se encuentra al pulsarla.
         "SELECT c.id, c.name, c.description, c.color, c.icon, c.kind, c.match_mode,
-                c.position, COUNT(cg.app_id)
+                c.position, COUNT(CASE WHEN g.app_id IS NOT NULL THEN 1 END)
            FROM collections c
       LEFT JOIN collection_games cg ON cg.collection_id = c.id
+      LEFT JOIN games g ON g.app_id = cg.app_id
+                       AND NOT (g.ownership_source = 'family_shared'
+                                AND g.family_availability <> 'confirmed')
+                       AND NOT EXISTS (SELECT 1 FROM game_archive a WHERE a.app_id = g.app_id)
        GROUP BY c.id
        ORDER BY c.position ASC, c.name COLLATE NOCASE ASC",
     )?;