Apply coverage ignore comments as applicable

Aperta
  • Qualcosa non va?
#263 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 5 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
55/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
dart
Ambito
cryptography, testing

Direzione di ricerca

Inizia esaminando le indicazioni del package coverage e il metodo requireSubtleCrypto in lib/src/crypto_subtle.dart intorno alle righe collegate, quindi controlla i risultati di coverage per altri percorsi simili. Il lavoro è completato quando il codice artificiale o intenzionalmente non testato concordato è escluso senza nascondere comportamenti che dovrebbero essere testati e l’esecuzione di coverage continua a terminare correttamente.

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

Descrizione

cleanup

From: https://pub.dev/packages/coverage

Ignore lines from coverage

  • // coverage:ignore-line to ignore one line.
  • // coverage:ignore-start and // coverage:ignore-end to ignore range of lines inclusive.
  • // coverage:ignore-file to ignore the whole file.

We should apply this to bit of the code that we don't want coverage for.
I'm sure we have a few places that we don't care to test.

Example, maybe it's overkill to test requireSubtleCrypto logic, we could just make that private method, and use these coverage-ignore comments to avoid counting it in coverage.
https://github.com/google/webcrypto.dart/blob/7a673ab4bdf2c4638e819aa75608e019a3488fb7/lib/src/crypto_subtle.dart#L81-L102

This is really just nice to have and generally users shouldn't hit this. And we don't really want to make testing setups where we can reproduce the behavior. So any testing we do is kind of artificial anyways.

Lingua principale
Dart
Stelle
115
Fork
115
Merge medio
3g 2h
PR unite (30g)
6

Preparare l'ambiente

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 google/webcrypto.dart

Tutte le issue di google/webcrypto.dart

Issue simili

Altre issue su Dart

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.