search

cunybpl / aiodal

async data access layer and related tools for sqlalchemy core
0 stars 0 forks source link

Add fixture to override transaction #2

Closed kgasiorowski closed 11 months ago

kgasiorowski commented 1 year ago

See: https://github.com/cunybpl/beist/pull/23/commits/deae8d47492d76738e501ccb1ff34a824d88308d

I was finding it difficult to write integration tests for the beist app, and it turned out I needed to mock the transaction to ensure that the tests and the application itself work with the same transaction object. @bsnacks000 suggested that we add this functionality to the aiodal package, so I've created this issue to track that.

bsnacks000 commented 11 months ago

@tinnaing347 investigate this case and see if it is worth adding here or just something we should manage in our projects case by case

bsnacks000 commented 11 months ago

@kgasiorowski Not sure if we should bother with this one or not... let me know what you think.

bsnacks000 commented 11 months ago

I'm going to close this and leave as something to override in projects...