tests fail after adding types to test helper function

[toinbis]

lato/tests/test_transaction_context.py

Line 8 in 18c2afa

def add(a, b):

after modifying tests/test_transaction_context.py

from

def add(a, b):
    return a + b

to

def add(a: int, b: int) -> int:
    return a + b

test suite fails:

========================================================== short test summary info ===========================================================
FAILED tests/test_transaction_context.py::test_call_with_dependencies - assert 4 == 3
FAILED tests/test_transaction_context.py::test_call_with_kwarg_and_dependency - assert 22 == 12
======================================================== 2 failed, 43 passed in 0.16s ========================================================

P.S. Thanks a lot for creating and open source'ing python-ddd repo ant lato! Welki dzeki! :)

[pgorecki]

This problem arises due to the limitations of

lato/lato/dependency_provider.py

Line 112 in 4c5f0ec

def resolve_func_params(

Function arguments are first resolved by type, then by name. After your modification, both a and b have int type, so type resolution (resolve by int and int) takes precedence over name resolution (resolve by a and b), which leads to incorrect values being injected during a function call. For sure this logic could be more sophisticated, but at this point I have no plans for working on it, but please feel free to contribute :)

As a workaround, you can implement your own DependencyProvider that implements your own resolve_func_params and instantiate an app or transaction context with it using

app = Application(dependency_provider=CustomDependencyProvider(...))

[toinbis]

Thanks @pgoreck for prompt and clear answeri ! Need to think how I want to approach this :) Kindly please let's keep this issue open for a little bit while. Thanks!