apache/apisix

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

Open

#7.990 geöffnet am 26. Sept. 2022

Auf GitHub ansehen
 (14 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Lua (16.597 Stars) (2.860 Forks)batch import
good first issuetech debt

Beschreibung

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.

Contributor Guide

Tech Stack
Keine
Domain
testing
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Lua basicsAPISIX testing conventions
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
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.