apache/apisix

Merge `extra_init_by_lua_start ` and `extra_init_by_lua ` directive in test cases

Open

#7,990 opened on 2022年9月26日

GitHub で見る
 (14 comments) (0 reactions) (0 assignees)Lua (16,597 stars) (2,860 forks)batch import
good first issuetech debt

説明

Discussed in https://github.com/apache/apisix/discussions/7988

Originally posted by monkeyDluffy6017 September 26, 2022 Maybe we should merge extra_init_by_lua_start and extra_init_by_lua directives in test cases, they are just different on position.

コントリビューターガイド

技術スタック
なし
領域
testing
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Lua basicsAPISIX testing conventions
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70
調査方針
Review the discussion in https://github.com/apache/apisix/discussions/7988 for context. Identify all test files that contain both `extra init by lua start` and `extra init by lua` directives, and merge them into a single directive where appropriate. Ensure the change maintains test coverage.