Files
mabibli/MaBibli.Tests/ServiceRevuesSouhaiteesTests.cs
T
mathieuandClaude Opus 5 94f0ef15c0 Compte les pages d'un livre, et fais des envies une liste à deux moitiés
Lot U — le nombre de pages
`Livre.NombrePages` est nullable, sans valeur par défaut : « 0 page » se lirait
comme une donnée là où l'on veut dire « on ne sait pas ». Même raison que pour
`TypeDocument.NonPrecise` — un défaut qui ne prétend rien n'a rien à reprendre,
d'où une migration réduite à un `AddColumn`. Le service refuse un zéro plutôt
que de l'écrire ; effacer le champ reste la façon de revenir à « inconnu ».

Le préremplissage vient de `dc:format`, que rien ne lisait jusqu'ici. ⚠️ Ce
champ n'est pas un nombre mais une phrase décrivant le support, et les notices
déjà enregistrées sous Fixtures/ le montrent : « 1 vol. (113 p.) : ill., couv.
ill. en coul. ; 18 cm », « 503 p. : couv. ill. ; 17 cm ». Les règles sont donc
étroites — un nombre suivi de « p. » ou de « page(s) », rien d'autre — et tout
le reste rend `null`. L'erreur n'est pas symétrique : un champ vide se remplit à
la main en trois secondes, un chiffre faux s'enregistre sans que personne ne le
voie. « 1 vol. » ne vaut pas 1, « 30 cm » ne vaut pas 30, et « (p. 45-90) »,
qui est une pagination de contribution, ne vaut rien. La valeur reste proposée
dans un champ modifiable, et rien n'est déduit pour un ebook.

Lot X — éditer une envie, et souhaiter une revue
`PUT /api/souhaits/{id}` recalcule la clé d'œuvre et l'auteur normalisé : sans
ce recalcul, le rapprochement « déjà au catalogue » continuerait de se faire sur
l'ancienne forme, et le signalement mentirait sans le dire. Le filtre sur
l'appelant fait partie de la clé de recherche, pas d'une vérification ultérieure
— l'envie d'un autre est introuvable (404), jamais refusée (403).

⚠️ Une édition peut heurter l'unicité (utilisateur, œuvre, auteur), ce qu'un
ajout ne peut pas : renommer une envie en une autre déjà présente répond par un
message lisible, jamais par « UNIQUE constraint failed ». 400 et non 409,
contrairement au doublon du catalogue : là-bas posséder deux exemplaires est
légitime et l'appel se reconfirme, ici l'index l'interdit et il n'y a rien à
confirmer. Le rang n'est pas touché — l'ordre a son propre point d'entrée.

⚠️ La couverture n'est écrite que si la charge utile en porte une. Aucun écran
n'offre de champ « URL de couverture » pour une envie (décision actée), donc un
remplacement inconditionnel l'aurait effacée à la première faute de frappe
corrigée.

`RevueSouhaitee` est une table sœur, et non des colonnes de plus sur
`LivreSouhaite` : un numéro n'a pas d'auteur et se distingue par son numéro,
deux choses que la clé d'unicité des envies de livres ne sait pas exprimer sans
devenir fausse pour tout le monde. `NumeroNormalise` est NOT NULL avec un défaut
vide — SQLite tient deux NULL pour distincts, et « Médor, sans numéro »
s'ajouterait autant de fois qu'on cliquerait. L'ISSN est canonisé avec son tiret,
seul code du projet rangé ainsi.

Le coût de la table sœur est payé partout où il devait l'être : affichage,
`.txt`, `.csv` et instantané hors-ligne `souhaits-revues`. ⚠️ Les revues forment
une SECTION à part plutôt que des lignes entrelacées : chaque table numérote son
rang indépendamment, et mélanger deux suites sans rapport produirait un ordre que
personne n'a choisi. Le `.txt`, groupé par auteur, ne pouvait de toute façon pas
les accueillir — elles n'en ont pas, et « Auteur non précisé » désigne des livres
dont l'auteur est inconnu. Le CSV gagne une colonne « Type » : sans elle, un tri
par titre rendrait revues et livres indiscernables, et la colonne des codes
mêlerait ISBN et ISSN en silence.

`ServiceRenormalisation` connaît la nouvelle table, avec la règle de collision
déjà en place. ⚠️ L'ISSN y est canonisé à part : `Renormaliser` n'applique rien
quand la clé ne bouge pas, un ISSN mal formé sur une ligne au titre inchangé y
échapperait.

`RevueSouhaitee` ne porte PAS de `CoverUrl` : rien à ajouter au garde de
`GET /api/couvertures`.

Vérifié en exécution : ISSN « 24666718 » rangé « 2466-6718 », édition de l'envie
d'un autre en 404, et les deux exports portant bien les deux moitiés.
602 tests au vert (552 au départ), aucun avertissement de compilation.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-21 14:02:09 +02:00

236 lines
7.9 KiB
C#

using MaBibli.Api.Data;
using MaBibli.Api.Services.Souhaits;
using MaBibli.Shared.Dtos;
using Microsoft.Data.Sqlite;
using Microsoft.EntityFrameworkCore;
namespace MaBibli.Tests;
/// <summary>
/// Les envies de revues, sur une base SQLite <b>en mémoire</b> : l'unicité
/// (utilisateur, titre, numéro) n'existe qu'en base.
/// </summary>
/// <remarks>
/// Mêmes invariants que la liste d'envies de livres, et ils comptent autant : la table est
/// <b>personnelle</b>, et l'envie d'un autre doit être introuvable, jamais refusée.
/// </remarks>
public class ServiceRevuesSouhaiteesTests : IDisposable
{
private readonly SqliteConnection _connexion;
private readonly MaBibliDbContext _db;
private readonly ServiceRevuesSouhaitees _envies;
private const string Mathieu = "mathieu";
private const string Camille = "camille";
public ServiceRevuesSouhaiteesTests()
{
_connexion = new SqliteConnection("Data Source=:memory:");
_connexion.Open();
_db = new MaBibliDbContext(
new DbContextOptionsBuilder<MaBibliDbContext>().UseSqlite(_connexion).Options);
_db.Database.EnsureCreated();
_envies = new ServiceRevuesSouhaitees(_db);
}
private Task<ResultatRevueSouhaitee> AjouterAsync(
string titre, string? numero = null, string? issn = null,
string? note = null, string utilisateur = Mathieu) =>
_envies.AjouterAsync(
new EnregistrementRevueSouhaitee
{
Titre = titre,
Numero = numero,
Issn = issn,
Note = note,
},
utilisateur);
/// <summary>
/// L'ISSN est le seul code du projet rangé <b>avec son tiret</b> : c'est la forme que
/// produit le code-barres et celle qu'interroge la BnF. Sans canonisation, un ISSN tapé à
/// la main ne se rapprocherait de rien.
/// </summary>
[Fact]
public async Task LIssn_est_canonise_avec_son_tiret()
{
var envie = (await AjouterAsync("Médor", "43", "24666718")).Envie!;
Assert.Equal("2466-6718", envie.Issn);
Assert.Equal("2466-6718", _db.RevuesSouhaitees.Single().Issn);
}
[Fact]
public async Task Un_code_qui_nest_pas_un_issn_est_conserve_tel_quel()
{
// Un code mal recopié doit se voir mal recopié, pas disparaître ni être déguisé.
var envie = (await AjouterAsync("Médor", issn: "pas-un-issn")).Envie!;
Assert.Equal("pas-un-issn", envie.Issn);
}
[Fact]
public async Task Deux_numeros_de_la_meme_revue_cohabitent()
{
await AjouterAsync("Médor", "43");
var second = await AjouterAsync("Médor", "44");
Assert.Null(second.Erreur);
Assert.Equal(2, _db.RevuesSouhaitees.Count());
}
/// <summary>
/// ⚠️ Le piège que la colonne <c>NumeroNormalise</c> non nullable ferme : SQLite tient deux
/// <c>NULL</c> pour distincts, et « Médor, sans numéro » s'ajouterait autant de fois qu'on
/// cliquerait.
/// </summary>
[Fact]
public async Task La_meme_revue_sans_numero_ne_sajoute_quune_fois()
{
await AjouterAsync("Médor");
var second = await AjouterAsync("Médor");
Assert.NotNull(second.Erreur);
Assert.Contains("déjà dans votre liste", second.Erreur);
Assert.Single(_db.RevuesSouhaitees);
}
[Fact]
public async Task La_revue_entiere_et_un_de_ses_numeros_sont_deux_envies()
{
await AjouterAsync("Médor");
var numero = await AjouterAsync("Médor", "43");
Assert.Null(numero.Erreur);
Assert.Equal(2, _db.RevuesSouhaitees.Count());
}
[Fact]
public async Task Deux_personnes_peuvent_souhaiter_le_meme_numero()
{
await AjouterAsync("Médor", "43");
var autre = await AjouterAsync("Médor", "43", utilisateur: Camille);
Assert.Null(autre.Erreur);
Assert.Single(await _envies.ListerAsync(Mathieu));
Assert.Single(await _envies.ListerAsync(Camille));
}
[Fact]
public async Task La_liste_suit_le_rang_et_une_envie_neuve_va_en_fin()
{
await AjouterAsync("Médor", "43");
await AjouterAsync("XXI");
var liste = await _envies.ListerAsync(Mathieu);
Assert.Equal(["Médor n° 43", "XXI"], liste.Select(e => e.TitreComplet));
Assert.Equal([0, 1], _db.RevuesSouhaitees.OrderBy(e => e.Id).Select(e => e.Rang));
}
[Fact]
public async Task Editer_lenvie_dun_autre_est_introuvable_et_ne_change_rien()
{
var sienne = (await AjouterAsync("Médor", "43", utilisateur: Camille)).Envie!;
var resultat = await _envies.ModifierAsync(
sienne.Id, new EnregistrementRevueSouhaitee { Titre = "Détournée" }, Mathieu);
Assert.True(resultat.NExistePas);
Assert.Null(resultat.Erreur);
Assert.Equal("Médor", _db.RevuesSouhaitees.Single().Titre);
}
[Fact]
public async Task Editer_recalcule_les_formes_et_garde_le_rang()
{
await AjouterAsync("Médor", "43");
var seconde = (await AjouterAsync("XXI")).Envie!;
var resultat = await _envies.ModifierAsync(
seconde.Id,
new EnregistrementRevueSouhaitee { Titre = "Six Mois", Numero = "Hors-Série 7" },
Mathieu);
Assert.Null(resultat.Erreur);
var ligne = _db.RevuesSouhaitees.Single(e => e.Id == seconde.Id);
Assert.Equal("six mois", ligne.TitreNormalise);
Assert.Equal("hors serie 7", ligne.NumeroNormalise);
Assert.Equal(1, ligne.Rang);
}
[Fact]
public async Task Editer_vers_une_envie_deja_presente_est_refuse_lisiblement()
{
await AjouterAsync("Médor", "43");
var seconde = (await AjouterAsync("Médor", "44")).Envie!;
var resultat = await _envies.ModifierAsync(
seconde.Id,
new EnregistrementRevueSouhaitee { Titre = "MÉDOR", Numero = "43" },
Mathieu);
Assert.False(resultat.NExistePas);
Assert.NotNull(resultat.Erreur);
Assert.DoesNotContain("UNIQUE", resultat.Erreur, StringComparison.OrdinalIgnoreCase);
Assert.Equal(2, _db.RevuesSouhaitees.Count());
}
/// <summary>
/// ⚠️ Sans exclusion explicite de sa propre ligne, toute édition se dirait doublon
/// d'elle-même — et il serait impossible de corriger une note.
/// </summary>
[Fact]
public async Task Reenregistrer_la_meme_envie_nest_pas_un_doublon_delle_meme()
{
var envie = (await AjouterAsync("Médor", "43")).Envie!;
var resultat = await _envies.ModifierAsync(
envie.Id,
new EnregistrementRevueSouhaitee { Titre = "Médor", Numero = "43", Note = "au kiosque" },
Mathieu);
Assert.Null(resultat.Erreur);
Assert.Equal("au kiosque", _db.RevuesSouhaitees.Single().Note);
}
[Fact]
public async Task Retirer_lenvie_dun_autre_est_refuse()
{
var sienne = (await AjouterAsync("Médor", "43", utilisateur: Camille)).Envie!;
Assert.False(await _envies.SupprimerAsync(sienne.Id, Mathieu));
Assert.Single(_db.RevuesSouhaitees);
Assert.True(await _envies.SupprimerAsync(sienne.Id, Camille));
Assert.Empty(_db.RevuesSouhaitees);
}
[Fact]
public async Task Sans_identite_il_ny_a_ni_liste_ni_ecriture()
{
Assert.Empty(await _envies.ListerAsync(null));
Assert.NotNull((await AjouterAsync("Médor", utilisateur: null!)).Erreur);
Assert.True((await _envies.ModifierAsync(
1, new EnregistrementRevueSouhaitee { Titre = "Médor" }, null)).NExistePas);
Assert.False(await _envies.SupprimerAsync(1, null));
}
[Fact]
public async Task Le_titre_est_obligatoire()
{
Assert.NotNull((await AjouterAsync(" ")).Erreur);
Assert.Empty(_db.RevuesSouhaitees);
}
public void Dispose()
{
_db.Dispose();
_connexion.Dispose();
GC.SuppressFinalize(this);
}
}