php / php/php-src

Zend: add missing ZPP tests

Offen
#23,280 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Category: Engine Category: Tests Good first issue
Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.1k
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
96

Beschreibung

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)

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit der Durchsicht der Arbeit aus Issue #23192 und der aufgeführten Zend-Spezifizierer für die Parameteranalyse. Füge Abdeckung für die nicht geprüften Makros und Varianten hinzu und überprüfe anschließend, dass die entsprechenden ZPP-Tests jeden aufgeführten Fall abdecken.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
c
Bereich
testing-qa
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
52/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.