Improve test coverage of the public API

Aperta
  • Qualcosa non va?
#1,867 1 commento 2 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
kotlin
Ambito
testing

Direzione di ricerca

Inizia con l’htmlReport.zip allegato oppure rigenera localmente il report di coverage con Coverage in IDEA, quindi esamina i file elencati nel modulo api. Aggiungi test per le funzioni non deprecate con coverage pari a zero, esegui nuovamente il report di coverage e conferma che la coverage dell’API pubblica pertinente sia migliorata.

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

Descrizione

tests

Some functions in the public API are not covered with tests.

The full report regarding the coverage is attached (it can also be generated locally with Coverage in IDEA).

Here is a list of files in the api module having at least one function that is not deprecated and has zero coverage (can be looked up in the report):

  1. cast.kt
  2. chunked.kt
  3. concat.kt
  4. constructors.kt
  5. convert.kt
  6. convertTo.kt
  7. cumSum.kt
  8. DataColumnType.kt
  9. DataFrameGet.kt
  10. DataRowApi.kt
  11. describe.kt
  12. drop.kt
  13. explode.kt
  14. format.kt
  15. formatHeader.kt
  16. gather.kt
  17. inferType.kt
  18. insert.kt
  19. into.kt
  20. join.kt
  21. JsonPath.kt
  22. map.kt
  23. max.kt
  24. mean.kt
  25. median.kt:
  26. merge.kt
  27. min.kt
  28. move.kt
  29. NameValueProperty (maybe shouldn’t be tested, very simple)
  30. Nulls.kt
  31. parse.kt
  32. ParserOptions.kt
  33. percentile.kt
  34. pivot.kt (maybe fine)
  35. reorder.kt
  36. replace.kt
  37. reverse.kt
  38. schema.kt
  39. shuffle.kt
  40. single.kt
  41. sort.kt
  42. split.kt
  43. std.kt
  44. sum.kt
  45. tail.kt
  46. take.kt
  47. toDataFrame.kt
  48. toSequence.kt
  49. typeConversions.kt
  50. unfold.kt
  51. update.kt (cast and toString methods?)
  52. valueCounts.kt
  53. values.kt

htmlReport.zip

We might want to start with them, see what really should be tested out of the functions in these files, add the tests, and later scan again looking for more subtle testing issues.

Lingua principale
Kotlin
Stelle
1.1k
Fork
87
Merge medio
2g 21h
PR unite (30g)
40

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

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 Kotlin/dataframe

Tutte le issue di Kotlin/dataframe

Issue simili

Altre issue su Kotlin

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.