apache/apisix

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

Open

Aperta il 26 set 2022

Vedi su GitHub
 (14 commenti) (0 reazioni) (0 assegnatari)Lua (16.597 star) (2860 fork)batch import
good first issuetech debt

Descrizione

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.

Guida contributor

Tech stack
Nessuno
Dominio
testing
Tipo issue
refactor
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Lua basicsAPISIX testing conventions
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70
Direzione di ricerca
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.