elastic/kibana

Expand feature registry validation test coverage

Open

#207.342 aperta il 21 gen 2025

Vedi su GitHub
 (6 commenti) (0 reazioni) (1 assegnatario)TypeScript (8021 fork)batch import
Team:Coregood first issue

Metriche repository

Star
 (19.065 star)
Metriche merge PR
 (Merge medio 1g 16h) (999 PR mergiate in 30 g)

Descrizione

In https://github.com/elastic/kibana/pull/202863#discussion_r1923664248 the registry had to be changed to accommodate replacing disabled privileges with disabled privileges, but this surfaced the fact that the existing test cases miss coverage for the following privilege replacement scenarios:

  • Disabled → Disabled privilege replacement
  • Disabled → Enabled privilege replacement

We currently only test that Enabled → Disabled is prevented

Tests are located in x-pack/platform/plugins/shared/features/server/feature_registry.test.ts

Guida contributor

Direzione di ricerca
I casi di test mancanti riguardano gli scenari di sostituzione dei privilegi disabled→disabled e disabled→enabled. I test esistenti si trovano in `x pack/platform/plugins/shared/features/server/feature registry.test.ts`. Rivedere la discussione PR collegata all'indirizzo https://github.com/elastic/kibana/pull/202863#discussion r1923664248 per il contesto sulla modifica del registro. Implementare due nuovi casi di test che coprano questi tipi di sostituzione, seguendo lo schema del test esistente enabled→disabled.
Tech stack
typescript
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.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptTesting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70