dal-go / dalgo2sql / 27255372519

90%

push

github

feat: bump dalgo to v0.59.0, honor InsertOptions in Insert/InsertMulti (#146)

Upgrades github.com/dal-go/dalgo from v0.46.0 to v0.59.0 in both the
main and end2end modules.

Insert, transaction.Insert and transaction.InsertMulti previously
ignored dal.InsertOption. They now follow the v0.59.0 contract:

- An explicit ID generator (e.g. dal.WithRandomStringKey) is run via
  dal.InsertWithIdGenerator with bounded retries (10) against the
  existing executeExists check while the generated ID is taken.
- dal.WithAdapterGeneratedID falls back to the default random-string
  generator dal.WithRandomStringKey(dal.DefaultRandomStringIDLength, 5)
  as generic SQL has no portable native ID allocation for arbitrary
  key types.
- Without options the record is inserted as is (unchanged behavior).

Adds unit tests (go-sqlmock) for first-attempt success, retry on taken
ID, attempts exhaustion, existence-check errors, the adapter-generated
ID fallback and InsertMulti inside a transaction. New code is at 100%
statement coverage; module total 90.3%.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

26 of 26 new or added lines in 1 file covered. (100.0%)

1066 of 1190 relevant lines covered (89.58%)

1.03 hits per line