arginfo_zpp_mismatch.phpt and friends only tests statically built extensions on CI
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C
- Estrellas
- 40.4k
- Forks
- 8.1k
- Merge medio
- 2 d 13 h
- PR fusionados (30 d)
- 96
Descripción
Description
There are a couple of tests (see below) which iterate over all declared classes, and instantiate them. However, these tests don't load any shared extensions on CI, so the respective classes are not tested. I don't think this is intentional, but rather an oversight (when we switched to loading shared extensions via dl()).
Test that there is no arginfo/zpp mismatch [.\Zend\tests\arginfo_zpp_mismatch.phpt]
Test that there is no arginfo/zpp mismatch in strict mode [.\Zend\tests\arginfo_zpp_mismatch_strict.phpt]
Try to instantiate all classes without arguments [.\Zend\tests\instantiate_all_classes.phpt]
Object free handler must call zend_weakrefs_notify [.\Zend\tests\weakrefs\notify.phpt]
PHP Version
PHP 8.1
Operating System
any
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.
Línea de trabajo
Empieza leyendo Zend/tests/arginfo_zpp_mismatch.phpt, Zend/tests/arginfo_zpp_mismatch_strict.phpt, Zend/tests/instantiate_all_classes.phpt y Zend/tests/weakrefs/notify.phpt, junto con la configuración de CI que carga las extensiones. Ejecuta estas pruebas en la configuración actual de CI y verifica que las extensiones compartidas se carguen y estén cubiertas. La tarea está terminada cuando las pruebas indicadas ejerciten tanto las extensiones compiladas estáticamente como las compartidas en CI.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- ci-cd, testing
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100