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 (2.860 forks)batch import
good first issuetech debt

Métricas do repositório

Stars
 (16.597 stars)
Métricas de merge de PR
 (Mesclagem média 6d 8h) (109 fundiu PRs em 30d)

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

Direção de pesquisa
Revise a discussão em https://github.com/apache/apisix/discussions/7988 para contexto. Identifique todos os arquivos de teste que contêm as diretivas `extra init by lua start` e `extra init by lua`, e mescle as em uma única diretiva quando apropriado. Certifique se de que a alteração mantenha a cobertura de testes.
Pilha de tecnologia
lua
Domain
backend
Tipo Issue
Teste
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 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitLua
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75