adiciona kwargs ao Client() permitindo sobreescrever a variavel de ambiente com named_keys

[FelipeSBarros]

Adiciona a possibilidade de passar as credenciais email e password, para autentiação do Client, a partir de named_keys, sobreescrevendo as credenciais definidas por variáveis de ambiente.

[cuducos]

Aprovei do celular. Agora que vi que os testes estão falhando na CI. Isso acontece pois o Python tenta primeiro resolver o valor padrão no kwargs.gey. Temos que mudar a implementação para evitar isso!

[FelipeSBarros]

@cuducos #help!
Pelo vistos o meu teste não está passando na CI. No meu PC, passa.
Estou tentando entender o motivo. Entendo que não tenho um .env e não estou usando um mock.
Entendo que a solução seria um novo mock que simule variavel de ambiente e produza Client passando as kwargs...
Faz sentido?

[cuducos]

Uma alternativa seria:

email = kwargs.get(‘xpto’)
if email is None:
    email = config(`XPTO`)

[FelipeSBarros]

Uma alternativa seria:

Vou testar. Estava entendendo que, primeiro ele tenta resolver o config. E por não termos um .env e volta com UndefinedValueError.
Deixa eu ver aqui...

[cuducos]

Estou tentando entender o motivo.

Quando vc chama kwargs.get(arg1, arg2) o Pyhton tem que saber o valor (calcular) de arg1 e de arg2. Então, antes de saber se kwargs tem ou não a chave que estamos procurando, o Python executa a chamada do config(…)

[FelipeSBarros]

Quando vc chama kwargs.get(arg1, arg2) o Pyhton tem que saber o valor (calcular) de arg1 e de arg2. Então, antes de saber se kwargs tem ou não a chave que estamos procurando, o Python executa a chamada do config(…)

Foi o que eu entendi.

[FelipeSBarros]

@cuducos, fiz a alteração sugerida, mantendo o tratamento do UndefinedValueError....

[cuducos]

Agora sim 🎉