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++) {