php / php/php-src

Zend: add missing ZPP tests

Ouverte
#23,280 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Category: Engine Category: Tests Good first issue
Langage dominant
C
Étoiles
40.4k
Forks
8.2k
Merge moyen
2 j 13 h
PR mergées (30 j)
96

Description

Effectively continuing the work started in #23192.

This is a good first issues for newcomers.

The missing specifiers are:

  • Z_PARAM_ENUM(dest, _ce)
  • old "a"
    • Z_PARAM_ARRAY_EX2(dest, check_null, deref, separate)
    • Z_PARAM_ARRAY_EX(dest, check_null, separate)
    • Z_PARAM_ARRAY(dest)
    • Z_PARAM_ARRAY_OR_NULL(dest)
  • old "A"
    • Z_PARAM_ARRAY_OR_OBJECT_EX2(dest, check_null, deref, separate)
    • Z_PARAM_ARRAY_OR_OBJECT_EX(dest, check_null, separate)
    • Z_PARAM_ARRAY_OR_OBJECT(dest)
  • old "h"
    • Z_PARAM_ARRAY_HT_EX2(dest, check_null, deref, separate)
    • Z_PARAM_ARRAY_HT_EX(dest, check_null, separate)
    • Z_PARAM_ARRAY_HT(dest)
    • Z_PARAM_ARRAY_HT_OR_NULL(dest)
  • array|int:
    • Z_PARAM_ARRAY_HT_OR_LONG_EX(dest_ht, dest_long, is_null, allow_null)
    • Z_PARAM_ARRAY_HT_OR_LONG(dest_ht, dest_long)
    • Z_PARAM_ARRAY_HT_OR_LONG_OR_NULL(dest_ht, dest_long, is_null)
  • array|string:
    • Z_PARAM_ARRAY_HT_OR_STR_EX(dest_ht, dest_str, allow_null)
    • Z_PARAM_ARRAY_HT_OR_STR(dest_ht, dest_str)
    • Z_PARAM_ARRAY_HT_OR_STR_OR_NULL(dest_ht, dest_str)
  • old "H"
    • Z_PARAM_ARRAY_OR_OBJECT_HT_EX2(dest, check_null, deref, separate)
    • Z_PARAM_ARRAY_OR_OBJECT_HT_EX(dest, check_null, separate)
    • Z_PARAM_ARRAY_OR_OBJECT_HT(dest)
  • old "f"
    • Z_PARAM_FUNC_EX2(dest_fci, dest_fcc, check_null, deref, free_trampoline)
    • Z_PARAM_FUNC_EX(dest_fci, dest_fcc, check_null, deref)
    • Z_PARAM_FUNC(dest_fci, dest_fcc)
    • Z_PARAM_FUNC_NO_TRAMPOLINE_FREE(dest_fci, dest_fcc)
    • Z_PARAM_FUNC_OR_NULL(dest_fci, dest_fcc)
    • Z_PARAM_FUNC_NO_TRAMPOLINE_FREE_OR_NULL(dest_fci, dest_fcc)
  • old "p"
    • Z_PARAM_PATH_EX(dest, dest_len, check_null, deref)
    • Z_PARAM_PATH(dest, dest_len)
    • Z_PARAM_PATH_OR_NULL(dest, dest_len)
  • old "P"
    • Z_PARAM_PATH_STR_EX(dest, check_null, deref)
    • Z_PARAM_PATH_STR(dest)
    • Z_PARAM_PATH_STR_OR_NULL(dest)
  • old "s"
    • Z_PARAM_STRING_EX(dest, dest_len, check_null, deref)
    • Z_PARAM_STRING(dest, dest_len)
    • Z_PARAM_STRING_OR_NULL(dest, dest_len)
  • old "S"
    • Z_PARAM_STR_EX(dest, check_null, deref)
    • Z_PARAM_STR(dest)
    • Z_PARAM_STR_OR_NULL(dest)
  • string|int
    • Z_PARAM_STR_OR_LONG_EX(dest_str, dest_long, is_null, allow_null)
    • Z_PARAM_STR_OR_LONG(dest_str, dest_long)
    • Z_PARAM_STR_OR_LONG_OR_NULL(dest_str, dest_long, is_null)
  • old "z"
    • Z_PARAM_ZVAL_EX2(dest, check_null, deref, separate)
    • Z_PARAM_ZVAL_EX(dest, check_null, separate)
    • Z_PARAM_ZVAL(dest)
    • Z_PARAM_ZVAL_OR_NULL(dest)
  • old "+" and "*"
    • Z_PARAM_VARIADIC_EX(spec, dest, dest_num, post_varargs)
    • Z_PARAM_VARIADIC(spec, dest, dest_num)
    • Z_PARAM_VARIADIC_WITH_NAMED(dest, dest_num, dest_named)

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 examiner le travail de l’issue #23192 et les spécificateurs d’analyse des paramètres de Zend listés. Ajoutez une couverture pour les macros et variantes non vérifiées, puis vérifiez que les tests ZPP correspondants couvrent chaque cas listé.

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

Évaluation

Stack technique
c
Domaine
testing-qa
Type d'issue
Fonctionnalité
Difficulté
4/5
Temps estimé
3-5 jours
Activité
Calme
Clarté
Plutôt claire
Accessibilité débutants
52/100

Recevez les nouvelles issues par e-mail

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