Andrew Truong comments

Results 54 comments of


                                            Andrew Truong

Bug: sqlalchemy factory `column_property` in `get_model_fields` not working

This looks like the same issues as https://github.com/litestar-org/polyfactory/issues/508. This is currently fixed in main but not released yet

feat(sqlalchemy-factory): Add support for SQLAlchemy custom types

Thanks for the work on this @bc291 . I think this use case is now covered by https://github.com/litestar-org/polyfactory/pull/513/files. Please reopen if missed off a use case here. You will need...

Bug: Infinite recursion error with discriminated unions

This may fixed now with the changes from https://github.com/litestar-org/polyfactory/pull/468. - The original MVC has no way to be constructed easily as fully recursive. If a type in one of the...

fix: Send correct field_meta to avoid over nesting

Hi @guacs, looking at that PR I think I copied what's there. Specifying patch versions keep things deterministic but should be safe to specify minor version only to unblock more...

Bug: sqlalchemy factory `column_property` in `get_model_fields` not working

Resolved in last release

feat(sqla_factory): refresh object in async_session after commit to db

@all-contributors add @wangxin688 for code

Bug: Custom Random instance passed to random does not get passed to list/set fields

Thanks for the detailed report! Really helpful. I'm able to reproduce with ```python from hypothesis import given from hypothesis import strategies as st from pydantic import BaseModel from polyfactory.factories.pydantic_factory import...

Docs: specify the compatibility with SQLModel more explicitely

Hi ddahan, this should work in most situations for `SQLModel` though may be some edges cases not supported. The current behaviour for `SQLAlchemyFactory` is to not resolve foreign keys by...

fix(sqla_factory): changed default value for __set_foreign_keys__ attr.

Closing in favour of adjusting as part of v3 to avoid backwards breaking change

feat(fields): implement Param and CallableParam for handling unmapped parameters that can be referenced during build

What is the use case of requiring the kwargs from the factory field itself passed to a callable declared at build time? This feels like quite a specific use case...