apache/apisix

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

Open

#7.990 aberto em 26 de set. de 2022

Ver no GitHub
 (14 comments) (0 reactions) (0 assignees)Lua (16.597 stars) (2.860 forks)batch import
good first issuetech debt

Description

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.

Guia do colaborador

Pilha de tecnologia
None
Domain
testing
Tipo Issue
refactor
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Lua basicsAPISIX testing conventions
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
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.