apache/apisix

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

Open

#7990 aperta il 26 set 2022

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

Metriche repository

Star
 (16.597 star)
Metriche merge PR
 (Merge medio 6g 8h) (109 PR mergiate in 30 g)

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

Direzione di ricerca
Rivedere la discussione in https://github.com/apache/apisix/discussions/7988 per il contesto. Identificare tutti i file di test che contengono sia le direttive `extra init by lua start` che `extra init by lua`, e unirle in una singola direttiva ove appropriato. Assicurarsi che la modifica mantenga la copertura dei test.
Tech stack
lua
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitLua
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75