Break sinon.spy(Component.prototype, 'method')

Description

TypeError: Attempted to wrap undefined property undefined as function
...

Guide contributeur

Stack technique

javascript

Domaine

frontend

Type d'issue

bug

DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.

3

Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.

1-3 hours

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

stale

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

needs investigation

Prérequis

basic understanding of JavaScript decoratorsfamiliarity with sinon spy

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

30

Direction de recherche

The issue occurs when using @autobind on a class component, then attempting to spy on a prototype method with sinon.spy. The error suggests that the property is undefined, likely because @autobind replaces the method with a getter or redefines properties. To reproduce, create a simple React component with @autobind on a method and test using sinon.spy(Component.prototype, 'method'). Investigate the autobind decorator source code (especially the core decorator logic) to understand how it binds methods and how that interferes with sinon's prototype spying. Consider if a workaround exists or if the decorator needs to be adjusted to preserve the property descriptor for spy compatibility.