php / php/php-src

Zend: add missing ZPP tests

オープン
#23,280 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

Category: Engine Category: Tests Good first issue
主要言語
C
スター
40.4k
フォーク
8.1k
平均マージ
2日 13時間
マージ済み PR(30日)
96

説明

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)

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず issue #23192 の作業と、一覧にある Zend のパラメータ解析指定子を確認します。未チェックのマクロとバリアントのカバレッジを追加し、対応する ZPP テストが一覧にある各ケースをカバーしていることを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
c
領域
testing-qa
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
52/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。