Controller specs pass unbound methods to expect() (unbound-method lint errors)
Los mantenedores suelen responder en 1 día
@DeFex-lab ya está trabajando en esto.
Desde el 19/10/2026.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 30/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- eslint, typescript
- Área
- testing
Línea de trabajo
Read the three flagged spots first: github.controller.spec.ts around lines 62 and 71, and github-webhooks.controller.spec.ts around line 142, comparing how each spec references controller.sync against how the rest of the suite sets up mocks. Replace the bare method references with jest.spyOn on the controller instance (or the prototype) so the assertions still target the same call. Done means npm run lint no longer reports @typescript-eslint/unbound-method and the affected controller specs still pass. Note that open PRs #627 and #660 already touch this, so coordinate before starting.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
@typescript-eslint/unbound-method errors at github.controller.spec.ts:62, :71 and github-webhooks.controller.spec.ts:142, where the spec does expect(controller.sync)-style references. These are errors under the repo's ESLint config, so npm run lint can't pass even after the other fixes.
Suggested fix
Use jest.spyOn or reference the method through the prototype or the mock, and keep the rule.
Related: #109
- Lenguaje dominante
- TypeScript
- Estrellas
- 2
- Forks
- 90
- Merge medio
- 31 min
- PR fusionados (30 d)
- 20
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de MergeFi/backend
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
bug security
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
bug testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
bug security
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
Todos los issues de MergeFi/backend
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
zapcooking/frontend#946 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
WiliamRosa/cosmos-semantic-platform#40 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
liberation-data/drivine#733 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
makecindy/cindy#10062 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día