Adopt shared mapper test data from engine-test-data

Aperta
#180 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
45/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
csharp
Ambito
testing

Direzione di ricerca

Inizia con MappersTest.cs e con i dati di test condivisi mapper_test_cases descritti in Flagsmith/engine-test-data#46. Confronta l'attuale JSON del contesto atteso inline con i casi condivisi, quindi verifica che mapper_test_cases passino e che il JSON inline sia stato rimosso.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Mapper tests use inline expected context data rather than shared test cases from engine-test-data repository.

Acceptance criteria

  • mapper_test_cases pass (see Flagsmith/engine-test-data#46)
  • Remove inline expected context JSON from MappersTest.cs

Technical context

Current implementation uses inline JSON:
https://github.com/Flagsmith/flagsmith-dotnet-client/pull/177/files#diff-f851c822fa0e162b75312b6b9ab5dc31d3631179a33360ec77bbb83f4ae7c50eR12-R140

Shared test data: https://github.com/Flagsmith/engine-test-data/pull/46

Related issues:

  • Flagsmith/flagsmith-python-client#181
  • Flagsmith/flagsmith-ruby-client#93
  • Flagsmith/flagsmith-java-client#197
  • Flagsmith/flagsmith-nodejs-client#228
  • Flagsmith/flagsmith-go-client#196
  • Flagsmith/flagsmith-rust-flag-engine#26
Lingua principale
C#
Stelle
20
Fork
17
Merge medio
21h 47m
PR unite (30g)
8

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Flagsmith/flagsmith-dotnet-client

Tutte le issue di Flagsmith/flagsmith-dotnet-client

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.