python / python/cpython

new_callable arg to unittest.mock.patch seems not to support partial

Aberta
#137,145 3 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

stdlib type-bug
Linguagem predominante
Python
Estrelas
77.2k
Forks
35.9k
Métricas de merge de PRs
Métricas de PR pendentes

Descrição

Bug report

Bug description:

As in the title. The following code seems to disregard the partial and proceed to call the wrapped function right away (instead of calling the wrapping partial object).

from functools import partial
from unittest.mock import patch

def wrapped():
  """an exemplary response from `some.directory.module.my_function` function"""
    return [
        {
            "some_key": "some_value",
        }]

def test_some_func():
    with patch(
        "some.directory.module.my_function", new_callable=partial(wrapped)
    ):

    # calls `my_function` internally
    some.directory.module.another_function()

Running the above will produce: TypeError: 'list' object is not callable.
Acc. to the docs an object passed as new_callable argument should be called by patch to create a new dummy object replacing the target (some.directory.module.my_function). However it seems that instead of partial being called and returning a wrapped function, a wrapped function itself is called returning the list.

CPython versions tested on:

3.11

Operating systems tested on:

macOS

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece pelo ponto de entrada unittest.mock.patch e pelo tratamento de new_callable, usando o reprodutor partial(wrapped) do relatório. Compare a sequência de chamadas observada com o contrato documentado e adicione ou atualize a cobertura de regressão para que o destino seja substituído pelo comportamento callable esperado.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
python
Domínio
testing
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.