python / python/cpython

mock.patch applied at class level behaves incorrectly

Ouverte
#93,582 6 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

stdlib type-bug
Langage dominant
Python
Étoiles
77.2k
Forks
36k
Métriques de merge des PR
Métriques de PR en attente

Description

Bug report

When applying @mock.patch to classes where the test_ method is inherited, this applies the patch to the base class,, instead of to the specific class. In this case, the patch to the g method applies in MockedFTests and MockedFStraightGTests.

# a.py
import unittest
from unittest import mock

def f():
    return 41

def g():
    return 43

class BaseFTests(unittest.TestCase):
    def test_a(self):
        self._asserts(f())

    def _asserts(self, val):
        print(self, val)

@mock.patch('a.f', new=lambda: 2)
class MockedFTests(BaseFTests):
    def _asserts(self, val):
        print(self, val)
        self.assertEqual(val, 2)
        self.assertEqual(g(), 43)

@mock.patch('a.g', new=lambda: 3)
class MockedFMockedGTests(MockedFTests):
    def _asserts(self, val):
        print(self, val)
        self.assertEqual(val, g()-1)

class MockedFStraightGTests(MockedFTests):
    def _asserts(self, val):
        print(self, val)
        self.assertEqual(val, 2)
        self.assertEqual(g(), 43)

Expect all tests to pass.
Actual output: 2 tests fail with error AssertionError: 3 != 43

Your environment

  • CPython versions tested on: Python 3.10.4 on Windows

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par reproduire l’exemple dans un petit module à l’aide de unittest.mock.patch, en vous concentrant sur les méthodes test_ héritées et les classes BaseFTests, MockedFTests, MockedFMockedGTests et MockedFStraightGTests. Suivez le comportement des patches au niveau de la classe dans unittest.mock et ajoutez un test de régression pour le cas d’héritage signalé. Le travail est terminé lorsque les tests de l’exemple passent sans appliquer le patch de la sous-classe aux classes sœurs.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
testing-qa
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
42/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.