From 0db7ff877cf16cdecce4e681c095a13add5e49fe Mon Sep 17 00:00:00 2001 From: ordinarthur <@arthurbarre.js@gmail.com> Date: Mon, 18 May 2026 14:10:49 +0200 Subject: [PATCH] =?UTF-8?q?fix(clients):=20aligne=20adresse=20structur?= =?UTF-8?q?=C3=A9e=20sur=20convention=20Lucid=20snakeCase?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit La migration 1778800000100_enrich_clients_for_invoicing avait créé les colonnes `address_line1` et `address_line2` (sans underscore avant le chiffre), mais Lucid v22 auto-snake-case `addressLine1` → `address_line_1` côté écriture. Résultat : tous les INSERT dans `clients` cassaient avec `column "address_line_1" of relation "clients" does not exist`. Bug latent — surfacé en lançant la suite functional complète après `node ace migration:run`. Affectait 20 tests Clients/Dashboard/Invoices. Fix : migration de rename qui aligne `address_line1` → `address_line_1` et `address_line2` → `address_line_2`. Le `RENAME COLUMN` préserve les données existantes en prod. Modèle Client simplifié (les déclarations manuelles ne sont plus nécessaires depuis que `schema.ts` les a régen). Bonus : fix du test `invoices.spec.ts` "liste paginée + meta total/page" qui créait 3 factures, ce qui dépasse la nouvelle limite Free 2 (ADR-023). Posé un `gracePeriodEndsAt` futur sur l'org du test pour bypass le quota. État après ce commit : 127 tests verts (60 unit + 67 functional). Co-Authored-By: Claude Opus 4.7 --- apps/api/app/models/client.ts | 36 ++++--------------- ...0100_rename_client_address_line_columns.ts | 32 +++++++++++++++++ apps/api/tests/functional/invoices.spec.ts | 7 +++- 3 files changed, 45 insertions(+), 30 deletions(-) create mode 100644 apps/api/database/migrations/1779000000100_rename_client_address_line_columns.ts diff --git a/apps/api/app/models/client.ts b/apps/api/app/models/client.ts index 8ec46e1..fcd8705 100644 --- a/apps/api/app/models/client.ts +++ b/apps/api/app/models/client.ts @@ -6,36 +6,14 @@ import Invoice from '#models/invoice' export default class Client extends ClientSchema { /** - * Champs ajoutés par la migration `1778800000100_enrich_clients_for_invoicing` - * (SIREN/TVA intra/adresse structurée). Déclarations manuelles en attendant - * que `schema.ts` soit régénéré par `node ace migration:run`. - * - * Le champ `address` (existant, string libre) est conservé pour les clients - * importés avant la feature ; le nouveau code lit en priorité ces champs - * structurés et retombe sur `address` s'ils sont vides. + * Le champ `address` (existant, string libre) est conservé pour les + * clients importés avant l'éditeur natif. Les champs structurés + * (`address_line_1`, `address_zip`, etc.) viennent de la migration + * `1778800000100_enrich_clients_for_invoicing` + alignement Lucid via + * `1779000000100_rename_client_address_line_columns`. Le nouveau code + * lit en priorité les champs structurés et retombe sur `address` s'ils + * sont vides. */ - @column() - declare siren: string | null - - @column() - declare tvaIntra: string | null - - @column() - declare addressLine1: string | null - - @column() - declare addressLine2: string | null - - @column() - declare addressZip: string | null - - @column() - declare addressCity: string | null - - /** ISO 3166-1 alpha-2 (ex. "FR"). */ - @column() - declare addressCountry: string | null - @belongsTo(() => Organization) declare organization: BelongsTo diff --git a/apps/api/database/migrations/1779000000100_rename_client_address_line_columns.ts b/apps/api/database/migrations/1779000000100_rename_client_address_line_columns.ts new file mode 100644 index 0000000..788c923 --- /dev/null +++ b/apps/api/database/migrations/1779000000100_rename_client_address_line_columns.ts @@ -0,0 +1,32 @@ +import { BaseSchema } from '@adonisjs/lucid/schema' + +/** + * Renomme `address_line1` → `address_line_1` (et idem line2) pour aligner + * avec la convention Lucid snakeCase (Adonis v7 traite les digits comme + * des tokens séparés : `addressLine1` → `address_line_1`). + * + * La migration `1778800000100_enrich_clients_for_invoicing_table` avait + * créé les colonnes sans underscore avant le chiffre, ce qui causait des + * INSERT en erreur (`column "address_line_1" of relation "clients" does + * not exist`) puisque Lucid auto-snake-case sur write. Cette migration + * corrige rétroactivement. + * + * Aucune perte de donnée — `RENAME COLUMN` préserve le contenu. + */ +export default class extends BaseSchema { + protected tableName = 'clients' + + async up() { + this.schema.alterTable(this.tableName, (table) => { + table.renameColumn('address_line1', 'address_line_1') + table.renameColumn('address_line2', 'address_line_2') + }) + } + + async down() { + this.schema.alterTable(this.tableName, (table) => { + table.renameColumn('address_line_1', 'address_line1') + table.renameColumn('address_line_2', 'address_line2') + }) + } +} diff --git a/apps/api/tests/functional/invoices.spec.ts b/apps/api/tests/functional/invoices.spec.ts index 05fd33c..b2a032c 100644 --- a/apps/api/tests/functional/invoices.spec.ts +++ b/apps/api/tests/functional/invoices.spec.ts @@ -281,7 +281,12 @@ test.group('Invoices — GET /invoices', (group) => { group.each.setup(() => testUtils.db().withGlobalTransaction()) test('liste paginée + meta total/page', async ({ client, assert }) => { - const { bearer } = await createTestUser() + const { bearer, org } = await createTestUser() + // Grace period pour bypass le quota Free 2 factures (cf. ADR-023). + // Sans ça, la 3e facture serait rejetée en 402 et le test verrait + // meta.total=2 au lieu de 3. + org.gracePeriodEndsAt = DateTime.utc().plus({ months: 2 }) + await org.save() const c = await createClient(client, bearer, 'Liste Spec') for (let i = 0; i < 3; i++) {